Skip to content

Updating API sha for proxy.#2181

Merged
utka merged 5 commits intoistio:release-1.1from
utka:release14
Apr 23, 2019
Merged

Updating API sha for proxy.#2181
utka merged 5 commits intoistio:release-1.1from
utka:release14

Conversation

@utka
Copy link
Contributor

@utka utka commented Apr 23, 2019

No description provided.

@utka utka requested a review from duderino April 23, 2019 17:39
@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Apr 23, 2019
@duderino
Copy link

Sha looks good. Ignoring the non-required tsan failure. Looks like all the tests pass but there's some cleanup issue:

Executed 0 out of 41 tests: 41 tests pass.
There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are.
INFO: Build completed successfully, 6 total actions
WARNING: The following rc files are no longer being read, please transfer their contents or import their path into one of the standard rc files:
/home/circleci/project/tools/bazel.rc
INFO: Invocation ID: 1bc0f771-1d67-45ce-8b4d-46c16aa7f826
WARNING: Waiting for server process to terminate (waited 5 seconds, waiting at most 60)
WARNING: Waiting for server process to terminate (waited 10 seconds, waiting at most 60)
WARNING: Waiting for server process to terminate (waited 30 seconds, waiting at most 60)
INFO: Waited 60 seconds for server process (pid=131) to terminate.
WARNING: Waiting for server process to terminate (waited 5 seconds, waiting at most 10)
WARNING: Waiting for server process to terminate (waited 10 seconds, waiting at most 10)
INFO: Waited 10 seconds for server process (pid=131) to terminate.
FATAL: Attempted to kill stale server process (pid=131) using SIGKILL, but it did not die in a timely fashion.
Makefile:58: recipe for target 'test_tsan' failed
make: *** [test_tsan] Error 36
Exited with code 2

/lgtm

@istio-testing
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: duderino, utka

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@utka utka merged commit 73fa9b1 into istio:release-1.1 Apr 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants