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

HorizontalPodAutoscaling/hpa chart values #172

Open
alextreme opened this issue Jan 27, 2025 · 3 comments · May be fixed by #173
Open

HorizontalPodAutoscaling/hpa chart values #172

alextreme opened this issue Jan 27, 2025 · 3 comments · May be fixed by #173
Assignees

Comments

@alextreme
Copy link
Member

Feedback from Utrecht, Taiga 329

Bij de helm-chart van Objects wordt geen hpa voor objects-worker ingesteld (terwijl de worker.replicaCount' genegeerd wordt als ‘autoscaling.enabled’=true. Bij laatste zou ‘worker.autoscaling.enabled’ verwacht worden.
Zelfde voor open-zaak-nginx, geen hpa terwijl de ‘nginx.replicaCount’ genegeerd wordt als ‘nginx.autoscaling.enabled’=true

@Tiho0135 of @SilviaAmAm mag dit dubbelchecken en indien nodig corrigeren

@SilviaAmAm SilviaAmAm self-assigned this Jan 28, 2025
@Tiho0135
Copy link

Tiho0135 commented Jan 28, 2025

I think Sjoerd did not add HPA for all applications in the same way. We can add the HPA for the mentioned above cases.
Whether we need HPA for all application in the same way I cannot say at the moment.

@Tiho0135
Copy link

Tiho0135 commented Jan 28, 2025

The autoscaler we have is same as in the k8s doc ->
https://raw.githubusercontent.com/kubernetes/website/main/content/en/examples/application/hpa/php-apache.yaml
And only uses cpu and memory. Probably that is enough, just wanted to write my findings.

@SilviaAmAm
Copy link
Collaborator

SilviaAmAm commented Jan 28, 2025

The helm chart of objecten is indeed missing the hpa for the worker and the one of OZ is missing the one for nginx.

Most applications currently only have HPA for the main app specified.

Todo

  • Add worker HPA for objecten and check that the values are correct
  • Add nginx HPA for openzaak and check that the values are correct
  • Add templates to the maykin-utils-lib, so that they can easily be added to projects

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 a pull request may close this issue.

3 participants