-
Notifications
You must be signed in to change notification settings - Fork 157
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
autoscaling.apiVersion #189
Conversation
Thanks for the update! What I usually do, however, is to enable automatic version detection, e.g: {{- if semverCompare ">=1.14-0" .Capabilities.KubeVersion.GitVersion -}}
apiVersion: networking.k8s.io/v1
{{- else -}}
apiVersion: extensions/v1beta1
{{- end }} This way it makes it easier to deploy charts to different versions of Kubernetes without changes to If you're willing to make this change, please do let me know. Otherwise I will close this PR and make the change myself. Thanks. |
@bokysan fixed |
Huh, tests do not go through. We will need to manually add the new schema to the repo as well to make the tests pass. |
https://github.com/gruntwork-io/terratest did You mentioned this test framework for helm? |
Nope. Check the executed tests. The code runs through several tests and there resulting schema is validated via kubeval. Unfortunately, there's no JSON schema for |
@bokysan ok, will take a look tomorrow |
I've pulled this in -- I will add the appropriate json file myself. |
@bokysan thank you! Sorry for not editing pr, i was too busy last days ( |
No prob. I know how it is. |
apiVersion: "autoscaling/v2beta1" is deprecated, adding value for overriding api version, fore example apiVersion: autoscaling/v2