Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xds: sync envoy proto to commit 62ca8bd2b5960ed1c6ce2be97d3120cee719ecab #8346

Merged
merged 2 commits into from
Jul 27, 2021

Conversation

sergiitk
Copy link
Member

Sync to the latest update to pick up envoyproxy/envoy#16942 for forward compatibility with upcoming xDS Rate Limiting features.
Internal Envoy import CL for 62ca8bd2b5960ed1c6ce2be97d3120cee719ecab: cl/381356375

Suppressed warnings for newly deprecated xDS proto fields

  1. PerXdsConfig xds_config to be replaced with GenericXdsConfig generic_xds_configs, but this work yet to be planned by @lidizheng
  2. HttpConnectionManager's uint32 setXffNumTrustedHops to be replaced with TypedExtensionConfig OriginalIpDetectionExtensions. I don't know if there's any significance in testing that this exact field isn't supported. I can add a test for OriginalIpDetectionExtensions too.

@sergiitk sergiitk merged commit bf6db5a into grpc:master Jul 27, 2021
@sergiitk sergiitk deleted the envoy-proto-sync branch July 27, 2021 00:01
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants