[ovs-dev] [PATCH] stream-ssl: Change "bootstrap race" log message from ERR to INFO.

Ben Pfaff blp at nicira.com
Mon Aug 13 18:00:51 UTC 2012


Thanks, pushed.

On Mon, Aug 13, 2012 at 10:59:41AM -0700, Justin Pettit wrote:
> Looks reasonable.
> 
> --Justin
> 
> 
> On Aug 13, 2012, at 10:54 AM, Ben Pfaff <blp at nicira.com> wrote:
> 
> > This situation can and will happen, and we handle it successfully, so it's
> > not an error.
> > 
> > Bug #12922.
> > Reported-by: Scott Hendricks <shendricks at nicira.com>
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> > AUTHORS          |    1 +
> > lib/stream-ssl.c |    2 +-
> > 2 files changed, 2 insertions(+), 1 deletions(-)
> > 
> > diff --git a/AUTHORS b/AUTHORS
> > index 6d7dc71..dee8931 100644
> > --- a/AUTHORS
> > +++ b/AUTHORS
> > @@ -158,6 +158,7 @@ Ramana Reddy            gtvrreddy at gmail.com
> > Rob Sherwood            rob.sherwood at bigswitch.com
> > Roger Leigh             rleigh at codelibre.net
> > Rogério Vinhal Nunes
> > +Scott Hendricks         shendricks at nicira.com
> > Sean Brady              sbrady at gtfservices.com
> > Sebastian Andrzej Siewior  sebastian at breakpoint.cc
> > Sébastien RICCIO        sr at swisscenter.com
> > diff --git a/lib/stream-ssl.c b/lib/stream-ssl.c
> > index db7b68e..4a9dd4f 100644
> > --- a/lib/stream-ssl.c
> > +++ b/lib/stream-ssl.c
> > @@ -479,7 +479,7 @@ ssl_connect(struct stream *stream)
> >              * certificate, but that's more trouble than it's worth.  These
> >              * connections will succeed the next time they retry, assuming that
> >              * they have a certificate against the correct CA.) */
> > -            VLOG_ERR("rejecting SSL connection during bootstrap race window");
> > +            VLOG_INFO("rejecting SSL connection during bootstrap race window");
> >             return EPROTO;
> >         } else {
> >             return 0;
> > -- 
> > 1.7.2.5
> > 
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
> 



More information about the dev mailing list