Skip to content

fix(flipt-v2): indent volumeClaimTemplates labels and annotations - #303

Merged
kodiakhq[bot] merged 1 commit into
flipt-io:mainfrom
brodin:fix-flipt-v2-sts-pvc-metadata-indent
Oct 5, 2026
Merged

kodiakhq[bot] merged 1 commit into
flipt-io:mainfrom
brodin:fix-flipt-v2-sts-pvc-metadata-indent

Conversation

@brodin

@brodin brodin commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

With kind: StatefulSet and persistence.enabled, setting persistence.labels or persistence.annotations makes the StatefulSet unrenderable for the API server. The template indents their contents at the same level as the labels: / annotations: keys:

  volumeClaimTemplates:
    - metadata:
        name: flipt-data
        labels:
        app: flipt
        team: platform

so the keys become siblings of metadata, and the apply fails:

StatefulSet in version "v1" cannot be handled as a StatefulSet: strict decoding error: unknown field "spec.volumeClaimTemplates[0].metadata.app", unknown field "spec.volumeClaimTemplates[0].metadata.team"

This moves both to nindent 10. Rendered with kind: StatefulSet, persistence.enabled: true and a label and annotation set, the StatefulSet now passes a server-side dry run on GKE 1.35; before the change the same values were rejected as above. The Deployment path and the standalone PVC template are unchanged.

Chart version is bumped to 2.13.2.

With `kind: StatefulSet` and `persistence.enabled`, `persistence.labels` and `persistence.annotations` were rendered at the same indentation as their `labels:` / `annotations:` keys, so they became sibling fields of `metadata` and the API server rejected the StatefulSet with `strict decoding error: unknown field "spec.volumeClaimTemplates[0].metadata.<key>"`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Johan Brodin <johan.brodin@gmail.com>
@brodin
brodin marked this pull request as ready for review October 5, 2026 13:51
@brodin
brodin requested a review from a team as a code owner October 5, 2026 13:51
@erka erka added the automerge label Oct 5, 2026

@erka erka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @brodin

@kodiakhq
kodiakhq Bot merged commit 11349f5 into flipt-io:main Oct 5, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants