[ovs-git] [ovn-org/ovn] e87012: controller: binding: Ignore changes to OVS interfa...

numansiddique noreply at github.com
Thu Sep 24 05:32:17 UTC 2020


  Branch: refs/heads/master
  Home:   https://github.com/ovn-org/ovn
  Commit: e8701206fd48d56468fa3ec49d7ea2ced3cee29b
      https://github.com/ovn-org/ovn/commit/e8701206fd48d56468fa3ec49d7ea2ced3cee29b
  Author: Numan Siddique <numans at ovn.org>
  Date:   2020-09-24 (Thu, 24 Sep 2020)

  Changed paths:
    M controller/binding.c
    M tests/ovn.at

  Log Message:
  -----------
  controller: binding: Ignore changes to OVS interfaces which doesn't belong to int bridge.

Consider the below commands.

ovn-nbctl ls-add sw0
ovn-nbctl lsp-add sw0 sw0-p1

ovs-vsctl add-br br-temp
ovs-vsctl add port br-temp t1 -- set interface t1 external_ids:iface-id=sw0-p1

When ovn-controller handles the OVS db changes incrementally, it binds the lport
sw0-p1, which is wrong as t1 is not in the integration bridge - br-int.

If a recompute is triggered, ovn-controller releases the lport sw0-p1.

This patch fixes this issue.

Fixes: 354bdba51abf ("ovn-controller: I-P for SB port binding and OVS interface in runtime_data.")
Acked-by: Han Zhou <hzhou at ovn.org>
Signed-off-by: Numan Siddique <numans at ovn.org>




More information about the git mailing list