[ovs-dev] [PATCH ovn v3 0/2] northd: Improve pause, resume and status

Frode Nordahl frode.nordahl at canonical.com
Wed Nov 27 07:45:54 UTC 2019


On Wed, Nov 27, 2019 at 8:34 AM Numan Siddique <numans at ovn.org> wrote:
>
> On Sat, Nov 23, 2019 at 3:20 PM Frode Nordahl
> <frode.nordahl at 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
> > dev at openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >


More information about the dev mailing list