[Fixes #14367] Error in src mount point#14368
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the docker-compose.yml file to mount the host directory directly to /usr/src/geonode by default. Feedback indicates that this change can overwrite the container's internal directory in environments where only configuration files are present, causing container startup failures. It is recommended to keep this volume mount commented out by default or move it to a local development override file.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| - .env${ENVIRONMENT:+_${ENVIRONMENT}} | ||
| volumes: | ||
| # - './src:/usr/src/geonode' # Uncomment for local development | ||
| - '.:/usr/src/geonode' |
There was a problem hiding this comment.
Mounting the host directory . directly to /usr/src/geonode by default (uncommented) will overwrite the container's internal /usr/src/geonode directory. When running the stack in environments where only the docker-compose.yml and .env files are present (e.g., production deployments using pre-built images), this mount will replace the application code and entrypoint script with just those configuration files, causing the container to fail to start with a "file not found" error for /usr/src/geonode/entrypoint.sh.
To prevent this, the volume mount for local development should either be commented out by default (as it was previously) or moved to a docker-compose.override.yml file specifically meant for local development.
# - '.:/usr/src/geonode' # Uncomment for local developmentThere was a problem hiding this comment.
Pull request overview
Updates the root docker-compose.yml to address the /usr/src/geonode source mount mismatch that causes entrypoint.sh to be missing when a local bind mount is used.
Changes:
- Replaces the prior (commented)
./src:/usr/src/geonodedevelopment volume with a bind mount of the repository root to/usr/src/geonode.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| volumes: | ||
| # - './src:/usr/src/geonode' # Uncomment for local development | ||
| - '.:/usr/src/geonode' | ||
| - statics:/mnt/volumes/statics |
Fixes #14367
Reinstate path before
729ce51#diff-e45e45baeda1c1e73482975a664062aa56f20c03dd9d64a827aba57775bed0d3L14
No backport needed since the issue is on master only.
Checklist
For all pull requests:
The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):
Submitting the PR does not require you to check all items, but by the time it gets merged, they should be either satisfied or inapplicable.