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

Upgrade helm and clean up generated code/files under config #940

Open
12 tasks
unmarshall opened this issue Nov 27, 2024 · 0 comments
Open
12 tasks

Upgrade helm and clean up generated code/files under config #940

unmarshall opened this issue Nov 27, 2024 · 0 comments
Labels
area/open-source Open Source (community, enablement, contributions, conferences, CNCF, etc.) related area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/cleanup Something that is not needed anymore and can be cleaned up
Milestone

Comments

@unmarshall
Copy link
Contributor

unmarshall commented Nov 27, 2024

How to categorize this issue?

/area quality
/kind cleanup

What would you like to be added:

  • Upgrade helm to 3.16.4
  • Start using intrinsic support for crds in Helm3. Change the existing directory structure and remove sub-directory charts/druid and instead just have top level charts. Add a new sub-directory charts/crds where the crds will be directly placed. Helm3 will automatically install the crds first.
  • Remove config directory
  • Move out config/samples to top level examples directory
  • Remove charts/resources and provide a script to generate the tls-artifacts which can be used by consumers. These should not be checked into git repository but will become part of the OCI artifact once pushed.
  • Remove the existing charts/druid/charts/crds and charts/druid/requirements.yaml
  • create a script under hack/install-helm-scripts.sh to install helm charts which will copy crds, generate tls-artifacts. Now use helm utillities like Files.Glob to directly use these generate tls artifact files and encode them.
  • Introduce a make target for building helm charts which will call hack/install-helm-scripts.sh
  • Introduce a make target to push helm charts to the OCI registry.
  • Add README.md under charts which will create detailed instructions on how to use the druid helm charts.
  • Add templates/NOTES.txt which is a templatized file which will be rendered onto the console when you install a helm chart.
  • Publish helm charts to a public OCI registry as an OCI artifact. Update the documentation and include steps on how to use helm OCI artifacts from the public OCI registry. Refer: https://helm.sh/docs/topics/registries/

Why is this needed:
We only wish to support deploying etcd-druid via helm charts. We have generated code under config directory to enable usage of kustomise but we not neither used it nor actively maintained it. It therefore makes sense to completely remove it and simplify the code base.

@gardener-robot gardener-robot added area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/cleanup Something that is not needed anymore and can be cleaned up labels Nov 27, 2024
@shreyas-s-rao shreyas-s-rao added the area/open-source Open Source (community, enablement, contributions, conferences, CNCF, etc.) related label Jan 6, 2025
@shreyas-s-rao shreyas-s-rao added this to the v0.27.0 milestone Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/open-source Open Source (community, enablement, contributions, conferences, CNCF, etc.) related area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/cleanup Something that is not needed anymore and can be cleaned up
Projects
None yet
Development

No branches or pull requests

3 participants