Change in vdsm[master]: BZ#891300 Reload udev rules before calling udevadm trigger.

lyarwood at redhat.com lyarwood at redhat.com
Fri Jan 4 15:56:06 UTC 2013


Lee Yarwood has uploaded a new change for review.

Change subject: BZ#891300 Reload udev rules before calling udevadm trigger.
......................................................................

BZ#891300 Reload udev rules before calling udevadm trigger.

Change-Id: Ic32ee84b789c7bfaace9a7c96fa835c891b3342f
Signed-off-by: Lee Yarwood <lyarwood at redhat.com>
---
M vdsm/storage/hsm.py
M vdsm/supervdsmServer.py
2 files changed, 9 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/56/10656/1

diff --git a/vdsm/storage/hsm.py b/vdsm/storage/hsm.py
index efb2749..bc63645 100644
--- a/vdsm/storage/hsm.py
+++ b/vdsm/storage/hsm.py
@@ -2924,6 +2924,7 @@
         Warning: Internal use only.
         """
         supervdsm.getProxy().appropriateDevice(guid, thiefId)
+        supervdsm.getProxy().udevReloadRules()
         supervdsm.getProxy().udevTrigger(guid)
 
     @public
diff --git a/vdsm/supervdsmServer.py b/vdsm/supervdsmServer.py
index 5effd41..994d043 100755
--- a/vdsm/supervdsmServer.py
+++ b/vdsm/supervdsmServer.py
@@ -215,6 +215,14 @@
         return configNetwork.setSafeNetworkConfig()
 
     @logDecorator
+    def udevReloadRules(self):
+        cmd = [EXT_UDEVADM, "control", "--reload-rules"]
+        rc, out, err = misc.execCmd(cmd, sudo=False)
+        if rc:
+            raise OSError(errno.EINVAL, "Could not reload udev rules \
+                          out %s\nerr %s" % (out, err))
+
+    @logDecorator
     def udevTrigger(self, guid):
         cmd = [EXT_UDEVADM, 'trigger', '--verbose', '--action', 'change',
                '--property-match=DM_NAME=%s' % guid]


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic32ee84b789c7bfaace9a7c96fa835c891b3342f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Lee Yarwood <lyarwood at redhat.com>


More information about the vdsm-patches mailing list