From Dan Kenigsberg <[email protected]>: Dan Kenigsberg has submitted this change and it was merged.
Change subject: net test: Fix test_ipv4_default_route default route assertion ...................................................................... net test: Fix test_ipv4_default_route default route assertion On some functional test runs, the test fails on the default route assertion, which checks that there is no default route at the end of the test. The checks is wrong, as it should actually check that the configured default route is not the one on the setup. The setup itself may have an original default route already or none. Note: It seems that we do not (always) preserve the default route that was originally set on the host. This should be fixed in future patches. Change-Id: Iacfce818f8faa35d533bdd8c0d6a784b4fc026d1 Signed-off-by: Edward Haas <[email protected]> --- M tests/functional/networkTests.py 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Edward Haas: Verified -- To view, visit https://gerrit.ovirt.org/69291 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iacfce818f8faa35d533bdd8c0d6a784b4fc026d1 Gerrit-PatchSet: 1 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: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list -- [email protected] To unsubscribe send an email to [email protected]
