-
Notifications
You must be signed in to change notification settings - Fork 944
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
replace kubeval with kubeconform #3607
replace kubeval with kubeconform #3607
Conversation
Signed-off-by: Rui Chen <rui@chenrui.dev>
# Kubeval Version | ||
ARG KUBEVAL_VERSION='v0.16.1' | ||
# Kubeconform Version | ||
ARG KUBECONFORM_VERSION='v0.5.0' |
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.
Given that Kubeconform has a container image available, can you please refactor this PR to use that? By doing that, we can leverage Dependabot for updates. Thanks!
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.
is there any documentation specifying which is preferred? I personally would vote for binary though.
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.
Basically, if a container image is available, we try using that so that Dependabot takes care of updating that. As you can see, in the current implementation, the Kubeconform image would need to be manually updated.
Added in #3800 |
Closes #3496
Closes #2460
replacing kubeval with kubeconform as kubeval is no longer maintained.