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

db: Modify upgrade cmd to migrate database from Postgres to Sqlite #153

Closed
wants to merge 12 commits into from

Conversation

harishsurf
Copy link
Contributor

This is part 2 of feat: Sqlite database migration work. Part one PR: #152

This change adds ansible tasks to ./mirror-registry upgrade cmd to migrate data from a running Postgres container to SQlite file based database which is mounted as a volume to Quay container.
It also stores a backup of postgres data in {{ quay_root }}/quay-postgres-backup inside the host machine where the binary is run

@harishsurf harishsurf requested a review from dmage May 15, 2024 21:00
@harishsurf harishsurf added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label May 15, 2024
@dmage
Copy link
Contributor

dmage commented May 22, 2024

@jonathankingfc can you take a look?

@harishsurf harishsurf added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Jun 27, 2024
@harishsurf harishsurf added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Jun 28, 2024
@harishsurf harishsurf added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Jul 1, 2024
@harishsurf harishsurf added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Jul 9, 2024
@harishsurf harishsurf added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Jul 9, 2024
@harishsurf harishsurf added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Jul 10, 2024
@harishsurf harishsurf added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Jul 10, 2024
@harishsurf harishsurf added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Jul 10, 2024
quay needs to be on latest version with all alembic migration
changes before db is migrated from postgres to sqlite, else
alembic migration fails
@harishsurf harishsurf added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Jul 15, 2024
@harishsurf
Copy link
Contributor Author

PR #157 replaces this

@harishsurf harishsurf closed this Aug 9, 2024
@harishsurf harishsurf deleted the db-migrate branch August 9, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Indicates a non-member PR verified by an org member that is safe to test.
Development

Successfully merging this pull request may close these issues.

3 participants