Dan Kenigsberg has submitted this change and it was merged. Change subject: net test: sourceroute thread crashes due to a test race ......................................................................
net test: sourceroute thread crashes due to a test race testRestoreToBlockingDHCP has caused sourceroute thread to crash. The test got into condition where we had two dhclient processes monitoring a single interface. Both are woken up by a dhcp server response, and write the *same* file /var/run/vdsm/sourceRoutes/$timeStamp supervdsm's inotify is woken up twice, but has only one file to handle. Change-Id: I1f4db3b9dbd846c46c29af8ce097e9942609ce0e Signed-off-by: Edward Haas <edwa...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/62995 Reviewed-by: Petr Horáček <phora...@redhat.com> Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M tests/functional/networkTests.py 1 file changed, 6 insertions(+), 0 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/62995 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f4db3b9dbd846c46c29af8ce097e9942609ce0e Gerrit-PatchSet: 4 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 https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org