On 9/1/22 22:17, Dumitru Ceara wrote:
> This implementation was removed by accident between V4 [0] and V5 [1]
> of a7c7d4519e50 ("controller: avoid recomputes triggered by SBDB
> Port_Binding updates.").
> 
> [0] 
> https://patchwork.ozlabs.org/project/ovn/patch/[email protected]/
> [1] 
> https://patchwork.ozlabs.org/project/ovn/patch/[email protected]/
> 
> Fixes: a7c7d4519e50 ("controller: avoid recomputes triggered by SBDB 
> Port_Binding updates.")
> Signed-off-by: Dumitru Ceara <[email protected]>
> ---

Hi Xavier,

This is essentially just your code, I didn't change anything.  If you're
ok with adding your "Signed-off-by" then I'm fine with changing mine
into an Acked-by.

Regards,
Dumitru

>  tests/ovn.at | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/tests/ovn.at b/tests/ovn.at
> index c2bcb721a..db1d29737 100644
> --- a/tests/ovn.at
> +++ b/tests/ovn.at
> @@ -105,6 +105,18 @@ m4_divert_text([PREPARE_TESTS],
>           test 1 -le $(as $hv1 ovs-ofctl dump-flows br-int | grep -c 
> "output:$ofport")
>       ])
>     }
> +
> +   ovn_wait_remote_input_flows () {
> +     hv1=$1
> +     hv2=$2
> +     echo "$3: waiting for flows for remote input on $hv1"
> +     # Wait for a flow outputing  to remote input
> +     OVS_WAIT_UNTIL([
> +         ofport=$(as $hv1 ovs-vsctl --bare --columns ofport find Interface 
> name=ovn-${hv2}-0)
> +         echo "tunnel port=$ofport"
> +         test 1 -le $(as $hv1 ovs-ofctl dump-flows br-int | grep -c 
> "in_port=$ofport")
> +     ])
> +   }
>  ])
>  
>  m4_define([OVN_CHECK_PACKETS],

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to