]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
tests: qemustatusxml2xml: Fix disk target mess
authorPeter Krempa <pkrempa@redhat.com>
Sat, 14 Sep 2019 07:54:07 +0000 (09:54 +0200)
committerPeter Krempa <pkrempa@redhat.com>
Mon, 16 Sep 2019 09:50:28 +0000 (11:50 +0200)
There were accidentally two disks with 'vdc' target with corresponding
blockjobs which made libvirt leak some references as there are not
supposed to be two blockjobs for a single disk. Fix this mess by
renaming some of the disks.

In addition the block job names also didn't correspond to the naming
convetion which also includes the disk target. Fix it as well.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
ACKed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Martin Kletzander <mkletzan@redhat.com>
tests/qemustatusxml2xmldata/blockjob-blockdev-in.xml

index 67252a3729d0bc057f33eb0fb206c721da0013c5..e9031b70874176c4dce69b3100df1424553b60cf 100644 (file)
       <disk dst='vdb'/>
       <base node='libvirt-14-format'/>
     </blockjob>
-    <blockjob name='pull-vdd-libvirt-17-format' type='active-commit' state='ready'>
-      <disk dst='vdd'/>
+    <blockjob name='commit-vde-libvirt-17-format' type='active-commit' state='ready'>
+      <disk dst='vde'/>
       <base node='libvirt-19-format'/>
       <top node='libvirt-17-format'/>
     </blockjob>
         </source>
       </src>
     </blockjob>
-    <blockjob name='pull-vdc-libvirt-4321-format' type='copy' state='ready' shallownew='yes'>
-      <disk dst='vdc'/>
+    <blockjob name='copy-vdd-libvirt-4321-format' type='copy' state='ready' shallownew='yes'>
+      <disk dst='vdd'/>
     </blockjob>
-    <blockjob name='pull-vdc-libvirt-9-format' type='commit' state='running'>
+    <blockjob name='commit-vdc-libvirt-9-format' type='commit' state='running'>
       <disk dst='vdc'/>
       <base node='libvirt-11-format'/>
       <top node='libvirt-9-format'/>
             </privateData>
           </source>
         </mirror>
-        <target dev='vdc' bus='virtio'/>
+        <target dev='vdd' bus='virtio'/>
         <alias name='virtio-disk3'/>
         <address type='pci' domain='0x0000' bus='0x00' slot='0x0d' function='0x0'/>
         <privateData>
           <format type='qcow2'/>
           <source file='/tmp/activecommit2.qcow2'/>
         </mirror>
-        <target dev='vdd' bus='virtio'/>
+        <target dev='vde' bus='virtio'/>
         <alias name='virtio-disk3'/>
         <address type='pci' domain='0x0000' bus='0x00' slot='0x0e' function='0x0'/>
         <privateData>