[ovs-git] [openvswitch/ovs] 767944: ovn-controller: Drop unknown datapath log message.

[GitHub]

  Branch: refs/heads/ovn
  Home:   https://github.com/openvswitch/ovs
  Commit: 767944131928487497579fd48a8a1efa0ca0d676
      https://github.com/openvswitch/ovs/commit/767944131928487497579fd48a8a1efa0ca0d676
  Author: Russell Bryant <rbryant at redhat.com>
  Date:   2015-05-29 (Fri, 29 May 2015)

  Changed paths:
    M ovn/controller/pipeline.c

  Log Message:
  -----------
  ovn-controller: Drop unknown datapath log message.

This patch emits a log message that is encountered a lot under normal
circumstances.  For example, I'm seeing it almost 20,000 times in a
test run of the OpenStack test suite and hundreds of times while doing
basic tasting under ovs-sandbox.

ovn-controller keeps a hash of logical data paths that have ports
associated with them (logical_datapaths).  This message occurs when
looping through all Pipeline entries and a Pipeline's logical datapath
does not exist in this hash.  However, a logical switch with no ports
will have entries in the Pipeline table, but nothing in Bindings (and
not end up in the logical_datapaths hash).  Currently, this message
will be emitted 6 times for every logical switch without any logical
ports every time the ovn-controller main loop comes around and runs
this code.  Instead, the right thing to do to silently ignore the
pipeline entry, because there's nothing useful to be done with it.

Signed-off-by: Russell Bryant <rbryant at redhat.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>