-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Localize legacy fields #4933
Localize legacy fields #4933
Conversation
kustomization := map[string]string{ | ||
// We test that `kustomize localize` ignores helm charts. The code neither | ||
// accesses the referenced file, which does not exist, nor copies it to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's discuss this with @yuwenma and @KnVerey after the holidays. I agree that we should not handle localization of helm fields in alpha, but silently ignoring it doesn't seem to be the right thing to do. I think our options here are to either copy it over as-is, throw a warning, or throw an error if we see use of the helm field.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After discussing offline, @natasha41575 and I concluded that this PR can be merged without addressing helm
fields, as the PR doesn't make any changes specific to helm
. We'll discuss the particulars after the holidays; in the meantime, I'll try to copy local files to the localize
destination.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
One minor point of discussion that we should resolve before release.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: annasong20, natasha41575 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
2d991a7
to
3022dd2
Compare
/lgtm |
This PR localizes the legacy kustomization fields, loaded in #4918.