sync from master. #71
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…mit hook (#16505) Signed-off-by: Justin Ely <justincely@gmail.com>
Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Signed-off-by: luckyxiaoqiang <xiaoqiangding@126.com>
The 'primary_' and element in 'replicas_' were `shared_ptr`s, this shared_ptr is created from different redis responses, so it would always returns false. Signed-off-by: gaoliangdut <gaoliang_dlut@126.com>
Risk Level: low (removing unused args) Testing: n/a Docs Changes: n/a Release Notes: n/a Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Ryan Northey <ryan@synca.io>
Risk Level: High (new tcp pool, lots of smoke testing) Testing: integration tests, unit tests, production validation by Pinterest Docs Changes: n/a Release Notes: inline Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Making it a bit easier to mix and match protocols and doing so more cleanly for the H3/H2 alpn tests. Risk Level: n/a (test only) Testing: tests pass Docs Changes: n/a Release Notes: n/a Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Commit Message: these tests fails under TSAN CI, see #16493 Signed-off-by: Dan Zhang <danzh@google.com>
…reamClusterManager (#16385) grid: Plumb the AlternateProtocolCache down to the grid from the UpstreamClusterManager. Create a new AlternateProtocolsCacheOptions proto message for configuring an AlternateProtocolCache, and add this message to AutoHttpConfig. Create a new AlternateProtocolCacheManager class for fetching/creating a Cache based on the config. Modify the AlternateProtocolCache to store state per thread. (Does not yet synchronize this state across threads) Risk Level: Low Testing: Unit tests Docs Changes: N/A Release Notes: N/A Platform Specific Features: N/A Signed-off-by: Ryan Hamilton <rch@google.com>
Commit Message: fix mac build with xcode 12 Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Risk Level: n/a Testing: new integration tests Docs Changes: n/a Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Enable `is_optional` field for `HttpFilter`. The default value is `false`, it will keep the same behavior with the current implementation. The envoy will reject the unsupported HTTP filter, also will reject the unsupported HTTP filter in typed per filter config. When the value is `true`, the unsupported HTTP filter will be ignored by the envoy, also will be ignored by typed per filter config, with a warning log. Risk Level: low Testing: unit tests and integration tests are added Docs Changes: API doc is added Release Notes: added as new feature Fixes #15770 Fixes #15025 Signed-off-by: He Jie Xu <hejie.xu@intel.com>
…gration_test under coverage (#16534) Signed-off-by: Dan Zhang <danzh@google.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]