-
Notifications
You must be signed in to change notification settings - Fork 363
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
feat: Enable the configuration of topologySpreadConstraints for the EG control plane. #3535
feat: Enable the configuration of topologySpreadConstraints for the EG control plane. #3535
Conversation
…oxy control plane.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3535 +/- ##
==========================================
+ Coverage 68.12% 68.15% +0.03%
==========================================
Files 165 165
Lines 20122 20122
==========================================
+ Hits 13708 13715 +7
+ Misses 5416 5411 -5
+ Partials 998 996 -2 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Alexander Volchok <[email protected]>
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
/retest to kick DCO |
DCO is broken in the server side, more in dcoapp/app#211 |
What this PR does / why we need it:
Given that Envoy Proxy is deployed with leader election enabled by default, it makes sense to provide an easy option to allow it to run across multiple availability zones to achieve a high availability (HA) deployment. One option currently available to achieve this is using pod affinity; however, adding topologySpreadConstraints offers even greater flexibility.
Fixes #
#3534