Re: [ovs-dev] [PATCH] tests: Fix race condition waiting for monitor thread.

2014-04-24 Thread Joe Stringer
Looking again, it's probably better to change the lines which look for "monitor thread created" and "..terminated" to be OVS_WAIT_UNTIL. I plan to send a fresh version. On 25 April 2014 12:34, Joe Stringer wrote: > Occasionally, test #770 "ofproto-dpif - ofproto-dpif-monitor 1" would > fail, be

[ovs-dev] [PATCH] tests: Fix race condition waiting for monitor thread.

2014-04-24 Thread Joe Stringer
Occasionally, test #770 "ofproto-dpif - ofproto-dpif-monitor 1" would fail, because the testsuite looked in the logs for evidence of a thread being created, but it checked before vswitchd was able to spawn the thread. This patch fixes the race by waiting until there is a reasonable amount of logs