Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v0.9] Keep CRDs when deleting a Bundle #2024

Merged
merged 3 commits into from Dec 20, 2023

Conversation

raulcabello
Copy link
Contributor

Just delete CRDs if deleteCRDResources is set to true in the fleet.yaml. Otherwise CRDs are kept in the cluster.

This PR adds the helm.sh/resource-policy annotation for CRDs if deleteCRDResources is false or not set. This will prevent helm from deleting CRDs when a Bundle is deleted.

Refers to #1978

@raulcabello raulcabello requested a review from a team as a code owner December 18, 2023 16:44
Just delete CRDs if deleteCRDResources is set to true in the fleet.yaml

Add the helm.sh/resource-policy annotation for CRDs if deleteCRDResources is false or not set. This will prevent helm from deleting CRDs when a Bundle is deleted

Signed-off-by: raul <raul.cabello@suse.com>
@raulcabello raulcabello changed the title Keep CRDs when deleting a Bundle [v0.9] Keep CRDs when deleting a Bundle Dec 19, 2023
Copy link
Contributor

@weyfonk weyfonk left a comment

Choose a reason for hiding this comment

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

Looking good, with a few suggestions :) Thanks!
Btw this feedback also applies to #2028.

charts/fleet-crd/templates/crds.yaml Outdated Show resolved Hide resolved
charts/fleet-crd/templates/crds.yaml Outdated Show resolved Hide resolved
charts/fleet-crd/templates/crds.yaml Outdated Show resolved Hide resolved
charts/fleet-crd/templates/crds.yaml Outdated Show resolved Hide resolved
internal/helmdeployer/deployer_test.go Outdated Show resolved Hide resolved
raulcabello and others added 2 commits December 19, 2023 15:10
Co-authored-by: Corentin Néau <tan.neau@suse.com>
Signed-off-by: raul <raul.cabello@suse.com>
@raulcabello raulcabello merged commit 1a500c4 into rancher:release/v0.9 Dec 20, 2023
9 checks passed
@raulcabello raulcabello deleted the crd-deletion branch December 20, 2023 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants