Skip to content
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

Merged
merged 2 commits into from
Apr 21, 2024
Merged

autoscaling.apiVersion #189

merged 2 commits into from
Apr 21, 2024

Conversation

shcherbak
Copy link
Contributor

apiVersion: "autoscaling/v2beta1" is deprecated, adding value for overriding api version, fore example apiVersion: autoscaling/v2

@bokysan
Copy link
Owner

bokysan commented Mar 27, 2024

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 values.yaml.

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.
B

@shcherbak
Copy link
Contributor Author

@bokysan fixed

@bokysan
Copy link
Owner

bokysan commented Mar 28, 2024

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.

@shcherbak
Copy link
Contributor Author

https://github.com/gruntwork-io/terratest did You mentioned this test framework for helm?

@bokysan
Copy link
Owner

bokysan commented Mar 29, 2024

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 autoscaling/v2 online (and depending on online capabilities is bad if you want to keep your tests independent anyhow) so a schema needs to be uploaded into corresponding folder.

@shcherbak
Copy link
Contributor Author

@bokysan ok, will take a look tomorrow

@bokysan bokysan merged commit 55f2f71 into bokysan:master Apr 21, 2024
3 of 4 checks passed
@bokysan
Copy link
Owner

bokysan commented Apr 21, 2024

I've pulled this in -- I will add the appropriate json file myself.

@shcherbak
Copy link
Contributor Author

@bokysan thank you! Sorry for not editing pr, i was too busy last days (

@bokysan
Copy link
Owner

bokysan commented Apr 23, 2024

No prob. I know how it is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants