Can we move this patches forward and get a backport to 2.8 / 2.7 branches? Otherwise we get a failure on openstack's:
neutron.tests.tempest.api.test_routers.RoutersTest.test_router_interface_status On Mon, Oct 2, 2017 at 1:33 PM, Miguel Angel Ajo Pelayo <majop...@redhat.com > wrote: > Acked-by: Miguel Angel Ajo <majop...@redhat.com> > > On Fri, Sep 29, 2017 at 8:06 PM, Mark Michelson <mmich...@redhat.com> > wrote: > >> This looks much better. I wrote out the truth table to confirm that the >> functionality is the same. >> >> On Fri, Sep 29, 2017 at 10:06 AM Jakub Sitnicki <j...@redhat.com> wrote: >> >> > No functional change. Make it obvious that we determine the logical >> > port 'up' state by checking for bound chassis, and update the NB DB only >> > when state has not been set yet or current state is different. >> > >> > Signed-off-by: Jakub Sitnicki <j...@redhat.com> >> > >> Acked-by: Mark Michelson <mmich...@redhat.com> >> >> > --- >> > ovn/northd/ovn-northd.c | 7 ++----- >> > 1 file changed, 2 insertions(+), 5 deletions(-) >> > >> > diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c >> > index 2db2380..37651a0 100644 >> > --- a/ovn/northd/ovn-northd.c >> > +++ b/ovn/northd/ovn-northd.c >> > @@ -5980,11 +5980,8 @@ update_logical_port_status(struct northd_context >> > *ctx) >> > continue; >> > } >> > >> > - if (sb->chassis && (!nbsp->up || !*nbsp->up)) { >> > - bool up = true; >> > - nbrec_logical_switch_port_set_up(nbsp, &up, 1); >> > - } else if (!sb->chassis && (!nbsp->up || *nbsp->up)) { >> > - bool up = false; >> > + bool up = sb->chassis ? true : false; >> > + if (!nbsp->up || *nbsp->up != up) { >> > nbrec_logical_switch_port_set_up(nbsp, &up, 1); >> > } >> > } >> > -- >> > 2.9.5 >> > >> > _______________________________________________ >> > dev mailing list >> > d...@openvswitch.org >> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev >> > >> _______________________________________________ >> dev mailing list >> d...@openvswitch.org >> https://mail.openvswitch.org/mailman/listinfo/ovs-dev >> > > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev