Skip to content
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.

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

Add db migrations #320

Merged
merged 5 commits into from
Sep 28, 2024
Merged

Add db migrations #320

merged 5 commits into from
Sep 28, 2024

Conversation

mfisher87
Copy link
Contributor

@mfisher87 mfisher87 commented Sep 27, 2024

Resolves #81

Our first migration implements changes from #301 into the live DB.

TODO:

  • Ensure we have database backup and restore scripted for the ops team. Support database backup and restore #79
  • Add a manual database backup to deployment script. Just in case! The DB and app are required by our operations team to be running on independent VMs, so running the backup from the deployment script won't really make sense with the time we have. We will need to request our operators to do backups and restores on the database VM itself when we submit deployment tickets, as the first step. I think we could make some progress on automating this in the long term, but there's infrastructure momentum to overcome.
  • Backup the prod database and restore it into pre-prod
  • Test deployment with migration in pre-prod.

📚 Documentation preview 📚: https://usaon-benefit-tool--320.org.readthedocs.build/en/320/

@mfisher87 mfisher87 requested a review from rmarow September 27, 2024 01:50
@mfisher87 mfisher87 force-pushed the add-db-migrations branch 3 times, most recently from 3bcca95 to 7061803 Compare September 27, 2024 23:40
@mfisher87 mfisher87 marked this pull request as ready for review September 27, 2024 23:45
@mfisher87 mfisher87 force-pushed the add-db-migrations branch 2 times, most recently from 7120534 to 859706d Compare September 28, 2024 00:00
@mfisher87
Copy link
Contributor Author

Migration in QA successful

@mfisher87 mfisher87 merged commit ca098ff into main Sep 28, 2024
8 checks passed
@mfisher87 mfisher87 deleted the add-db-migrations branch September 28, 2024 00:03
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.

Support database migrations with Alembic
1 participant