Dan Kenigsberg has submitted this change and it was merged.

Change subject: net test: test_ip_info - wait for IP settings with address
......................................................................


net test: test_ip_info - wait for IP settings with address

This is a second attempt to handle the random failing test on CI run.
This time, we are more specific in the 'waiting' statement, and include
the expected address part.

It is suspected that multiple duplicate netlink events are arriving,
causing a false waiting.

Change-Id: I1c2832c1ec7614e1df9c291a9f3ab2e4a1134242
Signed-off-by: Edward Haas <edwa...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/63660
Continuous-Integration: Jenkins CI
Reviewed-by: Petr Horáček <phora...@redhat.com>
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
---
M lib/vdsm/network/netlink/waitfor.py
M tests/network/netinfo_test.py
2 files changed, 30 insertions(+), 11 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/63660
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1c2832c1ec7614e1df9c291a9f3ab2e4a1134242
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwa...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Edward Haas <edwa...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phora...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to