Skip to content

feat(operator): add topologySpreadConstraints support to valkey-operator chart#195

Open
somaz94 wants to merge 1 commit into
valkey-io:mainfrom
somaz94:feat/operator-topologyspreadconstraints
Open

feat(operator): add topologySpreadConstraints support to valkey-operator chart#195
somaz94 wants to merge 1 commit into
valkey-io:mainfrom
somaz94:feat/operator-topologyspreadconstraints

Conversation

@somaz94

@somaz94 somaz94 commented Jun 24, 2026

Copy link
Copy Markdown

Adds an optional topologySpreadConstraints field to the valkey-operator chart's Deployment, resolving #188. It mirrors the existing nodeSelector / affinity / tolerations pattern, and values.yaml already carried a # TODO: topologySpreadConstraints placeholder. Default is [], so the default render is unchanged.

Validation (run locally):

  • helm lint ./valkey-operator: 0 failed
  • helm unittest ./valkey-operator: 14 passed (3 new cases: default-absent, single, multiple)
  • helm template default render is byte-identical to main (only the chart version label changed)

related: #188

@somaz94 somaz94 force-pushed the feat/operator-topologyspreadconstraints branch from e22d698 to 7cf216d Compare June 24, 2026 01:54
@somaz94 somaz94 marked this pull request as ready for review June 24, 2026 02:36
@Bloodraven21 Bloodraven21 added the chart:valkey-operator Related to the valkey-operator chart label Jun 24, 2026
@Bloodraven21

Copy link
Copy Markdown
Collaborator

please fix all the merge conflicts in all of your prs

…tor chart

Signed-off-by: somaz <genius5711@gmail.com>
@somaz94 somaz94 force-pushed the feat/operator-topologyspreadconstraints branch from 7cf216d to 933ffee Compare June 25, 2026 08:08
@somaz94

somaz94 commented Jun 25, 2026

Copy link
Copy Markdown
Author

@Bloodraven21 Thanks! Rebased onto the latest main and resolved the merge conflict. The only conflict was the chart version in valkey-operator/Chart.yaml — bumped it to 0.2.5 (next after the released 0.2.4), moved the CHANGELOG entry under a new ## 0.2.5 section, and updated the README version badge. helm lint passes and helm-unittest is green (25/25). The PR is mergeable again.

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

Labels

chart:valkey-operator Related to the valkey-operator chart

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants