Dan Kenigsberg has submitted this change and it was merged. Change subject: Do not attempt to run hook with an unknown vnic ......................................................................
Do not attempt to run hook with an unknown vnic If hotunplugNic() is called with a badly-specified vnic, we should not run the after_nic_hotunplug_fail() hook, as we cannot supply its required param: nicXml is unassigned at this stage. Change-Id: I0e4f1d8e0b7f83324edd903f9204bb05fdd19626 Bug-Url: https://bugzilla.redhat.com/919356 Signed-off-by: Dan Kenigsberg <[email protected]> --- M vdsm/libvirtvm.py 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Antoni Segura Puimedon: Looks good to me, but someone else must approve Mark Wu: Looks good to me, but someone else must approve Assaf Muller: Verified; Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Itzik Brown: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/12891 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e4f1d8e0b7f83324edd903f9204bb05fdd19626 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Assaf Muller <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Itzik Brown <[email protected]> Gerrit-Reviewer: Mark Wu <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
