On Mon, Sep 25, 2017 at 04:44:04PM +0200, Timothy Redaelli wrote:
> If possible, use OpenFlow 1.4 atomic bundle transaction to restore flows.
> The patch uses also the highest enabled OpenFlow version to do the queries.
>
> With the actual implementation, if you have the default OpenFlow version
> disabled then ovs-save fails. This patch also fixes that problem.
>
> Signed-off-by: Timothy Redaelli <[email protected]>
Thanks for working on this.
The get_highest_ofp_version might have surprising semantics. The
ovs-vsctl command will print protocols in alphabetical order, and
get_highest_ofp_version assumes that the alphabetically last OpenFlow
protocol name is the "best". That's currently true, but I am not sure
that it will be true forever. What if, instead of taking the last, we
took all of them? I believe that ovs-ofctl, given multiple protocols on
-O, will select the "best" or latest itself
Something like this would do the trick, I guess:
ovs-vsctl get bridge br0 protocols | tr -d '[]" '
This is pretty petty, though. It makes no difference for now. I
applied this to master.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
