Skip to content

Update Envoy to 36e0548 (Feb 24th 2021)#634

Closed
jiajunye wants to merge 3 commits intoenvoyproxy:mainfrom
jiajunye:envoy-update
Closed

Update Envoy to 36e0548 (Feb 24th 2021)#634
jiajunye wants to merge 3 commits intoenvoyproxy:mainfrom
jiajunye:envoy-update

Conversation

@jiajunye
Copy link
Copy Markdown
Contributor

@jiajunye jiajunye commented Feb 25, 2021

Update Nighthawk to the Envoy dependency

  • Sync .bazelrc from Envoy's version to update our build configurations.
  • Update ENVOY_COMMIT to the latest Envoy's commit and ENVOY_SHA in bazel/repositories.bzl.
  • Sync ci/run_envoy_docker.sh from Envoy's version.
  • Envoy updated the constructor of ClusterManagerFactory. So source/client/process_impl.cc needs to be updated accordingly.

Signed-off-by: jiajunye <jiajunye@google.com>
Signed-off-by: jiajunye <jiajunye@google.com>
@jiajunye
Copy link
Copy Markdown
Contributor Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

🔨 rebuilding ci/circleci: integration_test_coverage (failed build)
🔨 rebuilding ci/circleci: test (failed build)
🔨 rebuilding ci/circleci: tsan (failed build)
🔨 rebuilding ci/circleci: asan (failed build)
🔨 rebuilding ci/circleci: clang_tidy (failed build)
🔨 rebuilding ci/circleci: test_gcc (failed build)

🐱

Caused by: a #634 (comment) was created by @jiajunye.

see: more, trace.

@mum4k
Copy link
Copy Markdown
Collaborator

mum4k commented Feb 26, 2021

fyi, obsoleted by #635.

@mum4k mum4k closed this Feb 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants