Skip to content

fix(dashmate): "volume is in use" and "no such volume" - #1005

Merged
shumkov merged 1 commit into
v0.24-devfrom
fix/dashmate/remove_volume_issues
Apr 30, 2023
Merged

fix(dashmate): "volume is in use" and "no such volume"#1005
shumkov merged 1 commit into
v0.24-devfrom
fix/dashmate/remove_volume_issues

Conversation

@shumkov

@shumkov shumkov commented Apr 30, 2023

Copy link
Copy Markdown
Collaborator

Issue being fixed or feature implemented

During platform reset there are two issues happening:

(HTTP code 404) no such volume - get dash_masternode_testnet_drive_tenderdash: no such volume
(HTTP code 409) conflict - remove dash_masternode_testnet_drive_abci_data: volume is in use - [cab842c8b25f0589e80f5daad85faaf8b794489da200657d504c93e9b7039d22]

What was done?

  • Added retry logic to handle those errors

How Has This Been Tested?

Manually

Breaking Changes

None

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@shumkov
shumkov requested a review from antouhou as a code owner April 30, 2023 05:27
@shumkov
shumkov merged commit eb35f55 into v0.24-dev Apr 30, 2023
@shumkov
shumkov deleted the fix/dashmate/remove_volume_issues branch April 30, 2023 05:38
QuantumExplorer added a commit that referenced this pull request Sep 8, 2026
GroveDB #948 and rust-dashcore #1005 are merged. Point at the merge
commits on grovedb develop (985ece65) and rust-dashcore dev (057e79cd)
instead of the pull-request heads. The rust-dashcore tree is identical;
the grovedb merge only adds the #947 release workflow change on top of
the previously pinned tree.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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