Peter V. Saveliev has posted comments on this change.

Change subject: watchdog hook
......................................................................


Patch Set 5: (2 inline comments)

....................................................
File vdsm_hooks/watchdog/before_vm_start.py
Line 28:         watchdogaction = watchdog.split(':')[1]
Line 29: 
Line 30:         devices = domxml.getElementsByTagName('devices')[0]
Line 31:         watchdog_device = domxml.createElement('watchdog')
Line 32:         watchdog_device.setAttribute('model',watchdogmodel)
space after comma
Line 33:         watchdog_device.setAttribute('action',watchdogaction)
Line 34: 
Line 35:         devices.appendChild(watchdog_device)
Line 36: 


Line 29: 
Line 30:         devices = domxml.getElementsByTagName('devices')[0]
Line 31:         watchdog_device = domxml.createElement('watchdog')
Line 32:         watchdog_device.setAttribute('model',watchdogmodel)
Line 33:         watchdog_device.setAttribute('action',watchdogaction)
the same
Line 34: 
Line 35:         devices.appendChild(watchdog_device)
Line 36: 
Line 37:         hooking.write_domxml(domxml)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc1d21ff8e14013869d78532632e8485f0414793
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Peter V. Saveliev <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to