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