-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Finish goplugins (release blocker) #881
Comments
I can open some more PR's to add the rest of this. |
Nice. Now that you knocked off literals builtin, can you do env and files too? And add add some tests like this : https://github.com/kubernetes-sigs/kustomize/blob/master/pkg/target/generatoroptions_test.go#L97 Thanks! |
Ok working on it, also working on |
ah thanks - i'm out :) |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
The goplugin in |
Per the kep we need some more work goplugins started in #760
Release blockers:
kustomize reads config (plugins specifically) from (a subdir of)
$HOME/.config
, and should honorXDG_CONFIG_HOME
(see spec) as an override to this location.need flag
--alpha_enable_goplugin_and_accept_panic_risk
and associated behavior. The help for this flag should explain the problem.Nice to haves:
the existing three forms of KV generation (env, literals, files) should be expressible as
pluginType:builtin
.the existing
edit fix
command, which reads a kustomize file, updates it to new behavior, and writes it out again, should be modified to convert the three existing forms to builtin plugin form.The text was updated successfully, but these errors were encountered: