Skip to content

feat(opentelemetry-collector): support PodDisruptionBudget for statefulset mode#2113

Merged
TylerHelmuth merged 2 commits into
open-telemetry:mainfrom
nedecz:feat/issue-1963-add-pdb-to-statefulset-in-the-collector-chart
Apr 17, 2026
Merged

feat(opentelemetry-collector): support PodDisruptionBudget for statefulset mode#2113
TylerHelmuth merged 2 commits into
open-telemetry:mainfrom
nedecz:feat/issue-1963-add-pdb-to-statefulset-in-the-collector-chart

Conversation

@nedecz
Copy link
Copy Markdown
Contributor

@nedecz nedecz commented Mar 12, 2026

Extends the PodDisruptionBudget template to support statefulset mode in addition to deployment mode.

Previously, enabling podDisruptionBudget had no effect when running the collector as a StatefulSet. The condition in pdb.yaml only matched deployment mode, which meant statefulset users could not protect their pods from voluntary disruptions.

Existing issue:
#1963

@nedecz nedecz requested review from a team, TylerHelmuth, dmitryax and povilasv as code owners March 12, 2026 17:03
@nedecz nedecz changed the title feat(open-telemetry-collector): support PodDisruptionBudget for statefulset mode feat(opentelemetry-collector): support PodDisruptionBudget for statefulset mode Mar 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 3, 2026

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Apr 3, 2026
Previously, PodDisruptionBudget was only supported when mode was set to
"deployment". This change extends PDB support to "statefulset" mode as well.

Closes open-telemetry#1963
@nedecz nedecz force-pushed the feat/issue-1963-add-pdb-to-statefulset-in-the-collector-chart branch from 271c56c to 560a65a Compare April 12, 2026 09:09
@nedecz
Copy link
Copy Markdown
Contributor Author

nedecz commented Apr 12, 2026

Hi, @TylerHelmuth

There were some merge conflicts, I fixed them.

@github-actions github-actions Bot removed the Stale label Apr 13, 2026
@TylerHelmuth TylerHelmuth merged commit c738eda into open-telemetry:main Apr 17, 2026
4 checks passed
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.

2 participants