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

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

Conversation

igorrenquin
Copy link
Contributor

  • Adjust allocated ressources frontend, backend, export, usagers, admin
  • Set autoscale from specific to default setting

Adjust allocated ressources frontend, backend, export, usagers, admin
set autoscale from specific to default setting
@igorrenquin igorrenquin self-assigned this Jul 3, 2024
Copy link

sonarcloud bot commented Jul 3, 2024

@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.50%. Comparing base (b0f9f63) to head (63c50dc).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3393   +/-   ##
=======================================
  Coverage   66.50%   66.50%           
=======================================
  Files         911      911           
  Lines       15121    15121           
  Branches     2136     2140    +4     
=======================================
  Hits        10056    10056           
  Misses       4906     4906           
  Partials      159      159           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b0f9f63...63c50dc. Read the comment docs.

@igorrenquin igorrenquin requested review from a team and pYassine July 3, 2024 09:31
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

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.

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

memory: 5Gi
cpu: 1
memory: 1Gi
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

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.

4 participants