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

Update max_ejection_percent on outlier detection for peered clusters to 100% #14373

Merged
merged 1 commit into from
Aug 29, 2022

Conversation

erichaberkorn
Copy link
Contributor

Description

We can't trust health checks when service resolvers, splitters, and routers are used for peered clusters.

PR Checklist

  • updated test coverage
  • external facing docs updated
  • not a security concern

@github-actions github-actions bot added the theme/envoy/xds Related to Envoy support label Aug 29, 2022
@erichaberkorn erichaberkorn force-pushed the update-outlier-detection-peered-clusters branch from d1d405b to 43c2f4d Compare August 29, 2022 14:49
@erichaberkorn erichaberkorn marked this pull request as ready for review August 29, 2022 15:07
@erichaberkorn erichaberkorn requested review from a team August 29, 2022 15:07
…to 100%

We can't trust health checks on peered services when service resolvers,
splitters and routers are used.
@erichaberkorn erichaberkorn force-pushed the update-outlier-detection-peered-clusters branch from 43c2f4d to a20aef7 Compare August 29, 2022 17:22
@erichaberkorn erichaberkorn merged commit 72f9075 into main Aug 29, 2022
@erichaberkorn erichaberkorn deleted the update-outlier-detection-peered-clusters branch August 29, 2022 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/envoy/xds Related to Envoy support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants