On 27 March 2017 at 18:34, Mickey Spiegel <[email protected]> wrote:
> In two of the Gratuitous ARP tests, "options:router-port" > is not set correctly. This does not currently affect > validity of the tests since the next line resets > "options:router-port" to the correct value. > > Reported-by: Guruchuran Shetty <[email protected]> > Signed-off-by: Mickey Spiegel <[email protected]> > --- > tests/ovn.at | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tests/ovn.at b/tests/ovn.at > index bbbec90..8b7ba12 100644 > --- a/tests/ovn.at > +++ b/tests/ovn.at > @@ -5266,7 +5266,7 @@ ovn-nbctl create Logical_Router name=lr0 > options:chassis=hv1 > # Add router port to gateway router > ovn-nbctl lrp-add lr0 lrp0 f0:00:00:00:00:01 192.168.0.1/24 > ovn-nbctl lsp-add ls0 lrp0-rp -- set Logical_Switch_Port lrp0-rp \ > - type=router options:router-port=lrp0-rp addresses='"f0:00:00:00:00:01" > ' > + type=router options:router-port=lrp0 addresses='"f0:00:00:00:00:01"' > # Add nat-address option > ovn-nbctl lsp-set-options lrp0-rp router-port=lrp0 > nat-addresses="f0:00:00:00:00:01 192.168.0.2" > Looks like lsp-set-options expects all the options for that port. Otherwise, it deletes the ones not specified. This is very counter-intuitive. But, I wonder whether changing lsp-set-options now would constitute a API change. Nevertheless, it is independent of this patch. So I will apply this. > > @@ -5314,7 +5314,7 @@ ovn-nbctl create Logical_Router name=lr0 > options:chassis=hv1 > # Add router port to gateway router > ovn-nbctl lrp-add lr0 lrp0 f0:00:00:00:00:01 192.168.0.1/24 > ovn-nbctl lsp-add ls0 lrp0-rp -- set Logical_Switch_Port lrp0-rp \ > - type=router options:router-port=lrp0-rp addresses='"f0:00:00:00:00:01" > ' > + type=router options:router-port=lrp0 addresses='"f0:00:00:00:00:01"' > # Add nat-address option > ovn-nbctl lsp-set-options lrp0-rp router-port=lrp0 nat-addresses="router" > # Add NAT rules > -- > 1.9.1 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
