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

fix HPA api version #117

Merged
merged 11 commits into from
Jun 12, 2024
Merged

fix HPA api version #117

merged 11 commits into from
Jun 12, 2024

Conversation

Noamshmueli
Copy link
Contributor

fix HPA api version to autoscaling/v2

@pmint93
Copy link
Owner

pmint93 commented May 27, 2024

@Noamshmueli To keep this chart compatible with older k8s version, I suggest it detect kubernetes version and use the right apiVersion. #16 pointed out that we can use semverCompare function and .Capabilities.KubeVersion.GitVersion to do so

@Noamshmueli
Copy link
Contributor Author

@pmint93 I got you and look into it but seems like autoscaling/v2 is the stable one so we can set it hradcoded

@Noamshmueli Noamshmueli force-pushed the bugfix/hpa-version branch 2 times, most recently from 003a190 to 31091bd Compare June 4, 2024 12:37
@pmint93
Copy link
Owner

pmint93 commented Jun 5, 2024

@pmint93 I got you and look into it but seems like autoscaling/v2 is the stable one so we can set it hradcoded

of course it is stable version, but what I mean is: autoscaling/v2 is available since v1.23, so if user running version older than v1.23 (eg: v1.22), the installation will fail

@Noamshmueli
Copy link
Contributor Author

Done @pmint93

@pmint93 pmint93 merged commit c19994c into pmint93:master Jun 12, 2024
1 check passed
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