Change in vdsm[master]: lun: Serial attr should not passed to libvirt for lun disks.

mlipchuk at redhat.com mlipchuk at redhat.com
Wed Jan 13 08:59:32 UTC 2016


Maor Lipchuk has uploaded a new change for review.

Change subject: lun: Serial attr should not passed to libvirt for lun disks.
......................................................................

lun: Serial attr should not passed to libvirt for lun disks.

Following the change which was introduced at
libvirt-1.2.17-5 and qemu-kvm-rhev-2.3.0-18, libvirt now enforce
that a scsi-block 'lun' device should not provide the 'serial' property.

To comply with this change, this patch introduces a change
that a VM should ignore 'serial' property for lun devices when
passing the XML to libvirt.

This patch was verified by running a VM with lun disk which
had enabled SCSI Pass-Through and also plugging the same lun disk
while the VM was running. The operation succeeded.

Change-Id: I41abb2b7d7f5950e65710379a0655399014afd83
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
Tested-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M vdsm/virt/vmdevices/storage.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/66/51766/1

diff --git a/vdsm/virt/vmdevices/storage.py b/vdsm/virt/vmdevices/storage.py
index 3c4533d..8f3e021 100644
--- a/vdsm/virt/vmdevices/storage.py
+++ b/vdsm/virt/vmdevices/storage.py
@@ -364,7 +364,7 @@
             # they are readonly unless explicitely stated otherwise
             diskelem.appendChildWithArgs('readonly')
 
-        if hasattr(self, 'serial'):
+        if hasattr(self, 'serial') and (self.device != 'lun'):
             diskelem.appendChildWithArgs('serial', text=self.serial)
 
         if hasattr(self, 'bootOrder'):


-- 
To view, visit https://gerrit.ovirt.org/51766
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I41abb2b7d7f5950e65710379a0655399014afd83
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>


More information about the vdsm-patches mailing list