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

fix: save rescanning progress #472

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

sander2
Copy link
Member

@sander2 sander2 commented May 17, 2023

To address #471 .

Note that I went with another approach - rather than storing data in a hacky way inside the bitcoin wallet, I store it in a rocksdb (since we'll be stateful anyway with #246). I wrote it in such a way that the database can be removed without any bad side-effects: it will just rescan everything again. This means that users don't necessarily have to copy over the database when migrating their vaults to other servers

@sander2 sander2 force-pushed the fix/save-rescanning-progress branch from 90ba3dc to 520ca00 Compare May 23, 2023 09:56
vault/src/issue.rs Show resolved Hide resolved
vault/src/issue.rs Outdated Show resolved Hide resolved
vault/src/issue.rs Outdated Show resolved Hide resolved
Comment on lines +134 to +139
let _vault_id_manager = VaultIdManager::from_map(
new_vault_provider.clone(),
new_btc_rpc_master_wallet,
btc_rpcs,
"test_replace_succeeds1",
);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't look like we need this, or is the construction required?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's just to ensure that tests don't interfere with each other by reading eath other's databases. We could use something like tempdir but I think for now this is fine

@sander2 sander2 force-pushed the fix/save-rescanning-progress branch from 520ca00 to d1402d6 Compare May 23, 2023 14:35
@sander2 sander2 force-pushed the fix/save-rescanning-progress branch from d1402d6 to bba8a70 Compare June 5, 2023 08:16
@sander2 sander2 force-pushed the fix/save-rescanning-progress branch from bba8a70 to c8538f1 Compare June 5, 2023 09:12
@gregdhill gregdhill merged commit d8826bb into interlay:master Jun 5, 2023
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.

2 participants