Skip to content

common: proto dep update#3372

Closed
lambdai wants to merge 4 commits intoistio:masterfrom
lambdai:protostatus
Closed

common: proto dep update#3372
lambdai wants to merge 4 commits intoistio:masterfrom
lambdai:protostatus

Conversation

@lambdai
Copy link
Copy Markdown
Contributor

@lambdai lambdai commented Jun 6, 2021

Find build error while I am working on github.com/istio/tools/pull/1565

The goal of this PR is fix known errors with github.com/istio/tools/pull/1565 as a proof of success of building kafka network filters.

We may need more PRs to update the proxy code base to accommodate envoyproxy.

Signed-off-by: Yuchen Dai lambdai@google.com

lambdai added 2 commits June 5, 2021 18:12
Signed-off-by: Yuchen Dai <lambdai@google.com>
Signed-off-by: Yuchen Dai <lambdai@google.com>
@lambdai lambdai requested review from a team June 6, 2021 18:17
@google-cla google-cla bot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Jun 6, 2021
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 6, 2021
Signed-off-by: Yuchen Dai <lambdai@google.com>
@lambdai
Copy link
Copy Markdown
Contributor Author

lambdai commented Jun 7, 2021

/retest

@lambdai
Copy link
Copy Markdown
Contributor Author

lambdai commented Jun 8, 2021

/test test_proxy

@lambdai
Copy link
Copy Markdown
Contributor Author

lambdai commented Jun 8, 2021

so far so good

[9,808 / 12,916] Compiling external/envoy/source/extensions/filters/network/kafka/external/kafka_response_resolver.cc; 35s remote-cache, linux-sandbox ... (126 actions, 75 running)

Signed-off-by: Yuchen Dai <silentdai@gmail.com>
@lambdai
Copy link
Copy Markdown
Contributor Author

lambdai commented Jun 8, 2021

/retest

@lambdai
Copy link
Copy Markdown
Contributor Author

lambdai commented Jun 8, 2021

/test test_proxy

@istio-testing
Copy link
Copy Markdown
Collaborator

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

Test name Commit Details Rerun command
test-tsan_proxy 1cd907c link /test test-tsan_proxy
test-asan_proxy 1cd907c link /test test-asan_proxy
test_proxy 1cd907c link /test test_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.

@lambdai lambdai closed this Jun 15, 2021
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.

3 participants