-
Notifications
You must be signed in to change notification settings - Fork 0
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 for database container #80
base: main
Are you sure you want to change the base?
Conversation
strapi will only start up if database is healthy/ready
We should merge the current main branch here, apply these changes to the new |
Deployed to test,
See the |
I don't understand why the changes in the three files are so different: no |
You are spot on. Somehow i've messed up the files when editing in compare mode of my editor. Re-deployed to test. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, will deploy on prod with next deployment there.
(One note on our test deployment process: I prefer to merge the feature branch into main on test, instead of switching the branch on test. That way, we can deploy multiple feature branches at the same time, without losing changes from another branch.)
Alright, i will merge into main from now on. |
With this change strapi will only start up if database is healthy/ready.
If the database is not ready starting up using
docker compose up
will end withwhereas a successful start up will be confirmed by
Why?
As just happened the other day, creating a file
data/postgresql.conf.orig
on the host machine as root will make strapi start with errors like:It is not immediately obvious what went wrong because the the
docker compose up
command doesn't show any errors.