Skip to content

Move django migrations to dokku release phase #2736

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rw251 merged 6 commits into from
Jul 18, 2025
Merged

Move django migrations to dokku release phase #2736

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
66e6228
Move migrations into dokku `release` phase
rw251 Jul 16, 2025
b27998e
Fix CI yml files to output logs on failure
rw251 Jul 17, 2025
91a3e9f
Fix the smoke test
rw251 Jul 16, 2025
28c1e03
Fix docker/storage dir to always have correct owner
rw251 Jul 16, 2025
d2b6ab0
Added ADR
rw251 Jul 17, 2025
569792b
Tidy up following review
rw251 Jul 18, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion docker/justfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,12 @@ exec env="dev" *args="bash":


# run a basic functional smoke test against a running opencodelists
smoke-test host="http://localhost:7000":
smoke-test host="http://localhost:7000" env="prod":
#!/bin/bash
set -eu
# First run the release script to perform checks and migrations
Comment thread
rw251 marked this conversation as resolved.
Outdated
echo "Running release script (check and migrations)..."
docker compose exec {{ env }} bash /app/deploy/release.sh
# Then check if the service is responding
echo "Testing service connectivity..."
curl -I {{ host }} -s --compressed --fail --retry 20 --retry-delay 1 --retry-all-errors
4 changes: 2 additions & 2 deletions justfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -342,8 +342,8 @@ docker-exec *args="bash": _env


# run tests in docker container
docker-smoke-test host="http://localhost:7000": _env
{{ just_executable() }} docker/smoke-test {{ host }}
docker-smoke-test host="http://localhost:7000" env="prod": _env
{{ just_executable() }} docker/smoke-test {{ host }} {{env}}


# check migrations in the dev docker container
Expand Down