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

[BUGFIX] Corriger l'exécution de la tâche metrics #102

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

MathieuGilet
Copy link
Contributor

@MathieuGilet MathieuGilet commented Jun 13, 2024

🦄 Problème

Suite au passage en ESM, les events db-metrics ne sont pas loggés à cause d'un problème d'injection de dépendance. Cet event nous permet notamment de vérifier la charge CPU et l'utilisation de l'espace disque des bases de données chez Scalingo.
De plus, le scheduler n'étais jamais lancé.

🤖 Solution

  • Corriger l'injection de dépendance pour la tâche metrics.
  • Exécuter le scheduler.

💯 Pour tester

Regarder les logs du container et vérifier qu'il log bien les métriques suivantes (configuration similaire à la production sur pix-api-recette et pix-api-integration)

https://dashboard.scalingo.com/apps/osc-fr1/pix-db-stats-review-pr102/logs

@MathieuGilet MathieuGilet added Tech review needed team-captains This your captain speaking labels Jun 13, 2024
@MathieuGilet MathieuGilet self-assigned this Jun 13, 2024
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://db-stats-pr102.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-db-stats-review-pr102/environment

@pix-service-auto-merge pix-service-auto-merge merged commit 9f557a9 into master Jun 24, 2024
9 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the fix-task-metrics branch June 24, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants