camel-k icon indicating copy to clipboard operation
camel-k copied to clipboard

Should `make bundle` update kustomization.yaml?

Open squakez opened this issue 1 year ago • 3 comments

As discussed in https://github.com/apache/camel-k/pull/3601#discussion_r963234805 the make bundle is adding images property. Is that correct?

squakez avatar Sep 06 '22 08:09 squakez

Can you help @phantomjinx ?

squakez avatar Sep 06 '22 08:09 squakez

So, the changes should not be committed.

  1. The changes are transient to the kustomization config files
  2. A user is able to make use of these files without using our Makefiles by adding in their own references to the image and tag.

So the base version of the files remain under source control, while the changes come and go depending on user edits or executing make clean bundle. If I could gitignore them I would.

phantomjinx avatar Sep 06 '22 11:09 phantomjinx

Yes, I think so too. This is the most itchy part of kustomize for me. The kustomize developers has deliberately chosen that user-customised changes are directly applied to the kustomization.yaml file. And there seems to be no way to parameterise user configs other than tainting kustomization.yaml: https://kubectl.docs.kubernetes.io/faq/kustomize/eschewedfeatures/#build-time-side-effects-from-cli-args-or-env-variables

tadayosi avatar Sep 07 '22 04:09 tadayosi

Can this be closed as asked and answered?

phantomjinx avatar Oct 11 '22 13:10 phantomjinx