Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should make bundle update kustomization.yaml? #3606

Closed
squakez opened this issue Sep 6, 2022 · 4 comments
Closed

Should make bundle update kustomization.yaml? #3606

squakez opened this issue Sep 6, 2022 · 4 comments
Labels
kind/question Further information is requested

Comments

@squakez
Copy link
Contributor

squakez commented Sep 6, 2022

As discussed in #3601 (comment) the make bundle is adding images property. Is that correct?

@squakez
Copy link
Contributor Author

squakez commented Sep 6, 2022

Can you help @phantomjinx ?

@squakez squakez added the kind/question Further information is requested label Sep 6, 2022
@phantomjinx
Copy link
Contributor

phantomjinx commented Sep 6, 2022

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.

@tadayosi
Copy link
Member

tadayosi commented Sep 7, 2022

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

@phantomjinx
Copy link
Contributor

Can this be closed as asked and answered?

@squakez squakez closed this as completed Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants