Skip to content

Update Envoy to f9301ff (Feb 26th 2021).#635

Merged
mum4k merged 3 commits intoenvoyproxy:mainfrom
mum4k:envoy-update
Feb 26, 2021
Merged

Update Envoy to f9301ff (Feb 26th 2021).#635
mum4k merged 3 commits intoenvoyproxy:mainfrom
mum4k:envoy-update

Conversation

@mum4k
Copy link
Copy Markdown
Collaborator

@mum4k mum4k commented Feb 26, 2021

  • Sync .bazelrc from Envoy's repository.
  • Sync ci/run_envoy_docker.sh from Envoy's repository.
  • Update docker image version in .circleci/config.yml.
  • The constructor of ClusterManagerFactory now requires an instance of Envoy::Server::Options. Satisfy it with Envoy::OptionsImpl::OptionsImpl (envoyproxy/envoy@42af11e).
  • An instance of Envoy::Event::Dispatcher now needs to be explicitly shut down on the main thread. (envoyproxy/envoy@114d5ae).
  • adjusting coverage threshold from 94.3 to 94.2.
  • fixing a clang_tidy issue in output_transform_main_test.cc.

Signed-off-by: Jakub Sobon mumak@google.com

Signed-off-by: Jakub Sobon <mumak@google.com>
@mum4k mum4k added the waiting-for-review A PR waiting for a review. label Feb 26, 2021
Signed-off-by: Jakub Sobon <mumak@google.com>
Signed-off-by: Jakub Sobon <mumak@google.com>
Copy link
Copy Markdown
Member

@oschaaf oschaaf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@mum4k mum4k merged commit 1d7c1c0 into envoyproxy:main Feb 26, 2021
@mum4k mum4k deleted the envoy-update branch February 26, 2021 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting-for-review A PR waiting for a review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants