-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: added healthcheck to outline #469
base: master
Are you sure you want to change the base?
Conversation
ec585da
to
c971269
Compare
Can't convert this to draft on mobile so here goes: outline recently fixed itself so the Docker file is redundant. Will simplify things |
@stickyPiston pretty please? 🦫 |
LGTM, did not test tho. Edit: I see this is a draft. I have reverted my approval, please ping mw when done. |
I just finished it now, but I am not certain that my change will not affect the data stored. I will check on staging. |
After you reviewed I reverted the changes to the line where it states to what path the |
This PR aims to monitor the downage of Outline, by adding healthchecks to a custom image of outline.
In order to properly achieve this, the compose-up.yml tasks of the Docker role needed to be expanded to support rebuilding (outdated) images.
Closes #466