Skip to content

feat: add Pod Disruption Budget support#429

Merged
mgruner merged 3 commits into
zammad:mainfrom
PabloCastellano:pdb
May 20, 2026
Merged

feat: add Pod Disruption Budget support#429
mgruner merged 3 commits into
zammad:mainfrom
PabloCastellano:pdb

Conversation

@PabloCastellano
Copy link
Copy Markdown
Contributor

  • Add podDisruptionBudget config to root level of values.yaml (disabled by default)
  • Create pdb.yaml template that generates PDB for all 4 app components
  • Use maxUnavailable: 1 to handle single-replica services gracefully
  • Enable PDB in CI test values and production deployment
  • Allows coordinated pod eviction during node maintenance/upgrades

What this PR does / why we need it

  • Allows configuring PodDisruptionBudget

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, …) format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer

  • .

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • Upgrading instructions are documented in the zammad/README.md

- Add podDisruptionBudget config to root level of values.yaml (disabled by default)
- Create pdb.yaml template that generates PDB for all 4 app components
- Use maxUnavailable: 1 to handle single-replica services gracefully
- Enable PDB in CI test values and production deployment
- Allows coordinated pod eviction during node maintenance/upgrades
Comment thread zammad/Chart.yaml Outdated
Co-authored-by: André Bauer <monotek@users.noreply.github.com>
monotek
monotek previously approved these changes May 19, 2026
Copy link
Copy Markdown
Member

@monotek monotek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mgruner

Not sure why this was missing at all?
Imho PDB should be enabled by default.

@mgruner mgruner merged commit 505e33a into zammad:main May 20, 2026
8 checks passed
@mgruner
Copy link
Copy Markdown
Collaborator

mgruner commented May 20, 2026

Thank you so much @PabloCastellano

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants