Skip to content

[epic] ClusterExtension Controller embeds helm for managing installed content #733

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

Closed
3 tasks done
joelanford opened this issue Apr 4, 2024 · 3 comments
Closed
3 tasks done
Assignees
Labels
epic v1.0 Issues related to the initial stable release of OLMv1

Comments

@joelanford
Copy link
Member

joelanford commented Apr 4, 2024

@joelanford joelanford added this to OLM v1 Apr 4, 2024
@joelanford joelanford self-assigned this Apr 4, 2024
@joelanford joelanford converted this from a draft issue Apr 4, 2024
@joelanford joelanford added epic priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. v1.0 Issues related to the initial stable release of OLMv1 and removed priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Apr 4, 2024
@joelanford joelanford moved this to Needs Design in OLM v1 Apr 4, 2024
@varshaprasad96 varshaprasad96 moved this from Needs Design to In Progress in OLM v1 May 6, 2024
@varshaprasad96
Copy link
Member

Brief up for review: https://docs.google.com/document/d/1GvmHQxjwCOS6xEZqWTCf5UoQB3DzdSMKIKfJZLiG3X0/edit#heading=h.btd8kkom5km4

@itroyano
Copy link
Contributor

itroyano commented May 6, 2024

Hi, 2.5 cents

The engine for applying a helm chart already exists in the helm-operator-plugin code, so the two main feats here in my opinion are:

  1. Define a generic and "friendly" structure for a bundle that represents a Helm chart, and customizable values for its installation.
    The values part is key. it needs to have good & clear UX, and there needs to be an option for the bundle to provide a validation schema and/or webhooks, such that invalid input can be rejected before running the actual engine.

  2. Convert the bundle to something the existing helm engine understands. a temporary CR that goes into the Unstructured parser code for example, or some other impl.

@tmshort
Copy link
Contributor

tmshort commented Jul 22, 2024

This is done now.

@tmshort tmshort closed this as completed Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic v1.0 Issues related to the initial stable release of OLMv1
Projects
Archived in project
Development

No branches or pull requests

4 participants