openvswitch agent logging verbosity issues

[Robert Kukura]

The openvswitch agent, with no quantum ports existing, logs at a rate of about 3 MB per hour using devstack, which is about 10 times the rate of nova compute, the next most verbose service. The rate could be much higher on a busy system. Both to reduce disk usage and to make it possible to find messages related to real issues, openvswitch deployments will need to disable logging at the debug level. There currently are a number of abnormal conditions logged at the debug level that should be logged as warnings or errors instead. Also, the bridge mappings at startup should be logged at info rather than debug level.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/12686
Committed: http://github.com/openstack/quantum/commit/00dd97c7e9a852f4abd6e8460a0d094c01d3d0ba
Submitter: Jenkins
Branch: master

commit 00dd97c7e9a852f4abd6e8460a0d094c01d3d0ba
Author: Bob Kukura <email address hidden>
Date: Sun Sep 9 11:49:25 2012 -0400

    Add enable_tunneling openvswitch configuration variable

    Not all systems that support Open vSwitch support its GRE tunneling
    feature that is not in the Linux kernel source tree. Therefore, a new
    configuration variable, enable_tunneling, applies to both the server
    and agent. Its default value is False, so it must be set to True to
    enable tunneling for tenant and/or provider networks. If
    enable_tunneling is False, the server will not allow creation of GRE
    networks, and the agent will not initialize the tunnel bridge and will
    log an error if there is an attempt to provision a GRE network. If it
    is True, the agent now logs an error and exits if it fails to
    initialize the patch ports used for the tunnel bridge. Fixes bug
    1045610.

    When there is an attempt to provision a flat or VLAN network and the
    agent has no bridge mapping configured for a the specified physical
    network, the agent will now log an error rather than crash.

    An undefined variable in a logging statement has been corrected.

    Logging levels for openvswitch have been checked and updated where
    necessary, and logging statements now avoid the % string substitution
    syntax. Fixes bug 1045592.

    The corresponding devstack update has already been merged.

    Change-Id: I149db182dd132cc05802dcb20c6b552e293664a5