Skip to content

Update envoy-wasm SHA#2885

Closed
Shikugawa wants to merge 3 commits intoistio:masterfrom
Shikugawa:update-envoy-sha-2
Closed

Update envoy-wasm SHA#2885
Shikugawa wants to merge 3 commits intoistio:masterfrom
Shikugawa:update-envoy-sha-2

Conversation

@Shikugawa
Copy link
Copy Markdown
Member

What this PR does / why we need it:

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Release note:

@Shikugawa Shikugawa requested a review from a team June 30, 2020 13:45
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 30, 2020
@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 Jun 30, 2020
@Shikugawa Shikugawa requested a review from a team June 30, 2020 14:14
@istio-testing
Copy link
Copy Markdown
Collaborator

@Shikugawa: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
test-tsan_proxy e917519 link /test test-tsan_proxy
test-asan_proxy e917519 link /test test-asan_proxy
release-test_proxy e917519 link /test release-test_proxy
test_proxy e917519 link /test test_proxy
check-wasm_proxy e917519 link /test check-wasm_proxy
Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan
Copy link
Copy Markdown
Contributor

Hi this needs some code change and I've already had a pending branch for this: bianpengyuan@6c31bc0. I am waiting for Lizan's change on stats prefix before creating a PR: envoyproxy/envoy#11808

@Shikugawa
Copy link
Copy Markdown
Member Author

Hi this needs some code change and I've already had a pending branch for this: bianpengyuan@6c31bc0. I am waiting for Lizan's change on stats prefix before creating a PR: envoyproxy/envoy#11808

@bianpengyuan Sure. But, this PR is not required anymore. Thank you for pointing out.

@Shikugawa Shikugawa closed this Jun 30, 2020
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. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants