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

Fixed an issue that could cause the api service to exit(1) when using the default documented docker-compose.yml file #93

Closed
wants to merge 2 commits into from

Conversation

Saetch
Copy link

@Saetch Saetch commented Nov 5, 2023

It is possible for the api service to crash due to being unable to connect to the database which is not yet ready to accept connections. I added a healthcheck for the db and added a condition for the api service to wait until the db service is healthy

Copy link

github-actions bot commented Nov 5, 2023

Hi! Thank you for your contribution.
This repo is only a mirror and unfortunately we can't accept PRs made here. Please re-submit your changes to our Gitea instance.
Also check out the contribution guidelines.
Thank you for your understanding.

@github-actions github-actions bot closed this Nov 5, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Nov 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant