Re: [ovs-dev] [PATCH 4/7] ovn: Fix race in "ovn -- ipam" test.

[Guru Shetty]

On 5 October 2016 at 18:26, Ben Pfaff  wrote:

> After setting the subnet, ovn-northd needs to process the changes before
> setting the dynamic addresses.
>
> Signed-off-by: Ben Pfaff 
>
Acked-by: Gurucharan Shetty 

> ---
>  tests/ovn.at | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/ovn.at b/tests/ovn.at
> index 9baa1df..c4a600a 100644
> --- a/tests/ovn.at
> +++ b/tests/ovn.at
> @@ -4552,7 +4552,7 @@ ovn_start
>  # subnet which should result in an address being allocated for the port.
>  ovn-nbctl ls-add sw0
>  ovn-nbctl lsp-add sw0 p0 -- lsp-set-addresses p0 dynamic
> -ovn-nbctl add Logical-Switch sw0 other_config subnet=192.168.1.0/24
> +ovn-nbctl --wait=sb add Logical-Switch sw0 other_config subnet=
> 192.168.1.0/24
>  AT_CHECK([ovn-nbctl get Logical-Switch-Port p0 dynamic_addresses], [0],
>  ["0a:00:00:00:00:01 192.168.1.2"
>  ])
> --
> 2.1.3
>
> ___
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH 4/7] ovn: Fix race in "ovn -- ipam" test.

[Ben Pfaff]

After setting the subnet, ovn-northd needs to process the changes before
setting the dynamic addresses.

Signed-off-by: Ben Pfaff 
---
 tests/ovn.at | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/ovn.at b/tests/ovn.at
index 9baa1df..c4a600a 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -4552,7 +4552,7 @@ ovn_start
 # subnet which should result in an address being allocated for the port.
 ovn-nbctl ls-add sw0
 ovn-nbctl lsp-add sw0 p0 -- lsp-set-addresses p0 dynamic
-ovn-nbctl add Logical-Switch sw0 other_config subnet=192.168.1.0/24
+ovn-nbctl --wait=sb add Logical-Switch sw0 other_config subnet=192.168.1.0/24
 AT_CHECK([ovn-nbctl get Logical-Switch-Port p0 dynamic_addresses], [0],
 ["0a:00:00:00:00:01 192.168.1.2"
 ])
-- 
2.1.3

___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev