operator-controller icon indicating copy to clipboard operation
operator-controller copied to clipboard

[epic] ClusterExtension Controller embeds helm for managing installed content

Open joelanford opened this issue 1 year ago • 2 comments

Brief

Tasks

  • [ ] #610

joelanford avatar Apr 04 '24 17:04 joelanford

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

varshaprasad96 avatar May 06 '24 17:05 varshaprasad96

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.

itroyano avatar May 06 '24 19:05 itroyano

This is done now.

tmshort avatar Jul 22 '24 15:07 tmshort