[ovs-dev] [PATCH] tests: Add valgrind targets for ovn utilities and dameons.

Guru Shetty guru at ovn.org
Thu May 12 17:04:40 UTC 2016


Thank you William and Ryan. I pushed this to master.

On 12 May 2016 at 09:32, William Tu <u9012063 at gmail.com> wrote:

> Thanks for adding this, I will re-run the OVN-related valgrind tests.
>
> On Thu, May 12, 2016 at 9:14 AM, Ryan Moats <rmoats at us.ibm.com> wrote:
>
> >
> >
> > "dev" <dev-bounces at openvswitch.org> wrote on 05/12/2016 10:23:39 AM:
> >
> > > From: Gurucharan Shetty <guru at ovn.org>
> > > To: dev at openvswitch.org
> > > Cc: Gurucharan Shetty <guru at ovn.org>
> > > Date: 05/12/2016 10:42 AM
> > > Subject: [ovs-dev] [PATCH] tests: Add valgrind targets for ovn
> > > utilities and dameons.
> > > Sent by: "dev" <dev-bounces at openvswitch.org>
> > >
> > > Signed-off-by: Gurucharan Shetty <guru at ovn.org>
> > > ---
> > >  tests/automake.mk |    4 ++++
> > >  1 file changed, 4 insertions(+)
> > >
> > > diff --git a/tests/automake.mk b/tests/automake.mk
> > > index a5c6074..211a80d 100644
> > > --- a/tests/automake.mk
> > > +++ b/tests/automake.mk
> > > @@ -152,6 +152,10 @@ check-lcov: all tests/atconfig tests/atlocal $
> > > (TESTSUITE) $(check_DATA) clean-lc
> > >  # valgrind support
> > >
> > >  valgrind_wrappers = \
> > > +   tests/valgrind/ovn-controller \
> > > +   tests/valgrind/ovn-nbctl \
> > > +   tests/valgrind/ovn-northd \
> > > +   tests/valgrind/ovn-sbctl \
> > >     tests/valgrind/ovs-appctl \
> > >     tests/valgrind/ovs-ofctl \
> > >     tests/valgrind/ovstest \
> > > --
> >
> > This makes a lot of sense to me, we should especially be checking the
> > daemons...
> >
> > Acked-by: Ryan Moats <rmoats at us.ibm.com>
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
> >
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list