Bug 1168453
Summary: | Disk should be removed while using no-exist 'file' type volume with startupPolicy='optional' | ||
---|---|---|---|
Product: | Red Hat Enterprise Linux 7 | Reporter: | Pei Zhang <pzhang> |
Component: | libvirt | Assignee: | Martin Kletzander <mkletzan> |
Status: | CLOSED ERRATA | QA Contact: | Virtualization Bugs <virt-bugs> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 7.1 | CC: | dyuan, mzhan, rbalakri, xuzhang, yisun |
Target Milestone: | rc | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | libvirt-2.0.0-4.el7 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2016-11-03 18:11:19 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: |
Description
Pei Zhang
2014-11-27 02:26:19 UTC
Fixed upstream with v2.1.0-rc1-30-ge2705cfb6e50..v2.1.0-rc1-32-ga2b97a8d912a: commit e2705cfb6e50039a5f1d1c620f7ed18b37eca36d Author: Martin Kletzander <mkletzan> Date: Mon Aug 1 17:52:02 2016 +0200 qemu: Make qemuDomainCheckDiskStartupPolicy self-contained commit 779a4ea9066560b92b0df462b5a6d5c127c15920 Author: Martin Kletzander <mkletzan> Date: Mon Aug 1 17:56:26 2016 +0200 qemu: Remove unnecessary label and its only reference commit a2b97a8d912a37ba1935bf98be4c332f42332236 Author: Martin Kletzander <mkletzan> Date: Mon Aug 1 17:51:28 2016 +0200 qemu: Fix support for startupPolicy with volume/pool disks Verified on libvirt-2.0.0-4.el7.x86_64n & qemu-kvm-rhev-2.6.0-17.el7.x86_64 PASSED steps: # virsh dumpxml --inactive virtlab_test | grep disk -a10 ... <disk type='volume' device='disk'> <driver name='qemu' type='qcow2'/> <source pool='tmpdir' volume='qcow2.img' startupPolicy='optional'/> <target dev='vdb' bus='virtio'/> <address type='pci' domain='0x0000' bus='0x00' slot='0x0a' function='0x0'/> </disk> ... # virsh vol-list tmpdir Name Path ------------------------------------------------------------------------------ qcow2.img /tmp/dir/qcow2.img # virsh dumpxml virtlab_test | grep disk -a10 ... <disk type='volume' device='disk'> <driver name='qemu' type='qcow2'/> <source pool='tmpdir' volume='qcow2.img' startupPolicy='optional'/> <backingStore/> <target dev='vdb' bus='virtio'/> <alias name='virtio-disk1'/> <address type='pci' domain='0x0000' bus='0x00' slot='0x0a' function='0x0'/> </disk> ... # mv /tmp/dir/qcow2.img /tmp/dir/qcow2.img.bkup # virsh pool-refresh tmpdir Pool tmpdir refreshed # virsh vol-list tmpdir Name Path ------------------------------------------------------------------------------ qcow2.img.bkup /tmp/dir/qcow2.img.bkup # virsh destroy virtlab_test Domain virtlab_test destroyed # virsh start virtlab_test Domain virtlab_test started # virsh dumpxml virtlab_test | grep disk -a10 ... <the volume not listed as not existed anymore > ... #virsh edit virtlab_test //change the pool name to a non-existing name ... <disk type='volume' device='disk'> <driver name='qemu' type='qcow2'/> <source pool='non-exiting-pool' volume='qcow2.img' startupPolicy='optional'/> <target dev='vdb' bus='virtio'/> <address type='pci' domain='0x0000' bus='0x00' slot='0x0a' function='0x0'/> </disk> ... # virsh destroy virtlab_test virDomain virtlab_test destroyed # virsh start virtlab_test Domain virtlab_test started # virsh dumpxml virtlab_test | grep disk -a10 ... <the volume not listed as the pool not existed> ... Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://rhn.redhat.com/errata/RHSA-2016-2577.html |