On Mon, Mar 21, 2016 at 6:22 PM, Justin Pettit <jpet...@ovn.org> wrote:

>
> > On Mar 21, 2016, at 8:10 AM, Russell Bryant <russ...@ovn.org> wrote:
> >
> > ovn-northd used to only call ovnsb_db_run() and ovnnb_db_run() when
> > seqnos indicated that they had changed.  It now always calls these
> > functions from the main loop.  Remove this outdated log message that
> > indicates that we called this function because contents changed.
> >
> > Signed-off-by: Russell Bryant <russ...@ovn.org>
> > ---
> > ovn/northd/ovn-northd.c | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c
> > index bf8cde4..b9a6272 100644
> > --- a/ovn/northd/ovn-northd.c
> > +++ b/ovn/northd/ovn-northd.c
> > @@ -2157,7 +2157,6 @@ ovnnb_db_run(struct northd_context *ctx)
> >     if (!ctx->ovnsb_txn) {
> >         return;
> >     }
> > -    VLOG_DBG("ovn-nb db contents may have changed.");
>
> Should the similar debug message from ovnsb_db_run() also be removed?
>

Yes.  :-)


> Acked-by: Justin Pettit <jpet...@ovn.org>
>

Thanks!  I removed the other message and applied this to master.

-- 
Russell Bryant
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to