Yaniv Bronhaim has submitted this change and it was merged. Change subject: net sampling: drop reference to a non-existing function ......................................................................
net sampling: drop reference to a non-existing function Due to a sloppy backport of commit 1081e76, I have introduced to the stable ovirt-3.4 branch a call to the vlanSpeed function, which was never defined there (It was introduced in 386f9069d3, and was not backported) In my verification of the patch, I did not try a host with vlan devices, so I failed to recognise the bug. Change-Id: I01ee0fa79c77bbd33bf9d0cbf6bdfed13c8abe95 Label: ovirt-3.4-only Bug-Url: https://bugzilla.redhat.com/1121643 Signed-off-by: Dan Kenigsberg <[email protected]> Reviewed-on: http://gerrit.ovirt.org/30547 Reviewed-by: Antoni Segura Puimedon <[email protected]> Reviewed-by: Ondřej Svoboda <[email protected]> Reviewed-by: Stanislav Mikhalevich <[email protected]> Reviewed-by: Yaniv Bronhaim <[email protected]> --- M vdsm/sampling.py 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Stanislav Mikhalevich: Looks good to me, but someone else must approve Ondřej Svoboda: Looks good to me, but someone else must approve Yaniv Bronhaim: Looks good to me, approved Antoni Segura Puimedon: Looks good to me, but someone else must approve Dan Kenigsberg: Verified -- To view, visit http://gerrit.ovirt.org/30547 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I01ee0fa79c77bbd33bf9d0cbf6bdfed13c8abe95 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.4 Gerrit-Owner: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Stanislav Mikhalevich <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
