fix: Add capability to skip crd installation during helm install

estenrye commented 2 months ago

This PR

Alters the Makefile to move CustomResourceDefinition templates to the crds directory before packaging the helm chart.
Alters the .gitignore to ignore templates in the crds directory.
Updates and cleans up the CRD installation documentation.
Enables users to optionally skip installation of the CRDs using the --skip-crds flag when performing helm install

Related Issues

Fixes #624

Notes

Follow-up Tasks

How to test

make helm-package
helm template ./chart/open-feature-operator -s "templates/crds/*.yaml" | kubectl -f -
helm install open-feature ./chart/open-feature-operator --skip-crds --create-namespace -n open-feature

estenrye commented 2 months ago

@beeme1mr I ran the update script like the helm docs linter told me to. can you approve the workflows again?

estenrye commented 2 months ago

@beeme1mr I merged main into my branch since it was ahead of my branch, can you approve the workflow again?

beeme1mr commented 2 months ago

Hey @estenrye , I'll try and trigger the ci when I notice a change but feel free to ping me too. This is only an issue for first-time collaborators. Your next pr will be easier 🙂

estenrye commented 2 months ago

@beeme1mr can you re-run the workflows? I addressed PR feedback, rebased against main and squashed my commits.

open-feature / open-feature-operator