On Sun, 9 Sep 2018 at 05:20, Timothy Redaelli <[email protected]> wrote:

> Currently the default flow (actions=NORMAL) is present in the flow table
> after
> the flow table is restored also when the default flow is removed.
>
> This commit changes the behaviour of the "ovs-save save-flows" command to
> use
> "replace-flows" instead of "add-flows" to restore the flows. This is
> needed in
> order to always have the new flow table as it was before restoring it.
>
> Reported-by: Flavio Leitner <[email protected]>
> Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1626096
> Signed-off-by: Timothy Redaelli <[email protected]>
>

I applied this to master, 2.10 and 2.9


> ---
>  utilities/ovs-save | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/utilities/ovs-save b/utilities/ovs-save
> index ea8fb6a45..2294583d6 100755
> --- a/utilities/ovs-save
> +++ b/utilities/ovs-save
> @@ -121,7 +121,7 @@ save_flows () {
>               cnt++;printf "{class="$1",type="$2",len="$3"}->"$4}'
>          echo "'"
>
> -        printf "%s" "ovs-ofctl -O $ofp_version add-flows ${bridge} " \
> +        printf "%s" "ovs-ofctl -O $ofp_version replace-flows ${bridge} " \
>              "\"$workdir/$bridge.flows.dump\""
>
>          # If possible, use OpenFlow 1.4 atomic bundle transaction to add
> flows
> --
> 2.17.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

Reply via email to