kpt icon indicating copy to clipboard operation
kpt copied to clipboard

[WIP] Declerative targeting of packages to clusters

Open mortent opened this issue 3 years ago • 3 comments

mortent avatar Sep 08 '22 17:09 mortent

A few questions / thoughts, but looks good. Merge and iterate :-)

justinsb avatar Sep 08 '22 18:09 justinsb

I agree with the merge and iterate approach, but I think we should come up with an approach where all the "very alpha" controllers doesn't run by default for users trying out Porch. We had some users reporting issues due to our renaming of some of the CRDs in the latest release. We could disable them in the regular porch release and just ask users who want to try them out to build from source. Or even better, publish porch as a kpt package an allow users to opt-in to these controllers if they want to.

mortent avatar Sep 09 '22 05:09 mortent

I agree with the merge and iterate approach, but I think we should come up with an approach where all the "very alpha" controllers doesn't run by default for users trying out Porch. We had some users reporting issues due to our renaming of some of the CRDs in the latest release. We could disable them in the regular porch release and just ask users who want to try them out to build from source. Or even better, publish porch as a kpt package an allow users to opt-in to these controllers if they want to.

I also think controllers with GCP-specific functionality need to be separated from the rest of the controllers and optionally deployed. Preferably in a single controller-manager. Alternatively they can be part of KCC, especially if they depend on KCC CRDs.

johnbelamaric avatar Sep 09 '22 21:09 johnbelamaric