-
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: Allow configuring tolerations for the EG control plane installed via Helm #3587
Conversation
Signed-off-by: Connor Rogers <[email protected]>
Signed-off-by: Connor Rogers <[email protected]>
Looks like there's a flake in one of the CI jobs:
Could a maintainer please rerun this test? Thanks! |
you can reply with BTW, can you add a test for this new feature? |
add test for this under https://github.com/envoyproxy/gateway/tree/main/test/helm |
Signed-off-by: Connor Rogers <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3587 +/- ##
=======================================
Coverage 68.16% 68.16%
=======================================
Files 168 168
Lines 20408 20408
=======================================
Hits 13912 13912
Misses 5494 5494
Partials 1002 1002 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Connor Rogers <[email protected]>
/retest |
1 similar comment
/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 thanks !
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! Thanks for adding this!
What this PR does / why we need it:
Allows the EG operator to be scheduled on nodes with taints. For example, in our cluster, we have a core set of control plane nodes with a
NoSchedule
taint. This PR allows the EG operator to be scheduled on those nodes.Example Helm values.yaml:
This PR also fixes some indentation in the
topologySpreadConstraints
field for the Deployment template - it was inconsistently indented with the rest of the generated manifest.