Bug 1055479

Summary: Libvirt lose sheepdogs volumes on pool refresh or restart
Product: [Community] Virtualization Tools Reporter: joel.simoes
Component: libvirtAssignee: Libvirt Maintainers <libvirt-maint>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: acathrow, joel.simoes, jtomko
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-04-17 10:33:18 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Possible patch none

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