[ovs-dev] [PATCH v2 08/21] ovn-controller: Tolerate missing integration bridge.

Ben Pfaff blp at nicira.com
Wed Jul 29 15:54:15 UTC 2015


On Tue, Jul 28, 2015 at 04:40:42PM -0700, Justin Pettit wrote:
> Do you think we should print a warning (at least once) to indicate
> that the integration bridge is missing?  Otherwise, it could be
> confusing to someone why nothing's happening if they typo-ed their
> bridge name.
> 
> Acked-by: Justin Pettit <jpettit at nicira.com>

That's worthwhile.  I added:
    static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 1);
    VLOG_WARN_RL(&rl, "%s: integration bridge does not exist", name);
to get_bridge().

Thanks,

Ben.



More information about the dev mailing list