Martin Polednik has posted comments on this change.

Change subject: supervdsm: move udev-related functions to udev.py
......................................................................


Patch Set 6:

Logs from similar run:

MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:20,181::udev::67::SuperVdsm.ServerCallback::(appropriateMultipathDevice) 
Creating rule 
/etc/udev/rules.d/99-vdsm-36001405140d1aa6e14344aba634d08c6-9954ff1d-d008-4453-8615-b0b6761ef93f.rules:
 u'SYMLINK=="mapper/36001405140d1aa6e14344aba634d08c6", RUN+="/usr/bin/chown 
vdsm:qemu $env{DEVNAME}"\n'
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:20,181::supervdsmServer::99::SuperVdsm.ServerCallback::(wrapper) return 
appropriateMultipathDevice with None
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:20,182::supervdsmServer::92::SuperVdsm.ServerCallback::(wrapper) call 
udevTriggerMultipath with () {}
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:20,182::commands::70::root::(execCmd) /usr/bin/taskset --cpu-list 0-19 
/sbin/udevadm control --reload (cwd None)
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:20,187::commands::88::root::(execCmd) SUCCESS: <err> = ''; <rc> = 0
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:20,187::commands::70::root::(execCmd) /usr/bin/taskset --cpu-list 0-19 
/sbin/udevadm trigger --verbose --action change 
--property-match=DM_NAME=36001405140d1aa6e14344aba634d08c6 (cwd None)
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:20,231::commands::88::root::(execCmd) SUCCESS: <err> = ''; <rc> = 0
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:20,231::supervdsmServer::99::SuperVdsm.ServerCallback::(wrapper) return 
udevTriggerMultipath with None
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,241::supervdsmServer::92::SuperVdsm.ServerCallback::(wrapper) call 
appropriateUSBDevice with ('2',) {}
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,242::udev::151::SuperVdsm.ServerCallback::(appropriateUSBDevice) 
Creating rule /etc/udev/rules.d/99-vdsm-usb_1_2.rules: 'SUBSYSTEM=="usb", 
ATTRS{busnum}=="1", ATTRS{devnum}=="2", OWNER:="qemu", GROUP:="qemu"\n'
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,242::commands::70::root::(execCmd) /usr/bin/taskset --cpu-list 0-19 
/sbin/udevadm control --reload (cwd None)
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,248::commands::88::root::(execCmd) SUCCESS: <err> = ''; <rc> = 0
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,248::commands::70::root::(execCmd) /usr/bin/taskset --cpu-list 0-19 
/sbin/udevadm trigger --verbose --action change --attr-match=busnum=1 
--attr-match=devnum=2 (cwd None)
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,269::commands::88::root::(execCmd) SUCCESS: <err> = ''; <rc> = 0
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,269::supervdsmServer::99::SuperVdsm.ServerCallback::(wrapper) return 
appropriateUSBDevice with None
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,270::supervdsmServer::92::SuperVdsm.ServerCallback::(wrapper) call 
appropriateUSBDevice with ('4',) {}
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,271::udev::151::SuperVdsm.ServerCallback::(appropriateUSBDevice) 
Creating rule /etc/udev/rules.d/99-vdsm-usb_1_4.rules: 'SUBSYSTEM=="usb", 
ATTRS{busnum}=="1", ATTRS{devnum}=="4", OWNER:="qemu", GROUP:="qemu"\n'
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,271::commands::70::root::(execCmd) /usr/bin/taskset --cpu-list 0-19 
/sbin/udevadm control --reload (cwd None)
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,276::commands::88::root::(execCmd) SUCCESS: <err> = ''; <rc> = 0
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,276::commands::70::root::(execCmd) /usr/bin/taskset --cpu-list 0-19 
/sbin/udevadm trigger --verbose --action change --attr-match=busnum=1 
--attr-match=devnum=4 (cwd None)
MainProcess|Thread-49::DEBUG::2016-03-19 
22:11:21,296::commands::88::root::(execCmd) SUCCESS: <err> = ''; <rc> = 0

(8e53f0f22f3b52f39f4b47d273c279c9b27f1156)

As mentioned before, this tests appropriate{Multipath, USB, VFIO}Device.

-- 
To view, visit https://gerrit.ovirt.org/54458
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If8c4a5120887c2cbf13463f33fad2ef5e002e0d6
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to