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

misc(auth): added service healthcheck #1394

Conversation

iulianbarbu
Copy link
Contributor

@iulianbarbu iulianbarbu commented Nov 16, 2023

Description of change

Use an auth service health-check when deploying the stack.

How has this been tested? (if applicable)

Locally, by querying the container state before and after the health check addition.
E.g. before the health-check, this docker inspect --format='{{json .State.Health}}' <container_id>, returns null, while after adding the healthcheck, it shows something along:

{"Status":"healthy","FailingStreak":0,"Log":[{"Start":"2023-11-16T15:10:22.278007509Z","End":"2023-11-16T15:10:22.362298051Z","ExitCode":0,"Output":"  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current\n                                 Dload  Upload   Total   Spent    Left  Speed\n\r  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0\r  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0\n"},{"Start":"2023-11-16T15:11:22.371958801Z","End":"2023-11-16T15:11:22.447879968Z","ExitCode":0,"Output":"  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current\n                                 Dload  Upload   Total   Spent    Left  Speed\n\r  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0\r  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0\n"},{"Start":"2023-11-16T15:12:22.456275676Z","End":"2023-11-16T15:12:22.543789593Z","ExitCode":0,"Output":"  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current\n                                 Dload  Upload   Total   Spent    Left  Speed\n\r  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0\r  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0\n"},{"Start":"2023-11-16T15:14:17.750973049Z","End":"2023-11-16T15:14:17.832457424Z","ExitCode":0,"Output":"  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current\n                                 Dload  Upload   Total   Spent    Left  Speed\n\r  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0\r  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0\n"}]}

@jonaro00 jonaro00 changed the title auth: added service healthcheck misc(auth): added service healthcheck Nov 16, 2023
@iulianbarbu iulianbarbu merged commit 44dfa9b into main Nov 17, 2023
@iulianbarbu iulianbarbu deleted the feature/eng-1854-enable-auth-service-health-check-in-docker-compose branch November 17, 2023 16:13
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