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 |
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 |
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 --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,
Signed-off-by: Xavier Simonart <xsimonar@redhat.com> --- controller/binding.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)