secchan: Clarify log message about failing open.
authorJustin Pettit <jpettit@nicira.com>
Tue, 11 Aug 2009 01:09:33 +0000 (18:09 -0700)
committerJustin Pettit <jpettit@nicira.com>
Tue, 11 Aug 2009 17:04:42 +0000 (10:04 -0700)
commit5ec6690ca447dd25e3d04b60ca66d2e6e1cf37f2
treee570c9d63e8933468d5acdca3c2b25de4ae671d6
parent63920cb960ffe2c494f73982f6ea073a4dc3460d
secchan: Clarify log message about failing open.

In commit e10dfcf357 "rconn: Be pickier about what constitutes a
successful connection", the criteria for determining a successful
OpenFlow connection was tightened.  When rconn connects at a socket
level, it prints messages stating that it "connected" and the switch is
taken out of fail-open mode.  If it is determined that it is not a
"successful" OpenFlow connection, then the connection is closed and
fail-open is re-enabled.  When this occurs, fail-open logs the following
potentially confusing message:

    Could not connect to controller for XXX seconds, failing open

Where XXX is the number of seconds since the last "succesful" connection
rather than simple socket-level connection.  This commit changes the
message to:

    Could not establish OpenFlow channel to controller for XXX
    seconds, failing open

Bug #1163
secchan/fail-open.c