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: cdtn-admin update resources #1438

Merged
merged 2 commits into from
Jul 4, 2024
Merged

Conversation

igorrenquin
Copy link
Contributor

Adjust cdt-admin resources www, hasura, cnpg

Adjust cdt-admin resources www, hasura, cnpg
@igorrenquin igorrenquin requested a review from a team July 3, 2024 11:34
@igorrenquin igorrenquin self-assigned this Jul 3, 2024
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

memory: "4Gi"
requests:
cpu: "1000m"
cpu: "500m"
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.

ça me parait léger 1 Go en mémoire ? On charge beaucoup de donnée dans ce processus. Qu'en penses tu @maxgfr ? Il faudrait que l'on regarde la mémoire quand on lance une mise en prod.

memory: "4Gi"
requests:
cpu: "1000m"
cpu: "500m"
memory: "1Gi"
Copy link
Member

Choose a reason for hiding this comment

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

Ah oui, pas possible dans le cas où on lance un glossaire

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 peut laisser 4Gi pour la limite ? C'est un service qui peut être assez gourmand sur certaines tâches que l'on réalise très peu mais qui peu arriver.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Donc oui, c'est sur hasura pas sur export, donc c'est parfait

Copy link

sonarcloud bot commented Jul 4, 2024

Copy link

github-actions bot commented Jul 4, 2024

🎉 Deployment for commit a7462cb :

Ingresses
Docker images
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/export:sha-a7462cb0e9254bc0797e97b40975af043a7462ef
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/frontend:sha-a7462cb0e9254bc0797e97b40975af043a7462ef
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/hasura:sha-a7462cb0e9254bc0797e97b40975af043a7462ef
Debug

@maxgfr maxgfr merged commit 9ebda48 into master Jul 4, 2024
29 checks passed
@maxgfr maxgfr deleted the cdtn-admin_update-resources branch July 4, 2024 12:21
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.

5 participants