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) properly detect hpa capabilities, release 2.16.4 #744

Merged
merged 2 commits into from
Feb 14, 2023

Conversation

slavogiez
Copy link
Contributor

@slavogiez slavogiez commented Feb 14, 2023

What this PR does / why we need it:

When using ArgoCD to deploy Kong gateway using this Helm chart, the HorizontalPodAutoscaler api version is not well detected : on k8s 1.22, Argocd tries to use the autoscaling/v2 but it is not available, so it fails to deploy.
This fix is similar to this one: elastic/helm-charts#1684
This bug seems to have been introduced by #679

Special notes for your reviewer:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • PR is based off the current tip of the main branch.
  • Changes are documented under the "Unreleased" header in CHANGELOG.md
  • New or modified sections of values.yaml are documented in the README.md
  • Commits follow the Kong commit message guidelines

@CLAassistant
Copy link

CLAassistant commented Feb 14, 2023

CLA assistant check
All committers have signed the CLA.

@slavogiez slavogiez force-pushed the fix/hpa-api-versions-detection branch from 1fd2a2f to e207d38 Compare February 14, 2023 11:00
@slavogiez slavogiez marked this pull request as ready for review February 14, 2023 11:00
@slavogiez slavogiez requested a review from a team as a code owner February 14, 2023 11:00
@slavogiez slavogiez changed the title fix(hpa) enhance api version detection fix(hpa) properly detect hpa capabilities Feb 14, 2023
@czeslavo czeslavo changed the title fix(hpa) properly detect hpa capabilities fix(hpa) properly detect hpa capabilities, release 2.16.4 Feb 14, 2023
@czeslavo
Copy link
Contributor

Hey @slavogiez 👋 Thank you for the contribution! 🎖️

Let's just wait for the pipelines to complete and we'll release 2.16.4 with your patch.

@czeslavo czeslavo merged commit 440fd2d into Kong:main Feb 14, 2023
@slavogiez
Copy link
Contributor Author

Awesome! Thanks!

@slavogiez slavogiez deleted the fix/hpa-api-versions-detection branch February 14, 2023 13:56
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.

None yet

3 participants