Run locally with Docker by paigewilliams · Pull Request #19 · Ecotrust/wcbluepages

paigewilliams · 2026-02-07T00:02:01Z

I was not able to get the app to run locally without loading a data dump. I think this is because of the migration issues @rhodges mentioned.

This does not have updated python + django versions

To see the specific tasks where the Asana app for GitHub is being used, see below:
- https://app.asana.com/0/0/1213142767947194

Copilot

Pull request overview

This PR adds a Docker-based local development setup for the WC Bluepages Django app, including a PostGIS database container (optionally seeded from a SQL dump) and a web container that runs migrations/collectstatic on startup.

Changes:

Add docker-compose.yml + Dockerfile to build/run the app and a PostGIS DB locally.
Add container entrypoint/init scripts to import a DB dump and run Django management tasks.
Add docker_settings.py and update settings.py to support Docker-specific configuration; add Docker setup README and .dockerignore.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
`Dockerfile`	Builds a Python image with GIS/Postgres deps and uses a custom entrypoint script.
`docker-compose.yml`	Defines `db` (PostGIS) and `web` (Django) services, volumes, and env vars.
`docker-entrypoint.sh`	Waits for DB readiness, runs migrations/collectstatic, then starts the server.
`init-db.sh`	Initializes the DB container by importing `db_dump.sql` on first startup.
`bluepages/bluepages/settings.py`	Loads `docker_settings.py` when `DOCKER_CONTAINER` is set.
`bluepages/bluepages/docker_settings.py`	Provides Docker-specific DB/static/media settings.
`README.docker.md`	Documents how to run locally using Docker with a required DB dump.
`.dockerignore`	Reduces Docker build context size and excludes dev artifacts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-07T00:06:03Z

init-db.sh

+# Import the SQL dump, ignoring meta-command errors
+psql -U "$POSTGRES_USER" -d "$POSTGRES_DB" -v ON_ERROR_STOP=0 < /tmp/db_dump.sql


Using -v ON_ERROR_STOP=0 will continue on SQL errors and can silently produce a partially imported/corrupt initial database. Since this runs on first init of an empty DB, it’s safer to stop on the first error (ON_ERROR_STOP=1) and fail the container startup, or explicitly handle only expected errors.

Suggested change

# Import the SQL dump, ignoring meta-command errors

psql -U "$POSTGRES_USER" -d "$POSTGRES_DB" -v ON_ERROR_STOP=0 < /tmp/db_dump.sql

# Import the SQL dump, stopping on the first error to avoid partial/corrupt state

psql -U "$POSTGRES_USER" -d "$POSTGRES_DB" -v ON_ERROR_STOP=1 < /tmp/db_dump.sql

Copilot · 2026-02-07T00:06:04Z

Dockerfile

+# Copy and set entrypoint script
+COPY docker-entrypoint.sh /app/


COPY . /app/ already brings docker-entrypoint.sh into the image, so the later COPY docker-entrypoint.sh /app/ is redundant and can cause confusion when editing. Consider removing one of the copies (typically keep the explicit copy and move it before COPY . if you want better layer caching).

Suggested change

# Copy and set entrypoint script

COPY docker-entrypoint.sh /app/

# Set entrypoint script as executable

Copilot · 2026-02-07T00:06:04Z

bluepages/bluepages/docker_settings.py

+# Allow all hosts in Docker (configure properly for production)
+ALLOWED_HOSTS = ['*']


Setting ALLOWED_HOSTS = ['*'] makes the container accept requests for any Host header, which is unsafe if the port is ever exposed beyond localhost (e.g., on a shared network). Prefer restricting to local/dev hosts (localhost/127.0.0.1/0.0.0.0) or making this configurable via an env var.

Suggested change

# Allow all hosts in Docker (configure properly for production)

ALLOWED_HOSTS = ['*']

# Allowed hosts: configurable via env, with safe local defaults

_allowed_hosts_env = os.environ.get('DJANGO_ALLOWED_HOSTS')

if _allowed_hosts_env:

ALLOWED_HOSTS = [h.strip() for h in _allowed_hosts_env.split(',') if h.strip()]

else:

ALLOWED_HOSTS = ['localhost', '127.0.0.1', '0.0.0.0']

Copilot · 2026-02-07T00:06:04Z

bluepages/bluepages/docker_settings.py

+}
+
+# Debug mode from environment
+DEBUG = os.environ.get('DEBUG', 'False') == 'True'


DEBUG = os.environ.get('DEBUG', 'False') == 'True' is case-sensitive and treats values like true/1/yes as False, which is easy to misconfigure. Consider using a more robust boolean parser (e.g., normalize to lowercase and accept common truthy values).

Suggested change

DEBUG = os.environ.get('DEBUG', 'False') == 'True'

