[ovs-git] [ovn-org/ovn] 7b90de: ovn-controller: Fix port group I-P when they conta...
Dumitru Ceara
noreply at github.com
Thu Jul 1 19:30:00 UTC 2021
Branch: refs/heads/branch-21.03
Home: https://github.com/ovn-org/ovn
Commit: 7b90deff5386960a66417ab6231fb3f77731a325
https://github.com/ovn-org/ovn/commit/7b90deff5386960a66417ab6231fb3f77731a325
Author: Dumitru Ceara <dceara at redhat.com>
Date: 2021-07-01 (Thu, 01 Jul 2021)
Changed paths:
M controller/binding.c
M controller/binding.h
M controller/lflow.c
M controller/lflow.h
M controller/ovn-controller.c
M tests/ovn.at
Log Message:
-----------
ovn-controller: Fix port group I-P when they contain non-vif ports.
It's valid that port_groups contain non-vif ports, they can actually
contain any type of logical_switch_port.
Also, there's no need to allocate a new sset containing the local ports'
names every time the I-P engine processes a change. We were already
maintaining a set of "local_lport_ids". These correspond to port
bindings that are relevant locally (including non-vif ports). Extend
it to include the locally relevant lport names too and rename the
structure an its helper functions to related_lport*.
Reported-at: https://github.com/ovn-org/ovn/pull/61#issuecomment-865094163
Reported-by: Antonio Ojea <aojea at redhat.com>
Fixes: 0cfeba6b55e3 ("ovn-controller: Fix port group conjunction flow explosion problem.")
Signed-off-by: Dumitru Ceara <dceara at redhat.com>
Acked-by: Han Zhou <hzhou at ovn.org>
Signed-off-by: Numan Siddique <numans at ovn.org>
More information about the git
mailing list