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