Change in vdsm[ovirt-3.2]: udev: Race fix- load and trigger dev rule

amureini at redhat.com amureini at redhat.com
Sun Jan 27 13:33:12 UTC 2013


Allon Mureinik has posted comments on this change.

Change subject: udev: Race fix- load and trigger dev rule
......................................................................


Patch Set 1: Looks good to me, but someone else must approve

(1 inline comment)

....................................................
Commit Message
Line 12: udev functions - udevReloadRules() in supervdsmServer.py .
Line 13: Also added a check in appropriateDevice() (hsm.py) to make sure the
Line 14: mapping is indeed there.
Line 15: 
Line 16: Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=891300
This is rhev-m bug, which is already verified.
Let's get an oVirt bug for this, just so this patch makes sense?
Line 17: Change-Id: If3b2008a3d9df2dcaf54190721c2dd9764338627
Line 18: Signed-off-by: Lee Yarwood <lyarwood at redhat.com>
Line 19: Signed-off-by: Vered Volansky <vvolansk at redhat.com>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If3b2008a3d9df2dcaf54190721c2dd9764338627
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.2
Gerrit-Owner: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Lee Yarwood <lyarwood at redhat.com>


More information about the vdsm-patches mailing list