On Wed, Nov 27, 2019 at 8:34 AM Numan Siddique <num...@ovn.org> wrote: > > On Sat, Nov 23, 2019 at 3:20 PM Frode Nordahl > <frode.nord...@canonical.com> wrote: > > > > At present when the ``pause`` command is issued any lock the > > ``ovn-northd`` process holds on the SB DB will be kept. > > > > To be able to change this behaviour the tests surrounding pause and > > resume must be changed, thus I provide the proposed change as a > > series to demonstrate that the changed tests work both for the > > current code and the proposed code. > > > > Travis passed: https://travis-ci.org/fnordahl/ovn/builds/615918643 > > > > Note that the failure in job 5.11 has it's origin from upstream > > openvswitch where the tests appear to fail on master atm. > > > > Frode Nordahl (2): > > northd: Amend ovn-northd pause resume test > > northd: Improve handling of pause and resume > > Thanks for the v3. This approach makes more sense. > > I applied this series to master.
Thank you for your review and merge. Cheers! -- Frode Nordahl > Thanks > Numan > > > > > northd/ovn-northd.8.xml | 9 +++-- > > northd/ovn-northd.c | 89 +++++++++++++++++++++++++++-------------- > > tests/ovn-northd.at | 35 +++++++++++----- > > 3 files changed, 89 insertions(+), 44 deletions(-) > > > > -- > > 2.24.0 > > > > _______________________________________________ > > dev mailing list > > d...@openvswitch.org > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev