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

Sds backport #281

Merged
merged 2 commits into from
Nov 2, 2020
Merged

Sds backport #281

merged 2 commits into from
Nov 2, 2020

Conversation

lizan
Copy link

@lizan lizan commented Oct 30, 2020

Cherry-pick the following commits for istio/istio#22443

0d15fee (cluster manager: avoid immediate activation for dynamic inserted cluster when initialize (envoyproxy#12783), 2020-10-16)
e5a8c21 (sds: keep warming when dynamic inserted cluster can't be extracted secret entity (envoyproxy#13344), 2020-10-28)

…ter when initialize (envoyproxy#12783)

Signed-off-by: Shikugawa <rei@tetrate.io>
…cret entity (envoyproxy#13344)

This PR highly depends on envoyproxy#12783. Changed to keep warming if dynamic inserted clusters (when initialize doesn't finished) failed to extract TLS certificate and certificate validation context. They shouldn't be indicated as ACTIVE cluster.
Risk Level: Mid
Testing: Unit
Docs Changes:
Release Notes: Added
Fixes envoyproxy#11120, future work: envoyproxy#13777

Signed-off-by: Shikugawa <rei@tetrate.io>
@google-cla
Copy link

google-cla bot commented Oct 30, 2020

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@Shikugawa
Copy link
Member

@googlebot I consent.

@google-cla google-cla bot added cla: yes and removed cla: no labels Nov 2, 2020
@lizan lizan requested a review from bianpengyuan November 2, 2020 06:38
@bianpengyuan
Copy link

/retest

@istio-testing istio-testing merged commit 4bb1cfe into istio:release-1.8 Nov 2, 2020
bianpengyuan added a commit to bianpengyuan/envoy that referenced this pull request Nov 5, 2020
istio-testing pushed a commit that referenced this pull request Nov 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants