From Dan Kenigsberg <[email protected]>: Dan Kenigsberg has submitted this change and it was merged.
Change subject: net: Small cleanup of _add_network and _del_network ...................................................................... net: Small cleanup of _add_network and _del_network Removed unused arguments and made netinfo argument non optional. This patch includes also some cleanup for config_network_test which had to change to adjust to the changes. Change-Id: I0a493b2c6312a04cd258415cba1a537a4df4bf95 Signed-off-by: Edward Haas <[email protected]> --- M lib/vdsm/network/legacy_switch.py M lib/vdsm/network/netconfpersistence.py M tests/network/config_network_test.py 3 files changed, 61 insertions(+), 71 deletions(-) Approvals: Jenkins CI: Passed CI tests Petr Horáček: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Edward Haas: Verified -- To view, visit https://gerrit.ovirt.org/72652 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0a493b2c6312a04cd258415cba1a537a4df4bf95 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Edward Haas <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Edward Haas <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list -- [email protected] To unsubscribe send an email to [email protected]
