On 17 May 2017 at 11:37, Andy Zhou <az...@ovn.org> wrote: > Without the fix, this test currently consistently fail when running > on Travis CI. Connecting to the controller can take more time than > running locally. Because the exact connecting time is variable, the > exact output should not be used for correctness checking. > > Fixes: 85c55772a453(bridge: Fix controller status update) > Signed-off-by: Andy Zhou <az...@ovn.org> > ---
Thanks, this seems to improve the situation. > tests/bridge.at | 23 +++++++++++++++-------- > 1 file changed, 15 insertions(+), 8 deletions(-) > > diff --git a/tests/bridge.at b/tests/bridge.at > index 58b27d445062..cc7619d3f035 100644 > --- a/tests/bridge.at > +++ b/tests/bridge.at > @@ -49,23 +49,30 @@ OVS_VSWITCHD_START( > > dnl Start ovs-testcontroller > AT_CHECK([ovs-testcontroller --detach punix:controller], [0], [ignore]) > +OVS_WAIT_UNTIL([test -e controller]) > > dnl Add the controller to both bridges, 5 seconds apart. > AT_CHECK([ovs-vsctl set-controller br0 unix:controller]) > +AT_CHECK([ovs-vsctl set-fail-mode br0 secure]) > AT_CHECK([ovs-appctl time/warp 5000], [0], [ignore]) > AT_CHECK([ovs-vsctl set-controller br1 unix:controller]) > +AT_CHECK([ovs-vsctl set-fail-mode br1 secure]) > > -dnl Wait for the controller connection to come up > -for i in `seq 0 7` > +dnl Wait for the controller connectionsi to be up > +for i in `seq 0 19` > do > - AT_CHECK([ovs-appctl time/warp 10], [0], [ignore]) > + if ovs-vsctl --column=is_connected list controller |grep "false"; then > + : > + else > + break > + fi > + ovs-appctl time/warp 1100 > done Can we substitute this for something like OVS_WAIT_WHILE([ovs-vsctl --column=is_connected list controller | grep "false"]) ? _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev