Add supervisor for automatic server restart #25
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR provides a solution for issue #4.
By introducing supervisor, the Flask app can now be restarted when it fails, instead of directly finishing the docker container. The pushed configuration tries to restart the app a total of 35 times. In its current state, supervisor delays every retry by 1 second. This means:
If this approach is accepted, a follow-up PR with a workaround still needs to be added to restart the app once supervisor gives up restarting it, as suggested by Supervisor/supervisor#487 (comment). This is needed to keep the retry delays reasonable and on an infinite loop.