Fix: Add missing dependencies and improve Docker setup #1023
+24
−3
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.
Description
This MR fixes critical Docker deployment issues and improves the containerized setup:
Missing Dependency: Adds
google-auth-oauthlibpackage which is required byhorilla_backupmodule but was missing from requirements.txt, causingModuleNotFoundErrorduring application startup.Translation Compilation: Installs
gettexttools in Dockerfile and adds automatic translation file compilation in entrypoint.sh, fixing the language selection feature that wasn't working due to missing compiled.mofiles.Startup Reliability: Improves entrypoint.sh with database connection wait logic and better error handling to ensure migrations and static file collection complete successfully before starting the server.
Dependencies Required:
google-auth-oauthlib(new dependency)gettextsystem package (for translation compilation)Ticket Link
N/A
Summary of Changes
google-auth-oauthlibto requirements.txtgettextpackage in Dockerfile runtime stagecompilemessages) in entrypoint.sh--noinputflags to migration commands for non-interactive executionAdditional implementation details (OPTIONAL)
|| echofallback to prevent startup failure if some translations fail--noinputflag for Docker compatibilityset -eat the beginning of entrypoint.sh for better error handlingDeployment Notes (OPTIONAL)
gettextpackage adds ~10MB to the Docker image sizeScreenshot
Before
Error during startup:
Language selection not working:
.mofiles)Potential race conditions:
After
Successful startup:
Improved reliability: