Skip to content

feat(cnpg): Adding topologySpreadConstraints support to the deployment #372

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

Merged
merged 2 commits into from
Apr 11, 2025

Conversation

elliotcourant
Copy link
Contributor

This adds support for specifying
topologySpreadConstraints for the cloudnative-pg operator deployment.

Resolves #371

@elliotcourant
Copy link
Contributor Author

Super small PR, hopefully I'm not missing anything? I was just following https://github.com/cloudnative-pg/charts/blob/main/CONTRIBUTING.md

Trying to get this working for a cluster I'm setting up, thank you very much!

@itay-grudev itay-grudev added the chart( operator ) Related to the operator (cloudnative-pg) chart label Sep 3, 2024
This adds support for specifying
[`topologySpreadConstraints`](https://kubernetes.io/docs/concepts/scheduling-eviction/topology-spread-constraints/)
for the cloudnative-pg operator deployment.

Resolves cloudnative-pg#371

Signed-off-by: Elliot Courant <me@elliotcourant.dev>
Signed-off-by: Itay Grudev <itay@verito.digital>
@itay-grudev itay-grudev self-requested a review as a code owner April 10, 2025 20:12
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 10, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 10, 2025
@itay-grudev itay-grudev merged commit 722b04e into cloudnative-pg:main Apr 11, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart( operator ) Related to the operator (cloudnative-pg) chart lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Topology spread constraints
2 participants