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

[Feature Request] helm chart for the operator #335

Open
gazal-k opened this issue Feb 10, 2023 · 3 comments
Open

[Feature Request] helm chart for the operator #335

gazal-k opened this issue Feb 10, 2023 · 3 comments
Assignees

Comments

@gazal-k
Copy link

gazal-k commented Feb 10, 2023

I believe it would be nice to have a helm chart for the operator. Happy to give a go at contributing. Thoughts?

@jsullivan-CS
Copy link
Contributor

jsullivan-CS commented Feb 10, 2023

Hi Gazal,

We agree that there should be a helm chart for the operator; we're currently in the planning stages of creating one, we're just awaiting some input from the CrowdStrike Product team regarding some design decisions.

We're always happy to accept contributions, but in this case it may make sense to hold off a bit as we may have some design constraints that we'll need to account for. I'll try and update you with an ETA once we've been able to nail a few things down.

Thank you for your request!

@jsullivan-CS jsullivan-CS self-assigned this Feb 10, 2023
@redhatrises
Copy link
Contributor

There are some issues to be aware of here around CRDs and helm that make helm less than ideal

  1. You cannot upgrade CRDs.
  2. You cannot delete CRDs.

There are also other problems that make creating a helm chart less than ideal.

@gazal-k
Copy link
Author

gazal-k commented Feb 18, 2023

There are ways around the conventional helm chart structure to install/upgrade CRDs. There are a few charts that does it too. This one is probably a good reference: https://github.com/external-secrets/external-secrets/tree/main/deploy/charts/external-secrets/templates/crds

And that's if every helm chart revision is accompanied by a change in the CRD spec, changes to the operator image version / deployment (& pod) spec, RBAC etc can easily be managed using a helm chart.

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

3 participants