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 manages Deployment.replicas when configured #94

Conversation

rsicart
Copy link
Contributor

@rsicart rsicart commented Apr 15, 2024

As documented:

If a HorizontalPodAutoscaler (or any similar API for horizontal scaling) is managing scaling for a Deployment, don't set .spec.replicas.

https://kubernetes.io/docs/concepts/workloads/controllers/deployment/#replicas

As documented:

If a HorizontalPodAutoscaler (or any similar API for horizontal scaling) is managing scaling for a Deployment, don't set .spec.replicas.

https://kubernetes.io/docs/concepts/workloads/controllers/deployment/#replicas
@rsicart
Copy link
Contributor Author

rsicart commented Apr 15, 2024

This PR is related to #92

Not very important but maybe we can add this fix before releasing v0.1.30 in #93.

@JAORMX
Copy link
Collaborator

JAORMX commented Apr 15, 2024

@rsicart That PR is only the docs. The release has been done. But I can easily push another patch release to add this.

@JAORMX JAORMX merged commit 786a6a7 into equinixmetal-helm:main Apr 15, 2024
11 of 12 checks 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