Skip to content

docs: add Helm ownership labeling step to CRD migration guide#245

Merged
Shivs11 merged 1 commit intomainfrom
docs/crd-migration-helm-ownership
Mar 25, 2026
Merged

docs: add Helm ownership labeling step to CRD migration guide#245
Shivs11 merged 1 commit intomainfrom
docs/crd-migration-helm-ownership

Conversation

@Shivs11
Copy link
Member

@Shivs11 Shivs11 commented Mar 25, 2026

Summary

  • Adds a required kubectl label + kubectl annotate step to the CRD migration instructions
  • Without this, helm install of the new CRDs chart fails with "cannot be imported into the current release: invalid ownership metadata"
  • Discovered during demo setup when migrating from previously-installed CRDs to the new charts format

Extracted from #217 per this comment — this needs to land independently in the chart version where CRDs are first split out.

Test plan

  • Verify the migration steps work on a cluster with pre-existing CRDs installed via the old crds/ directory

🤖 Generated with Claude Code

Without stamping Helm ownership labels/annotations onto existing CRDs
before running `helm install`, the install fails with "invalid ownership
metadata". This adds the required kubectl label/annotate commands as a
new step in the migration instructions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Shivs11 Shivs11 requested review from a team and jlegrone as code owners March 25, 2026 21:13
@Shivs11 Shivs11 enabled auto-merge (squash) March 25, 2026 21:15
@Shivs11 Shivs11 merged commit 8bc3cdb into main Mar 25, 2026
16 checks passed
@Shivs11 Shivs11 deleted the docs/crd-migration-helm-ownership branch March 25, 2026 21:23
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