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

feat(charts/dex): use updated HorizontalPodAutoscaler API Version #111

Merged
merged 2 commits into from
Jun 27, 2023
Merged

feat(charts/dex): use updated HorizontalPodAutoscaler API Version #111

merged 2 commits into from
Jun 27, 2023

Conversation

sgmccullough
Copy link
Contributor

Overview

Use updated autoscaling API Version for HorizontalPodAutoscaler.

What this PR does / why we need it

The autoscaling/v2beta1 API version of HorizontalPodAutoscaler isn't served on K8s versions >=1.25. The Dex chart should support the newer version of the API, autoscaling/v2 which is available on K8s versions >=1.23.

Special notes for your reviewer

Checklist

  • Change log updated in Chart.yaml (see the contributing guide for details)
  • Chart version bumped in Chart.yaml (see the contributing guide for details)
  • Documentation regenerated by running make docs

Signed-off-by: Scott McCullough <mcculloughsco12@gmail.com>
Signed-off-by: Scott McCullough <mcculloughsco12@gmail.com>
@sgmccullough
Copy link
Contributor Author

When you have some time @sagikazarmark it would be great to get a review on this.

@sagikazarmark
Copy link
Member

Thanks for the ping and patience @sgmccullough! LGTMT

@rlandesman
Copy link

Fantastic, thank you!!

@sagikazarmark sagikazarmark merged commit 9a64526 into dexidp:master Jun 27, 2023
10 checks passed
sgmccullough added a commit to sgmccullough/dex-helm-charts that referenced this pull request Jun 27, 2023
sgmccullough added a commit to sgmccullough/dex-helm-charts that referenced this pull request Jun 27, 2023
Signed-off-by: GitHub <noreply@github.com>
nabokihms added a commit that referenced this pull request Jul 10, 2023
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: Maksim Nabokikh <max.nabokih@gmail.com>
Co-authored-by: Maksim Nabokikh <max.nabokih@gmail.com>
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.

3 participants