Bug 1055479 - Libvirt lose sheepdogs volumes on pool refresh or restart
Summary: Libvirt lose sheepdogs volumes on pool refresh or restart
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Virtualization Tools
Classification: Community
Component: libvirt
Version: unspecified
Hardware: Unspecified
OS: Linux
unspecified
unspecified
Target Milestone: ---
Assignee: Libvirt Maintainers
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-01-20 11:16 UTC by joel.simoes
Modified: 2016-01-04 13:11 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-04-17 10:33:18 UTC
Embargoed:


Attachments (Terms of Use)
Possible patch (4.08 KB, patch)
2014-01-20 11:18 UTC, joel.simoes
no flags Details | Diff

Description joel.simoes 2014-01-20 11:16:50 UTC
Description of problem:

Libvirt lose all volumes on refresh

Patch driver on attachment.


src/storage/storage_backend_sheepdog.c

Comment 1 joel.simoes 2014-01-20 11:18:03 UTC
Created attachment 852715 [details]
Possible patch

Comment 2 Ján Tomko 2014-01-20 12:51:53 UTC
Thank you for the patch, could you please send it to libvir-list? (See http://libvirt.org/hacking.html for instructions) It's much more likely to get noticed and reviewed there.

Comment 3 Ján Tomko 2014-04-17 10:33:18 UTC
Pushed upstream as:
commit 97410063337801048125e3945ed4713890aac762
Author:     Joel SIMOES <joel.simoes>
AuthorDate: 2014-02-11 11:25:33 +0000
Commit:     Daniel P. Berrange <berrange>
CommitDate: 2014-02-11 11:32:04 +0000

    Libvirt lose sheepdogs volumes on pool refresh or restart.
    When restarting sheepdog pool, all volumes are missing.
    This patch add automatically all volume from the added pool.
    
    Adding last Daniel P. Berrange's syntaxes correction.
    Adding vol on separeted function 'inspired' from parallels_storage :
    parallelsAddDiskVolume

git describe: v1.2.1-150-g9741006 contains: v1.2.2-rc1~142

Comment 4 joel.simoes 2016-01-04 12:52:14 UTC
Ok


Note You need to log in before you can comment on or make changes to this bug.