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
