[ovs-git] [openvswitch/ovs] 07dc75: dpif: Remove a warning

GitHub noreply at github.com
Fri May 20 06:14:00 UTC 2016


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 07dc758e050cc8c8588d4feaf88ec7876f62d307
      https://github.com/openvswitch/ovs/commit/07dc758e050cc8c8588d4feaf88ec7876f62d307
  Author: YAMAMOTO Takashi <yamamoto at ovn.org>
  Date:   2016-05-20 (Fri, 20 May 2016)

  Changed paths:
    M lib/dpif.c

  Log Message:
  -----------
  dpif: Remove a warning

Remove "attempted to unregister a datapath provider that is not registered"
warning.  It's normal for --enabled-dummy=system with userland-only build.
ovn-controller-vtep.at tests use the flag and fail on the extra warning.

Alternatively, we can make the tests ignore this specific warning.
But currently it doesn't make much sense as dp_unregister_provider
is only used for --enabled-dummy.

Signed-off-by: YAMAMOTO Takashi <yamamoto at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 857570c483d4991ddaf91a46a0447e114c671a69
      https://github.com/openvswitch/ovs/commit/857570c483d4991ddaf91a46a0447e114c671a69
  Author: YAMAMOTO Takashi <yamamoto at ovn.org>
  Date:   2016-05-20 (Fri, 20 May 2016)

  Changed paths:
    M tests/ovsdb-server.at

  Log Message:
  -----------
  ovsdb-server.at: Fix races

As ovsdb-server creates pid file before unixctl socket, waiting
for pid file creation is not enough.  Fix the race by retrying
with "version" command before assuming the server is up.

Signed-off-by: YAMAMOTO Takashi <yamamoto at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 7e04dd8583c45d413a0419c7b197599a937f4d49
      https://github.com/openvswitch/ovs/commit/7e04dd8583c45d413a0419c7b197599a937f4d49
  Author: YAMAMOTO Takashi <yamamoto at ovn.org>
  Date:   2016-05-20 (Fri, 20 May 2016)

  Changed paths:
    M tests/ovn-controller-vtep.at

  Log Message:
  -----------
  ovn-controller-vtep.at: Pre-sort output before feeding to "sort -d"

NetBSD's "sort -d" preserves the order of lines which doesn't have
alphanumeric and blanks.  eg. empty lines and [].
It means it sometimes preserve unstable order of the list output.

Also, simply remove -d option where the expected output doesn't
include [].

Signed-off-by: YAMAMOTO Takashi <yamamoto at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: af51e6f0156eb82e2e2a7f0ba369c0e4cbfb6853
      https://github.com/openvswitch/ovs/commit/af51e6f0156eb82e2e2a7f0ba369c0e4cbfb6853
  Author: YAMAMOTO Takashi <yamamoto at ovn.org>
  Date:   2016-05-20 (Fri, 20 May 2016)

  Changed paths:
    M utilities/ovs-check-dead-ifs.in
    M utilities/ovs-dpctl-top.in
    M utilities/ovs-l3ping.in
    M utilities/ovs-parse-backtrace.in
    M utilities/ovs-pcap.in
    M utilities/ovs-tcpundump.in
    M utilities/ovs-test.in
    M utilities/ovs-vlan-test.in

  Log Message:
  -----------
  utilities: Tweak python shebangs to use env

"python" command provided by pkg_alternatives is a shell script.
At least on NetBSD-7, execve can't execute scripts whose interpreter
is another shell script.  (While some "rich" shells like zsh seem
to have handle the case by itself, NetBSD's /bin/sh doesn't.)
Workaround the issue by using env command for shebangs for
these scripts.

Noticed with the recent tunnel-push-pop.at tests using ovs-pcap command.

Signed-off-by: YAMAMOTO Takashi <yamamoto at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/6195e2e7b828...af51e6f0156e


More information about the git mailing list