Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Add Django database migration to Dockerfile entrypoint #1048

Closed
AetherUnbound opened this issue Dec 15, 2022 · 0 comments · Fixed by #1049
Closed

Add Django database migration to Dockerfile entrypoint #1048

AetherUnbound opened this issue Dec 15, 2022 · 0 comments · Fixed by #1049
Assignees
Labels
💻 aspect: code Concerns the software code in the repository 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🟨 priority: medium Not blocking but should be addressed soon 🐳 tech: docker Requires familiarity with Docker

Comments

@AetherUnbound
Copy link
Contributor

Description

As part of the ECS-ification of the Django service, we are moving towards an automated database migration handling approach. Under this new approach, migrations will be automatically applied when a new version of the Django application is deployed. The easiest way to accomplish this would be to add it as an additional step in the Dockerfile entrypoint.

We may want to explore also adding an environment variable which could be used as a flag to determine if the migrations should be run. This would default to true for production, but could be turned off in order to speed up local development (and prevent issuing a migration before it's fully baked during local testing).

Alternatives

Additional context

See the motivation for this change and the rest of the discussion here: #1030

@AetherUnbound AetherUnbound added 🐳 tech: docker Requires familiarity with Docker 💻 aspect: code Concerns the software code in the repository 🟨 priority: medium Not blocking but should be addressed soon 🧰 goal: internal improvement Improvement that benefits maintainers, not users labels Dec 15, 2022
@AetherUnbound AetherUnbound self-assigned this Dec 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💻 aspect: code Concerns the software code in the repository 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🟨 priority: medium Not blocking but should be addressed soon 🐳 tech: docker Requires familiarity with Docker
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant