Skip to content

Update deployment workflow #675#686

Open
drakeredwind01 wants to merge 8 commits into
mainfrom
drake_675_update_deployment_workflow
Open

Update deployment workflow #675#686
drakeredwind01 wants to merge 8 commits into
mainfrom
drake_675_update_deployment_workflow

Conversation

@drakeredwind01

@drakeredwind01 drakeredwind01 commented Jun 12, 2026

Copy link
Copy Markdown
Member

Fixes #675

Update deployment workflow #675

Most action items already handled in #672–674

  • deploy-dev.yml already uses docker build -f Dockerfile-aws
  • requirements-aws.txt referenced inside Dockerfile-aws — picked up automatically
  • settings_aws.py set via ENV DJANGO_SETTINGS_MODULE in Dockerfile-aws
  • entrypoint-aws.sh is the ENTRYPOINT in Dockerfile-aws
  • ✓ gunicorn started by entrypoint-aws.sh

Pre-implementation concerns

1. BuildKit not explicitly enabled — NOT mentioned in the issue
Dockerfile-aws uses --mount=type=cache which requires BuildKit. GitHub Actions enables BuildKit by default on recent Docker versions but it is not guaranteed and is not explicitly set in the workflow. If missing, the build fails with a syntax error on the --mount lines.
Fix: added DOCKER_BUILDKIT: 1 to the build step's env block in deploy-dev.yml.

2. Documentation update — IS mentioned in the issue
docs/contributing/reference/aws-resources.md already references Dockerfile-aws and entrypoint-aws.sh but needed additions: gunicorn as the WSGI server, whitenoise for static files, and settings_aws.py as the production settings module.

Files updated:

  • .github/workflows/deploy-dev.yml — added DOCKER_BUILDKIT: 1 to build step env block
  • docs/contributing/reference/aws-resources.md — updated step 4 to mention gunicorn and whitenoise; added settings_aws.py to Docker & App Startup section

Closes #219

@@ -0,0 +1,37 @@
from .settings import * # noqa: F401, F403
@drakeredwind01 drakeredwind01 requested a review from fyliu June 12, 2026 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Update deployment workflow Create deployment docker setup (for dev env)

1 participant