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

CRDs Part of Chart #6

Open
abebars opened this issue Jun 6, 2024 · 2 comments
Open

CRDs Part of Chart #6

abebars opened this issue Jun 6, 2024 · 2 comments

Comments

@abebars
Copy link

abebars commented Jun 6, 2024

Can the CRDs become part of the helm chart to make the installation process easier?

@smuneebahmad
Copy link
Collaborator

Hi @abebars , thank you for filing this request. We agree that including CRDs in the Helm Chart simplifies the installation process, but unfortunately, it complicates the upgrade and deletion processes. We made a conscious decision to exclude CRDs from the Helm Chart because Helm currently does not support upgrading or deleting CRDs. This decision followed extensive community discussion due to the risk of unintentional data loss. Unfortunately, there is no community consensus on handling CRDs and their lifecycle at this time [1]

An alternative approach is to create a separate Helm Chart for the CRDs. If we offer this method, the CRD Helm Chart would need to be installed separately.

Please let us know if you would still like to see CRDs in the Helm Chart. We would be happy to consider this as an optional feature (disabled by default).

[1] https://helm.sh/docs/chart_best_practices/custom_resource_definitions/#some-caveats-and-explanations

@abebars
Copy link
Author

abebars commented Jun 26, 2024

Hi @abebars , thank you for filing this request. We agree that including CRDs in the Helm Chart simplifies the installation process, but unfortunately, it complicates the upgrade and deletion processes. We made a conscious decision to exclude CRDs from the Helm Chart because Helm currently does not support upgrading or deleting CRDs. This decision followed extensive community discussion due to the risk of unintentional data loss. Unfortunately, there is no community consensus on handling CRDs and their lifecycle at this time [1]

An alternative approach is to create a separate Helm Chart for the CRDs. If we offer this method, the CRD Helm Chart would need to be installed separately.

Please let us know if you would still like to see CRDs in the Helm Chart. We would be happy to consider this as an optional feature (disabled by default).

[1] https://helm.sh/docs/chart_best_practices/custom_resource_definitions/#some-caveats-and-explanations

@smuneebahmad Yeah I think there is a benefit of having the CRDs as optional in the Helm chart to simplify the installation

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

No branches or pull requests

2 participants