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

Allow recovery of empty storage pools #12108

Merged
merged 3 commits into from
Aug 22, 2023

Conversation

MusicDin
Copy link
Member

@MusicDin MusicDin commented Aug 2, 2023

Currently, empty storage pools cannot be recovered, because the missing storage pool DB entries are restored during the recovery only if the unknown volumes are detected within the storage pool.

Not being able to recover empty storage pools, prevents the creation of a new storage pool with the same.

This PR changes the recovery process to first recover the missing storage pool DB entries and then the unknown volumes.

@MusicDin MusicDin force-pushed the fix/recover-empty-storage-pool branch 2 times, most recently from 2f2a976 to 1b37a1c Compare August 7, 2023 09:24
@MusicDin MusicDin marked this pull request as ready for review August 7, 2023 11:12
@tomponline
Copy link
Member

Hey @MusicDin please could you rebase this

@MusicDin MusicDin force-pushed the fix/recover-empty-storage-pool branch from 1b37a1c to 986b295 Compare August 10, 2023 07:38
Signed-off-by: Din Music <din.music@canonical.com>
Signed-off-by: Din Music <din.music@canonical.com>
Signed-off-by: Din Music <din.music@canonical.com>
@MusicDin MusicDin force-pushed the fix/recover-empty-storage-pool branch from 986b295 to 5083d3f Compare August 10, 2023 09:03
@tomponline tomponline merged commit e94ec83 into canonical:main Aug 22, 2023
25 checks passed
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