-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Remove --strict from helm linter #36996
Conversation
/test release-notes_istio |
just want a sec to review before automerge |
cc @ericvn any reason why you are removing this? Failing on lint warnings seems like the behavior we'd want, but maybe there's additional things I'm missing. |
@jacob-delgado Reason it's failing is that we moved up Helm versions in the build image. Separate Slack with John as well. Not sure if it makes sense to remove the --strict, update the manifests to fix warnings (will this break us on some older supported kk8s versions, or revert the Helm update. |
Ah its from deprecated APIs... we cannot remove those yet |
Removing DNM as John approved. Still waiting on @istio/wg-environments-maintainers . |
Please provide a description of this PR:
The Helm version was updated in the tooling, so there are now some WARNINGS (which fail because of --strict) on deprecated APIs.
The
make gen
also makes one other manifest change.To help us figure out who should review this PR, please put an X in all the areas that this PR affects.