Skip to content

Fix docker compose to make it running with older docker images#307

Merged
MarcelGeo merged 1 commit intomasterfrom
fix_docker_compose
Oct 25, 2024
Merged

Fix docker compose to make it running with older docker images#307
MarcelGeo merged 1 commit intomasterfrom
fix_docker_compose

Conversation

@varmar05
Copy link
Collaborator

After split of celery-* and server containers running docker-compose.yml with CE/EE older images results in confusing situation, e.g. running services multiple times or not running them all. This is because we changed entrypoint.sh to accept any runtime command from docker-compose file which is not respected by older images.

To overcome this limitation new entrypoint.sh is mounted to replace the one built in images.
Also missing keys were added for celery containers in compose file.

Logs:

merginmaps-server  | [INFO] [7] Starting gunicorn 19.9.0
merginmaps-server  | [INFO] [7] Listening at: http://0.0.0.0:5000 (7)
merginmaps-server  | [INFO] [7] Using worker: gevent
merginmaps-server  | [INFO] [9] Booting worker with pid: 9
merginmaps-server  | [INFO] [10] Booting worker with pid: 10
celery-beat        | [2024-10-25 13:41:48,935: INFO/MainProcess] beat: Starting...
celery-worker      | [2024-10-25 13:41:49,133: INFO/MainProcess] Connected to redis://merginmaps-redis:6379/0
celery-worker      | [2024-10-25 13:41:49,142: INFO/MainProcess] mingle: searching for neighbors
celery-worker      | [2024-10-25 13:41:50,155: INFO/MainProcess] mingle: all alone
celery-worker      | [2024-10-25 13:41:50,185: INFO/MainProcess] celery@74aa5e3990ab ready.

@varmar05 varmar05 requested a review from MarcelGeo October 25, 2024 12:49
Copy link
Collaborator

@MarcelGeo MarcelGeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@MarcelGeo MarcelGeo merged commit 801545b into master Oct 25, 2024
@MarcelGeo MarcelGeo deleted the fix_docker_compose branch October 25, 2024 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants