gitops-catalog icon indicating copy to clipboard operation
gitops-catalog copied to clipboard

add policy-generator component

Open strangiato opened this issue 1 year ago • 6 comments

This adds a new component for patching the argocd instance to enable the ACM policy generator plugin.

Since this component has a conflict with the kustomize-build-enable-helm component I have added some notes on both to indicate that they are not compatible with each other.

strangiato avatar Oct 11 '23 16:10 strangiato

@adetalhouet @gnunn1 this is awaiting your reviews :)

pittar avatar Nov 01 '23 14:11 pittar

Actually, thinking more about this; the kustomize plugins are versioned with ACM release. Shouldn't we have an overlay structure to account for the different ACM releases. Also, shouldn't this patch be part of the ACM sub-folder. Thoughts?

adetalhouet avatar Nov 22 '23 14:11 adetalhouet

I don't know what the official ACM policy is on this, technically there is no issue using the latest and greatest since it is completely decoupled from ACM and brings the benefit of having the latest version available with the newest features.

gnunn1 avatar Nov 22 '23 18:11 gnunn1

Otherwise LGTM

gnunn1 avatar Nov 22 '23 18:11 gnunn1

@gnunn1

technically there is no issue using the latest and greatest since it is completely decoupled from ACM

I'm not as sure as you are on this. Given the plugin generates CR out of CRDs, and the CRDs are coupled to ACM release cycle, I do foresee potential disconnect if using the latest plugin with say ACM 2.3 release. (but that concern is more theoretical than something I actually faced).

adetalhouet avatar Nov 22 '23 18:11 adetalhouet

I'm not as sure as you are on this. Given the plugin generates CR out of CRDs, and the CRDs are coupled to ACM release cycle, I do foresee potential disconnect if using the latest plugin with say ACM 2.3 release. (but that concern is more theoretical than something I actually faced).

That's a fair point, thanks for pointing that out.

gnunn1 avatar Nov 22 '23 18:11 gnunn1

README.md should be corrected as in sections "Purpose" and "Known Incompatibilities" it looks like copied from kustomize-build-enable-helm component README.md file. Probably the latter was used as a template.

bemarek avatar Aug 05 '24 08:08 bemarek