Dan Kenigsberg has submitted this change and it was merged. Change subject: net tests: wait for a concurrent iptables call to finish ......................................................................
net tests: wait for a concurrent iptables call to finish test_local_auto_with_dynamic_address_from_ra may execute in parallel on the same machine. This patch waits until a concurrent iptables call finishes instead of failing immediately. The down side is that a waiting test may starve forever, but I do not believe it is a real problem, as iptables executions are rare and short. This patch also plays with indentation to keep el7's pep8 happy. Change-Id: Ib95f679558dd0553b24fb16589beac76991f89bf Signed-off-by: Dan Kenigsberg <[email protected]> Reviewed-on: https://gerrit.ovirt.org/55904 Continuous-Integration: Jenkins CI Reviewed-by: Edward Haas <[email protected]> --- M tests/network/firewall.py 1 file changed, 10 insertions(+), 6 deletions(-) Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Verified; Looks good to me, approved Edward Haas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/55904 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib95f679558dd0553b24fb16589beac76991f89bf Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <[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] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
