On Tue, May 23, 2017 at 7:05 AM, Alin Serdean <[email protected]> wrote: > From: Alin Serdean <[email protected]> > > 'multiple bridges share a controller' hangs on windows because it is > lacking the exit information (it will hang when the test has finished) > > Introduce a pidfile to 'ovs-testcontroller' and end it on exit based on > the pidfile. > > Signed-off-by: Alin Gabriel Serdean <[email protected]>
Thanks Alin for the fix and Joe for the review. It looks good to me. Acked-by: Andy Zhou <[email protected]> I will push it in a few minutes. > --- > v2: move 'on_exit' after 'ovs-testcontroller' creation (Joe Stringer) > --- > tests/bridge.at | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tests/bridge.at b/tests/bridge.at > index cc7619d..f6c2327 100644 > --- a/tests/bridge.at > +++ b/tests/bridge.at > @@ -48,7 +48,8 @@ OVS_VSWITCHD_START( > set bridge br1 datapath-type=dummy other-config:datapath-id=1234 ]) > > dnl Start ovs-testcontroller > -AT_CHECK([ovs-testcontroller --detach punix:controller], [0], [ignore]) > +AT_CHECK([ovs-testcontroller --detach punix:controller --pidfile], [0], > [ignore]) > +on_exit 'kill `cat ovs-testcontroller.pid`' > OVS_WAIT_UNTIL([test -e controller]) > > dnl Add the controller to both bridges, 5 seconds apart. > -- > 2.10.2.windows.1 > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
