Change in vdsm[master]: clientIF: rescan devices on failed hotplugDisk

ykaplan at redhat.com ykaplan at redhat.com
Sun Mar 31 07:49:21 UTC 2013


Yeela Kaplan has uploaded a new change for review.

Change subject: clientIF: rescan devices on failed hotplugDisk
......................................................................

clientIF: rescan devices on failed hotplugDisk

Change-Id: Ie5252a8ba6bf76c9eda3588c665ad379da3e9cfc
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=923773
Signed-off-by: Yeela Kaplan <ykaplan at redhat.com>
---
M vdsm/clientIF.py
1 file changed, 4 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/54/13454/1

diff --git a/vdsm/clientIF.py b/vdsm/clientIF.py
index ef9a19b..d669d86 100644
--- a/vdsm/clientIF.py
+++ b/vdsm/clientIF.py
@@ -276,13 +276,14 @@
 
             # GUID drive format
             elif "GUID" in drive:
-                volPath = os.path.join("/dev/mapper", drive["GUID"])
-
-                if not os.path.exists(volPath):
+                visibility = self.irs.getDevicesVisibility(drive["GUID"])
+		if visibility[drive["GUID"]] == False:
+		    self.log.error("GUID: %s is not visible", drive["GUID"])
                     raise vm.VolumeError(drive)
 
                 res = self.irs.appropriateDevice(drive["GUID"], vmId)
                 if res['status']['code']:
+		    self.log.error("Appropriate device %s failed", drive["GUID"])
                     raise vm.VolumeError(drive)
 
             # UUID drive format


--
To view, visit http://gerrit.ovirt.org/13454
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie5252a8ba6bf76c9eda3588c665ad379da3e9cfc
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykaplan at redhat.com>


More information about the vdsm-patches mailing list