diff mbox series

[ovs-dev,v2,3/3] Fix flows not removed in ha migration

Message ID 20230920154557.4010537-3-xsimonar@redhat.com
State Superseded
Headers show
Series [ovs-dev,v2,1/3] binding: slight refactor if no local binding in consider_iface_release | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/github-robot-_ovn-kubernetes fail github build: failed

Commit Message

Xavier Simonart Sept. 20, 2023, 3:45 p.m. UTC 
Signed-off-by: Xavier Simonart <xsimonar@redhat.com>
---
 controller/binding.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Mark Michelson Sept. 25, 2023, 7:34 p.m. UTC | #1 
Thanks, Xavier.

Acked-by: Mark Michelson <mmichels@redhat.com>

On 9/20/23 11:45, Xavier Simonart wrote:
> Signed-off-by: Xavier Simonart <xsimonar@redhat.com>
> ---
>   controller/binding.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/controller/binding.c b/controller/binding.c
> index 92ca3ebbe..bac38fc5f 100644
> --- a/controller/binding.c
> +++ b/controller/binding.c
> @@ -1853,7 +1853,9 @@ consider_nonvif_lport_(const struct sbrec_port_binding *pb,
>       }
>   
>       if (pb->chassis == b_ctx_in->chassis_rec ||
> -            is_additional_chassis(pb, b_ctx_in->chassis_rec)) {
> +            is_additional_chassis(pb, b_ctx_in->chassis_rec)
> +            || if_status_is_port_claimed(b_ctx_out->if_mgr,
> +                                         pb->logical_port)) {
>           return release_lport(pb, b_ctx_in->chassis_rec,
>                                !b_ctx_in->ovnsb_idl_txn,
>                                b_ctx_out->tracked_dp_bindings,
diff mbox series

Patch

diff --git a/controller/binding.c b/controller/binding.c
index 92ca3ebbe..bac38fc5f 100644
--- a/controller/binding.c
+++ b/controller/binding.c
@@ -1853,7 +1853,9 @@  consider_nonvif_lport_(const struct sbrec_port_binding *pb,
     }
 
     if (pb->chassis == b_ctx_in->chassis_rec ||
-            is_additional_chassis(pb, b_ctx_in->chassis_rec)) {
+            is_additional_chassis(pb, b_ctx_in->chassis_rec)
+            || if_status_is_port_claimed(b_ctx_out->if_mgr,
+                                         pb->logical_port)) {
         return release_lport(pb, b_ctx_in->chassis_rec,
                              !b_ctx_in->ovnsb_idl_txn,
                              b_ctx_out->tracked_dp_bindings,