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

Add extensible `values` field to Extension spec

Open oceanc80 opened this issue 1 year ago • 1 comments

In scope:

  • Registry+v1 bundles only
  • watchNamespaces
  • Stretch: subscription.spec.config

oceanc80 avatar Feb 01 '24 16:02 oceanc80

I think we should hold off on making watchNamespaces a first-class field in the Extension API.

Instead, I think we should implement a generic values-style field in the Extension API (similar to Carvel's PackageInstall.spec.values), which we would pass down into the App CR.

For registry+v1 bundles, we will control the YTT template, so we can implement that template to handle a watchNamespaces value if it is defined.

cc @varshaprasad96

joelanford avatar Feb 06 '24 15:02 joelanford

I implemented a proof of concept as part of https://github.com/operator-framework/operator-controller/pull/1418. We should revisit that PR when we have time to pick this issue up.

joelanford avatar Jan 21 '25 14:01 joelanford

replaced by #1750

perdasilva avatar Feb 12 '25 09:02 perdasilva