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

Add healthcheck on monitorfish-pipeline-flows container #882

Merged
merged 3 commits into from Dec 30, 2021

Conversation

VincentAntoine
Copy link
Collaborator

@VincentAntoine VincentAntoine changed the title Add flows healthcheck Add healthcheck on monitorfish-pipeline-flows container Dec 30, 2021
@VincentAntoine
Copy link
Collaborator Author

Les agents dans Prefect peuvent implémenter un endpoint de healthcheck pour vérifier que le process agent est sain :

https://docs.prefect.io/orchestration/agents/overview.html#health-checks

Ce endpoint est ensuite utilisable par Docker pour indiquer la santé du container :

https://docs.docker.com/engine/reference/run/#healthcheck

En revanche Docker n'offre pas de fonctionnalité de redémarrage automatique des containers unhealthy :
docker/compose#4826

Un workaround :

2 * * * * docker restart $(docker ps | grep unhealthy | cut -c -12) 2>/dev/null

Un peu crado peut efficace a priori.

@louptheron apparemment swarm a une fonctionnalité native de redémarrage des containers unhealthy. A discuter si un intérêt de passer en swarm par ailleurs ou si on reste comme ça!

@sonarcloud
Copy link

sonarcloud bot commented Dec 30, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@VincentAntoine VincentAntoine merged commit 2fa6acb into master Dec 30, 2021
@VincentAntoine VincentAntoine deleted the add_flows_healthcheck branch January 17, 2022 10:42
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.

Ajouter un healthcheck dans le container de flows
1 participant