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

Create helm chart for metrics-operator #2181

Closed
Tracked by #1618
mowies opened this issue Sep 27, 2023 · 0 comments · Fixed by #2189
Closed
Tracked by #1618

Create helm chart for metrics-operator #2181

mowies opened this issue Sep 27, 2023 · 0 comments · Fixed by #2189
Assignees
Labels
Milestone

Comments

@mowies
Copy link
Member

mowies commented Sep 27, 2023

Goal

Set up a standalone Helm chart for the metrics operator taking what is currently in the main KLT chart and moving it into a separate chart.

Details

  • the helm chart should live under metrics-operator/chart
  • the metrics-operator relevant helm values should be moved over to the new chart
  • CRDs that are specific to the metrics-operator should be moved to the new chart and removed from the old chart
  • change the default kubernetes labels so that they make more sense in the umbrella/subchart context
  • change the container names to something better than manager
  • adapt the integration test keptn install action to make sure that everything still works

DoD

  • new chart is created
  • new chart can be installed standalone and can work together with an existing KLT chart without the metrics-operator

Notes

Some inspiration can be taken from the initial PoC for the umbrella chart here, but that still works with helmify.

@mowies mowies changed the title create helm chart for metrics-operator Create helm chart for metrics-operator Sep 27, 2023
@mowies mowies added status: ready-for-refinement Issue is relevant for the next backlog refinment helm enhancement New feature or request metrics-operator labels Sep 27, 2023
@mowies mowies added this to the 0.9 milestone Sep 27, 2023
@RealAnna RealAnna self-assigned this Sep 27, 2023
@mowies mowies removed the status: ready-for-refinement Issue is relevant for the next backlog refinment label Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants