Bug 1235406
Summary: | virt-manager needs to import backing file on volume creation, so we can detect its format | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Community] Virtualization Tools | Reporter: | isomarcte | ||||||
Component: | virt-manager | Assignee: | Cole Robinson <crobinso> | ||||||
Status: | CLOSED UPSTREAM | QA Contact: | |||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | unspecified | ||||||||
Version: | unspecified | CC: | berrange, crobinso, fvzwieten, gscrivan, isomarcte, rbalakri, rhefner1 | ||||||
Target Milestone: | --- | ||||||||
Target Release: | --- | ||||||||
Hardware: | x86_64 | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2016-06-17 01:31:51 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
isomarcte
2015-06-24 17:35:52 UTC
Thanks for the report. Please attach virt-manager --debug output when reproducing the issue, namely specifying the backingstore for a new disk volume, attaching it to a VM/creating a VM, then starting the VM Created attachment 1042805 [details]
Output of virt-manager --debug with requested operations
Thanks. Yeah I think we just need to try to import the backing volume, so libvirt can tell us what format it is. Can you attach sudo virsh vol-dumpxml --pool default testBackingStore.qcow2 , just to confirm. note to self: when making this change, also log the XML after object creation. in fact we should do that when creating every object... Created attachment 1042807 [details]
XML for the testBackingStore.qcow2 image
Yeah that confirms it, libvirt just defaulting it to raw. So we will need to import the directory as a pool to determine the correct volume. We already have plumbing for that elsewhere so it shouldn't be too hard to wire up This is still a useful addition to virt-manager. Is there any work being done on it? Thanks! No one is actively working on it that I know of I just found that the backing_store option of virt-install has the same problem. It prevents using virt-install as a one-liner for creating a vm with a backing_store. Fixed upstream now: commit 5398282e12e12ac48dd10becea33456a483b7f5e Author: Cole Robinson <crobinso> Date: Thu Jun 16 20:36:30 2016 -0400 storage: Detect backing_store format automatically By attempting to manage/import the passed path. This makes it work via both virt-install and virt-manager https://bugzilla.redhat.com/show_bug.cgi?id=1235406 |