Skip to content

add utilities and docs for debugging with docker#190

Merged
davidwyld merged 2 commits intomainfrom
dw/debug_docker
Jan 21, 2026
Merged

add utilities and docs for debugging with docker#190
davidwyld merged 2 commits intomainfrom
dw/debug_docker

Conversation

@davidwyld
Copy link
Collaborator

Finally worked out a more elegant way of doing this without interfering with the running of the compose stack in normal operations.

This makes the dockerised debugging opt-in and stops the debugger from interrupting the normal running of the django app.

As documented, opt-in and run the debugger by using the additional compose.debug.yml file with docker compose -f compose.yml -f compose.debug.yml up

The only interference with the normal docker/compose usage (which is of course only for development anyway) is that debugpy is installed by default in the app image.

@github-actions
Copy link

github-actions bot commented Jan 21, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
polarrouteserver
   __init__.py30100% 
   _version.py130100% 
   celery.py1530 80%
polarrouteserver/route_api
   __init__.py40100% 
   models.py7220 97%
   responses.py330100% 
   serializers.py12054 94%
   tasks.py124217 82%
   utils.py112104 91%
   views.py276174 94%
polarrouteserver/route_api/management
   __init__.py00100% 
polarrouteserver/route_api/management/commands
   __init__.py00100% 
   ensure_adminuser.py22220 0%
   insert_mesh.py26260 0%
polarrouteserver/utils
   __init__.py00100% 
   loggers.py10100 0%
TOTAL83011685% 

Copy link
Contributor

@thomaszwagerman thomaszwagerman left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Co-authored-by: Thomas Zwagerman <36264706+thomaszwagerman@users.noreply.github.com>
@davidwyld davidwyld merged commit 554251e into main Jan 21, 2026
7 checks passed
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