-
-
Notifications
You must be signed in to change notification settings - Fork 462
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
[FEATURE] Config file compatible with Kustomize #945
Conversation
@all-contributors please add @erikgb for code |
I've put up a pull request to add @erikgb! 🎉 |
Move name to metadata.name in Simple kind
fc8f2ac
to
a4bc1b6
Compare
Hmmm, I am not sure if I understand why the e2e-test is failing. Maybe you can assist me a bit, @iwilltry42? 🙏 |
Hi @erikgb , thank you for this PR, great! :) |
Thanks @iwilltry42! K3d is such a great tool, and you are doing an excellent job as a maintainer! ❤️
Ok, good to hear. I couldn't find find an apparent reason for the failure, but it might be something I missed. Note: I was unable to run the e2e-test locally. Is there some special setup that needs to be done? This is the log for an attempt:
|
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.
Great! Very clean, love it 👍
Thanks especially for the test enhancement :)
Thank you very much for your contribution and the positive feedback 😃 |
What
Move
name
tometadata.name
ink3d.io/v1alpha4 Simple
config file format.Why
Fixes #937
This change in config format allow users to kustomize cluster-config using
kustomize
.Implications
This is a breaking change in the
v1alpha4
config-format, but this is a new and currently unreleased version.