DEBUG = os.environ.get('DEBUG', 'False').strip().lower() in ('true', '1', 'yes', 'y', 'on')

Copilot · 2026-02-07T00:06:04Z

docker-compose.yml

+    image: postgis/postgis:16-3.4
+    platform: linux/amd64
+    container_name: bluepages_db


Hard-coding platform: linux/amd64 reduces portability (e.g., Apple Silicon will run under emulation, slower; some setups may fail). If this is only needed for specific environments, consider making it optional/configurable (e.g., via an env var) or documenting why it’s required.

Copilot · 2026-02-07T00:06:05Z

Dockerfile

+# Use Python 3.11 slim image as base
+FROM python:3.11-slim-bookworm


The base requirements pin Django to <3.3 (Django 3.2.x), which is not compatible with Python 3.11 per Django’s supported Python versions. This image is likely to fail at runtime or during dependency install; either use a Python 3.10 base image or upgrade Django (and any deps) to a version that supports 3.11.

Suggested change

# Use Python 3.11 slim image as base

FROM python:3.11-slim-bookworm

# Use Python 3.10 slim image as base (compatible with Django 3.2.x)

FROM python:3.10-slim-bookworm

Copilot · 2026-02-07T00:06:05Z

docker-entrypoint.sh

+echo "Waiting for PostgreSQL..."
+while ! pg_isready -h "$DB_HOST" -p "$DB_PORT" -U "$DB_USER" > /dev/null 2>&1; do
+    sleep 1
+done


The PostgreSQL wait loop has no timeout, so a misconfigured DB_* env or an unavailable database will hang the container indefinitely. Add a max retry count / deadline and exit non-zero with a clear error when exceeded.

Copilot · 2026-02-07T00:06:05Z

bluepages/bluepages/settings.py

+# Import Docker settings if running in Docker
+if os.environ.get('DOCKER_CONTAINER'):
+    try:
+        from .docker_settings import *


Import pollutes the enclosing namespace, as the imported module bluepages.docker_settings does not define 'all'.

Copilot · 2026-02-07T00:06:05Z

bluepages/bluepages/settings.py

 except Exception as e:
    print("Unable to import local_settings")
-    pass
+    pass


Unnecessary 'pass' statement.

Copilot · 2026-02-07T00:06:05Z

bluepages/bluepages/settings.py

+        print("Docker settings loaded")
+    except Exception as e:
+        print(f"Unable to import docker_settings: {e}")
+        pass


Unnecessary 'pass' statement.

pollardld

This is in good shape for use after the migration issue from the PR description is solved.

paigewilliams added 3 commits February 6, 2026 12:13

try docker; not working yet

9b68d7e

add entrypoint and load dump into db when first initialized

538baa8

update docs

35f34fe

paigewilliams requested review from Copilot, pollardld and rhodges February 7, 2026 00:02

paigewilliams self-assigned this Feb 7, 2026

Copilot started reviewing on behalf of paigewilliams February 7, 2026 00:02 View session

Copilot AI reviewed Feb 7, 2026

View reviewed changes

pollardld reviewed Feb 11, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

Run locally with Docker#19

Run locally with Docker#19
paigewilliams wants to merge 3 commits intomainfrom
try-docker

paigewilliams commented Feb 7, 2026 •

edited by pollardld

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

pollardld left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		# Import the SQL dump, ignoring meta-command errors
		psql -U "$POSTGRES_USER" -d "$POSTGRES_DB" -v ON_ERROR_STOP=0 < /tmp/db_dump.sql

		# Copy and set entrypoint script
		COPY docker-entrypoint.sh /app/

	# Copy and set entrypoint script
	COPY docker-entrypoint.sh /app/
	# Set entrypoint script as executable

		# Allow all hosts in Docker (configure properly for production)
		ALLOWED_HOSTS = ['*']

-# Allow all hosts in Docker (configure properly for production)
-ALLOWED_HOSTS = ['*']
+# Allowed hosts: configurable via env, with safe local defaults
+_allowed_hosts_env = os.environ.get('DJANGO_ALLOWED_HOSTS')
+if _allowed_hosts_env:
+    ALLOWED_HOSTS = [h.strip() for h in _allowed_hosts_env.split(',') if h.strip()]
+else:
+    ALLOWED_HOSTS = ['localhost', '127.0.0.1', '0.0.0.0']

	DEBUG = os.environ.get('DEBUG', 'False') == 'True'
	DEBUG = os.environ.get('DEBUG', 'False').strip().lower() in ('true', '1', 'yes', 'y', 'on')

		# Use Python 3.11 slim image as base
		FROM python:3.11-slim-bookworm

Comments

Conversation

paigewilliams commented Feb 7, 2026 • edited by pollardld Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

pollardld left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

paigewilliams commented Feb 7, 2026 •

edited by pollardld

Loading