Giuseppe Vallarelli has posted comments on this change.
Change subject: Update device hooks:
......................................................................
Patch Set 1: (1 inline comment)
....................................................
File vdsm/libvirtvm.py
Line 1730: nic_to_update = self._get_device(params['alias'])
Line 1731: nic_xml =
nic_to_update.getXML().toprettyxml(encoding='utf-8')
Line 1732:
Line 1733: self.log.debug('Nic to update: %s' % nic_xml)
Line 1734: nic_xml = hooks.before_update_device(nic_xml, self.conf)
My wrong assumption was that I had to use updateDeviceFlags and provide the
updated nicXml.
Line 1735:
Line 1736: try:
Line 1737: self._dom.updateDeviceFlags(
Line 1738: nic_xml, libvirt.VIR_DOMAIN_AFFECT_LIVE)
--
To view, visit http://gerrit.ovirt.org/13972
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8fe7f5e6d8291cc227a323b324c977bcd5bc8295
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Giuseppe Vallarelli <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Giuseppe Vallarelli <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
