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.

Sign up for GitHub

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

Jump to bottom

feat: add controller.topologySpreadConstraints #1133

Merged

timja merged 2 commits into jenkinsci:main from msoderberg:feat/controller-topology-spread

Jun 19, 2024

Contributor

msoderberg commented Jun 19, 2024 •

edited

Loading

What does this PR do?

We host multiple jenkins controllers in one of our EKS clusters and we need to be able to spread the controller pods across topology domains.

Submitter checklist

Beta Give feedback

I bumped the "version" key in `./charts/jenkins/Chart.yaml`.

I bumped the "version" key in ./charts/jenkins/Chart.yaml.
Options
Successfully updated the issue's project

There was an error updating the issue's project
I added a new changelog entry to `./charts/jenkins/CHANGELOG.md`.

I added a new changelog entry to ./charts/jenkins/CHANGELOG.md.
Options
Successfully updated the issue's project

There was an error updating the issue's project
I followed the [technical requirements](https://github.com/jenkinsci/helm-charts/blob/main/CONTRIBUTING.md#technical-requirements).

I followed the technical requirements.
Options
Successfully updated the issue's project

There was an error updating the issue's project
I ran `.github/helm-docs.sh` from the project root.

I ran .github/helm-docs.sh from the project root.
Options
Successfully updated the issue's project

There was an error updating the issue's project
Options

Special notes for your reviewer


          feat: add controller.topologySpreadConstraints

8fe73fc

msoderberg requested a review from a team as a code owner

June 19, 2024 11:51

pashcan reviewed

View reviewed changes

charts/jenkins/CHANGELOG.md Outdated Show resolved Hide resolved


          chore: update CHANGELOG.md

53995d1

Co-authored-by: pashcan <pashcan.dev@gmail.com>

timja approved these changes

View reviewed changes

timja merged commit f069a63 into jenkinsci:main

6 checks passed

msoderberg deleted the feat/controller-topology-spread branch

June 19, 2024 13:37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment