Skip to content

feat(kminion): add opt-in topologySpreadConstraints to deployment and daemonset - #1754

Merged
RafalKorepta merged 1 commit into
redpanda-data:mainfrom
somaz94:feat/kminion-topology-spread-constraints
Jun 25, 2026
Merged

feat(kminion): add opt-in topologySpreadConstraints to deployment and daemonset#1754
RafalKorepta merged 1 commit into
redpanda-data:mainfrom
somaz94:feat/kminion-topology-spread-constraints

Conversation

@somaz94

@somaz94 somaz94 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Adds an opt-in topologySpreadConstraints field to the kminion chart's Deployment and DaemonSet, mirroring the existing knob in the sibling connect chart (charts/connect). It defaults to [], so the rendered output is unchanged unless a user sets it.

This rounds out kminion's scheduling knobs (it already exposes nodeSelector, affinity and tolerations).

Validation (run locally):

  • ct lint --charts charts/kminion: all charts linted successfully
  • actionlint: clean
  • helm template with default values: field is absent, render is byte-identical to before
  • helm template with a constraint set: renders correctly on both the Deployment and the DaemonSet
  • README regenerated with helm-docs v1.14.2

related: #1040

@CLAassistant

CLAassistant commented Jun 23, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@somaz94
somaz94 marked this pull request as ready for review June 23, 2026 09:40
@RafalKorepta
RafalKorepta merged commit 1cf3af0 into redpanda-data:main Jun 25, 2026
8 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.

4 participants