[ovs-git] [openvswitch/ovs] d3b41d: ovs-save: Use --bundle to restore flows (on OpenFl...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: d3b41dd957fc74001e43e59367890461c6f4f159
      https://github.com/openvswitch/ovs/commit/d3b41dd957fc74001e43e59367890461c6f4f159
  Author: Timothy Redaelli <tredaelli at redhat.com>
  Date:   2017-10-27 (Fri, 27 Oct 2017)

  Changed paths:
    M utilities/ovs-save

  Log Message:
  -----------
  ovs-save: Use --bundle to restore flows (on OpenFlow 1.4+)

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 <tredaelli at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Flavio Leitner <fbl at sysclose.org>


  Commit: 463a057879630c7b99fd1cb6393d2a92d28f8412
      https://github.com/openvswitch/ovs/commit/463a057879630c7b99fd1cb6393d2a92d28f8412
  Author: Timothy Redaelli <tredaelli at redhat.com>
  Date:   2017-10-27 (Fri, 27 Oct 2017)

  Changed paths:
    M utilities/ovs-save

  Log Message:
  -----------
  ovs-save: Use a file to restore flows instead of heredoc

This patch makes ovs-save to use a file to restore flows instead of using
shell script here-document.
This is needed since eval + here-documents are much slower than reading a file
with the rules directly.

Signed-off-by: Timothy Redaelli <tredaelli at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Flavio Leitner <fbl at sysclose.org>


  Commit: 221df99c5a82e58f8bd19356c0e6fce0ca7eab39
      https://github.com/openvswitch/ovs/commit/221df99c5a82e58f8bd19356c0e6fce0ca7eab39
  Author: Timothy Redaelli <tredaelli at redhat.com>
  Date:   2017-10-27 (Fri, 27 Oct 2017)

  Changed paths:
    M utilities/ovs-save

  Log Message:
  -----------
  ovs-save: Replace "echo -n" with "printf"

This is neeed since "echo -n" is not POSIX and may not work with some shells.

Signed-off-by: Timothy Redaelli <tredaelli at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Flavio Leitner <fbl at sysclose.org>


Compare: https://github.com/openvswitch/ovs/compare/a694ead1e278...221df99c5a82