Repository navigation
Use 2 chart for upgrading the crds using helm (backport #234) - #235
Merged
Merged
Conversation
CRDs are shipped in a separate sub-chart (bootc-operator-crds) and declared as a Helm dependency of the operator chart. This keeps CRDs in templates/ so helm upgrade can update them, while preserving install/upgrade ordering automatically. The dependency is conditional on crds.install (default true), allowing users who manage CRDs separately to disable it. Assisted-by: AI Signed-off-by: Alice Frosi <afrosi@redhat.com> (cherry picked from commit 7ab222f)
Publish the CRD chart to the OCI registry before the operator chart. Run helm dependency build for the operator so the CRD sub-chart is bundled into the package. Sync the dependency version at release time. Assisted-by: AI Signed-off-by: Alice Frosi <afrosi@redhat.com> (cherry picked from commit 72f9c2f)
Document single-chart install and upgrade using the CRD dependency. Include instructions for managing CRDs separately with crds.install=false. Assisted-by: AI Signed-off-by: Alice Frosi <afrosi@redhat.com> (cherry picked from commit 9746dce)
Assisted-by: AI Signed-off-by: Alice Frosi <afrosi@redhat.com> (cherry picked from commit 18e350f)
alicefr
approved these changes
Oct 16, 2026
alicefr
enabled auto-merge (rebase)
October 16, 2026 06:09
alicefr
approved these changes
Oct 16, 2026
Merged
HarshwardhanPatil07
pushed a commit
that referenced
this pull request
Oct 16, 2026
Bump the release to 0.2.1. Since, we merged #235 which contains the 2 helm chart to separate the crds from the operator deployment, we should create a new release which contains the 2 charts. This will also allows us to start testing the upgrade using the helm chart instead of the raw manifest in the test TestOperatorUpgrade. Signed-off-by: Alice Frosi <afrosi@redhat.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Helm doesn't upgrade or delete the crds (see the official documentation),. In the official documentation, one of the method to solve this is to create a separte chart for the crds, in this way the can be installed and upgrade using helm. In particular, we are interested in the fact that the crds are released as oci artifact.
Additionally, the documentation now also covers the upgrade and uninstall paths
This is an automatic backport of pull request #234 done by Mergify.