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.

Sign up for GitHub

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: domifa update-ressources #3393

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: domifa update-ressources #3393

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
56 changes: 35 additions & 21 deletions .kontinuous/env/prod/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,17 +83,15 @@ backend-cron:
backend: &backendProd
~needs: [build-backend]
host: "domifa-api.{{ .Values.global.domain }}"
autoscale:
minReplicas: 4
maxReplicas: 8
enabled: true
resources:
requests:
cpu: 2
memory: 2Gi
cpu: 1
memory: 1.5Gi
limits: # exports need a lot of CPU/RAM ATM
cpu: 3
memory: 3Gi
cpu: 1.5
memory: 2Gi
autoscale:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@igorrenquin est-ce qu'on peut garder un minimum de replicas toujours up ? Le nombre de replicas max est à combien ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

L'autoscale est maintenant géré nativement par Kontinuous. Plus besoin de la faire apparaitre dans le fichier de value come nous l'avons fait dans le passé.

Le paramétrage par défaut est :

min = 2
max = 10

enabled: true
certSecretName: backend-crt
envFrom:
- secretRef:
Expand All @@ -111,17 +109,15 @@ backend: &backendProd
backend-export:
~needs: [build-backend]
host: "domifa-api.{{ .Values.global.domain }}"
autoscale:
minReplicas: 2
maxReplicas: 5
enabled: true
resources:
requests:
cpu: 1
memory: 5Gi
memory: 768i
limits: # exports need a lot of CPU/RAM ATM
cpu: 3
memory: 5Gi
cpu: 1
memory: 1Gi
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@igorrenquin sur l'export on peut passer la barre du giga de mémoire consomme d'après les tests qu'on a fait.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pas constaté cela dans le tableau de bord
https://grafana-ovh.fabrique.social.gouv.fr/d/cafbf60d-79e0-4e61-982e-7752ec2a9c22/tableau-de-bord-domifa?orgId=1

autoscale:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

problème d'indentation ici si on compare à l'état précédent de la config (le problème apparaitrait mieux sur la diff github si tu laissais le block autoscale au même endroit dans le fichier, en ne changeant que les sous-clés à changer

enabled: true
certSecretName: backend-crt
envFrom:
- secretRef:
Expand All @@ -143,22 +139,40 @@ backend-export:
- /stats/home

frontend:
resources:
requests:
cpu: 100m
memory: 128Mi
limits:
cpu: 250m
memory: 256Mi
autoscale:
minReplicas: 2
maxReplicas: 4
enabled: true
host: "{{ .Values.global.host }}"
certSecretName: frontend-crt

portail-admins:
resources:
requests:
cpu: 100m
memory: 128Mi
limits:
cpu: 200m
memory: 256Mi
autoscale:
minReplicas: 2
maxReplicas: 4
enabled: true
host: "admin-{{ .Values.global.host }}"
certSecretName: portail-admins-crt

portail-usagers:
resources:
requests:
cpu: 100m
memory: 128Mi
limits:
cpu: 200m
memory: 256Mi
autoscale:
minReplicas: 2
maxReplicas: 4
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@igorrenquin on avait mit 2 replicas pour avoir un replica par node suite à un plantage d'un des noeud y'a quelques semaines.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

L'autoscale est maintenant géré nativement par Kontinuous. Plus besoin de la faire apparaitre dans le fichier de value come nous l'avons fait dans le passé.

Le paramétrage par défaut est :

  • min = 2
  • max = 10

enabled: true
host: "mon-{{ .Values.global.host }}"
certSecretName: portail-usagers-crt
Loading