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

Implement MigrationService #2939

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

alvasw
Copy link
Contributor

@alvasw alvasw commented Oct 20, 2024

The MigrationService allows us to perform necessary migrations when upgrading to a new Bisq version. So far, the only option is to try to perform the migrations on each run. The MigrationService runs the necessary migrations if the data directory version is below the app version.

I tested this PR on:

  • an instance with a fresh data directory
  • an instance with a data directory of v2.1.1 (applies migration)
  • an instance with an already migrated data directory (v2.1.2)

This feature is needed to remove redundant Tor files after merging PR #2938. A follow-up PR will contain the required changes.

Changes:

Tor failed to start for some users because their operating system didn't
ship Tor's dependencies. This change instructs the package manager to
install tor and all its required during the Bisq installation.

Fixes bisq-network#2906
During Bisq's installation the OS's package manager automatically
installs Tor and its required dependecies.
During Bisq's installation the OS's package manager automatically
installs Tor and its required dependecies.
The MigrationService allows us to perform necessary migrations when
upgrading to a new Bisq version. So far, the only option is to try to
perform the migrations on each run. The MigrationService runs the
necessary migrations if the data directory version is below the app
version.
- Test with data directory before MigrationService was introduced
- Test with invalid data directory version
- Test with valid data directory version
This migration writes the current version number to the data directory.
The remaining migrations depend on another open PR and will follow soon.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant