Thanks for adding this, I will re-run the OVN-related valgrind tests.

On Thu, May 12, 2016 at 9:14 AM, Ryan Moats <rmo...@us.ibm.com> wrote:

>
>
> "dev" <dev-boun...@openvswitch.org> wrote on 05/12/2016 10:23:39 AM:
>
> > From: Gurucharan Shetty <g...@ovn.org>
> > To: dev@openvswitch.org
> > Cc: Gurucharan Shetty <g...@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-boun...@openvswitch.org>
> >
> > Signed-off-by: Gurucharan Shetty <g...@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 <rmo...@us.ibm.com>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to