-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Description
Eschewed features
- This issue is not requesting templating, unstuctured edits, build-time side-effects from args or env vars, or any other eschewed feature.
What would you like to have added?
We add the warning message when the apiVersion
or kind
fields are omitted.
Why is this needed?
Today, kustomize
accept to omit the apiVersion
and kind
fields.
This syntax is easy to start for new users, but it isn't very clear and maybe break the user config when we have a few apiVersion
versions (Ex. when we add v1
for apiVersion
).
So, Could we consider deprecating this syntax that omits the apiVersion
and kind
fields?
I think we can start by showing the warning message and adding the auto-fix function. And we can make more consideration to whether remove or not.
Can you accomplish the motivating task without this feature, and if so, how?
Probably, we can decide to use current values (Ex. apiVersion: kustomize.config.k8s.io/v1beta1
) when that fields omitted.
These values were hardcoded on codes and we can announce for we require set these values when you need to use v1
and new functions.
But I think we need to add the warning message because we must remove the old apiVersion in the future.
What other solutions have you considered?
Probably, we have enough to add the warning message only.
But if we don't deprecate that, maybe user got to break their config when updating kustomize
.
Anything else we should know?
No response
Feature ownership
- I am interested in contributing this feature myself! 🎉