[ovs-git] [openvswitch/ovs] 89fa2a: ovn-performance.at: Missing steps for connecting L...

Han Zhou noreply at github.com
Thu Jul 11 21:06:57 UTC 2019


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 89fa2a74c9256406f42a40f51f03c6a912dff043
      https://github.com/openvswitch/ovs/commit/89fa2a74c9256406f42a40f51f03c6a912dff043
  Author: Han Zhou <hzhou8 at ebay.com>
  Date:   2019-07-10 (Wed, 10 Jul 2019)

  Changed paths:
    M tests/ovn-performance.at

  Log Message:
  -----------
  ovn-performance.at: Missing steps for connecting LS to LR.

The test creates 2 logical switches and connect them with a logical router.
However, it didn't set the option "router-port", so the 2 LS datapaths
were not connected. This results in missing test coverage for port-binding
incremental processing: assume I-P has a bug and port-binding change always
trigger recompute, since each HV monitors only its own datapath (i.e.
HV1 -> ls1, HV2 -> ls2) then it never got notification of the other
port-binding change, thus recompute is never triggered when port-binding
is updated on the other datapath. With this fix, each HV's local datapaths
will include both ls1 and ls2, so port-binding change notification will
be received properly and unexpected recompute would be captured.

Signed-off-by: Han Zhou <hzhou8 at ebay.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


  Commit: 809ac71dccaba4ecfa3b623d2aacba1760eb17a2
      https://github.com/openvswitch/ovs/commit/809ac71dccaba4ecfa3b623d2aacba1760eb17a2
  Author: Han Zhou <hzhou8 at ebay.com>
  Date:   2019-07-10 (Wed, 10 Jul 2019)

  Changed paths:
    M tests/ovn-performance.at

  Log Message:
  -----------
  ovn-performance.at: Fix syntax error in ACL.

This doesn't impact the effectiveness of the test but just fix an
obvious error in ACL syntax which was noticed when looking at test
logs.

Signed-off-by: Han Zhou <hzhou8 at ebay.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/2a9679e3b2c6...809ac71dccab


More information about the git mailing list