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 migration test workflow #4973

Merged

Conversation

SchrodingersGat
Copy link
Member

@SchrodingersGat SchrodingersGat commented Jun 5, 2023

Adds a new CI workflow step.

Test that the database migrations apply cleanly to an existing "production" database which has all previous migrations applied.

Keep track of "production" databases at each stable release, to ensure that updates from previous stable installations won't immediately fail.


Future work: Keep track of "master" production database, to ensure latest commits merge and migrate correctly. Push any changes to the migrations back upstream.


Ref: #4984

@SchrodingersGat SchrodingersGat added the CI CI / unit testing ecosystem label Jun 5, 2023
@SchrodingersGat SchrodingersGat added this to the 0.12.0 milestone Jun 5, 2023
@wolflu05
Copy link
Contributor

wolflu05 commented Jun 5, 2023

I can recommend using tmate, that way you don't need to push and wait for new ideas to fix this.

@matmair
Copy link
Contributor

matmair commented Jun 5, 2023

@SchrodingersGat is there a reason to keep this separate from QC or is this just for testing?

@SchrodingersGat SchrodingersGat added the migration Data or schema migrations label Jun 11, 2023
@SchrodingersGat SchrodingersGat merged commit 66ec82d into inventree:master Jun 20, 2023
13 checks passed
@SchrodingersGat SchrodingersGat deleted the production-db-testing branch June 20, 2023 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI CI / unit testing ecosystem migration Data or schema migrations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants