Ayal Baron has posted comments on this change.
Change subject: udev: Race fix- load and trigger dev rule
......................................................................
Patch Set 7: (1 inline comment)
....................................................
File vdsm/supervdsmServer.py
Line 235: cmd = [EXT_UDEVADM, 'trigger', '--verbose', '--action',
'change',
Line 236: '--property-match=DM_NAME=%s' % guid]
Line 237: rc, out, err = misc.execCmd(cmd, sudo=False)
Line 238: if rc:
Line 239: _SuperVdsm.log.error("Udevadm trigger command failed
rc=%s, \
need to use self (everywhere)
Line 240: out=\"%s\", err=\"%s\"", rc, out, err)
Line 241: raise OSError(errno.EINVAL, "Could not trigger change for
device \
Line 242: %s" % guid)
Line 243:
--
To view, visit http://gerrit.ovirt.org/10758
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If3b2008a3d9df2dcaf54190721c2dd9764338627
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Lee Yarwood <[email protected]>
Gerrit-Reviewer: Royce Lv <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
