[ovs-git] [openvswitch/ovs] cea4a6: tests: Make ofproto-dpif tests more resilient.

GitHub noreply at github.com
Thu Dec 4 22:32:51 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: cea4a6d714d4079819162cccf9d0cc220399a910
      https://github.com/openvswitch/ovs/commit/cea4a6d714d4079819162cccf9d0cc220399a910
  Author: Joe Stringer <joestringer at nicira.com>
  Date:   2014-12-03 (Wed, 03 Dec 2014)

  Changed paths:
    M tests/ofproto-dpif.at

  Log Message:
  -----------
  tests: Make ofproto-dpif tests more resilient.

We have previously used a combination of time/warp and revalidator/wait
to try to synchronize the datapath flow statistics and status up to the
ofproto layer, so that tests may confirm their expected behaviours using
OpenFlow or other protocols operating at that layer of vswitchd (eg LACP).

This patch switches these tests over to using revalidator/purge, which
tells the ofproto-dpif to flush the datapath, causing these statistics
to be updated. In local testing this appears to improve the success rate
of these tests considerably.

Signed-off-by: Joe Stringer <joestringer at nicira.com>


  Commit: 5b7278a122c9c6aac0da3c34a41d3642f87f3ff0
      https://github.com/openvswitch/ovs/commit/5b7278a122c9c6aac0da3c34a41d3642f87f3ff0
  Author: Joe Stringer <joestringer at nicira.com>
  Date:   2014-12-04 (Thu, 04 Dec 2014)

  Changed paths:
    M tests/ofproto-dpif.at

  Log Message:
  -----------
  tests: Fix race in async config test.

Occasionally, the testsuite would send the OFPT_SET_ASYNC before
ovs-vswitchd got a chance to send its OFPT_ROLE_REPLY message, causing a
reordering of the testsuite output and a false positive. Give the test
script something extra to do so this is less likely to happen.

Signed-off-by: Joe Stringer <joestringer at nicira.com>


  Commit: 819e3cdec67332b3cfad771f6856924a1f2e7130
      https://github.com/openvswitch/ovs/commit/819e3cdec67332b3cfad771f6856924a1f2e7130
  Author: Joe Stringer <joestringer at nicira.com>
  Date:   2014-12-04 (Thu, 04 Dec 2014)

  Changed paths:
    M build-aux/cccl

  Log Message:
  -----------
  cccl: Respect silent flags.

Automake sets $V to tell the compiler whether to print verbose messages
as it compiles or not. Add support for this variable in cccl, allowing
more quiet build output on windows if the build is configured with
--silent or the developer runs make V=0.

Signed-off-by: Joe Stringer <joestringer at nicira.com>


Compare: https://github.com/openvswitch/ovs/compare/64bb477f0568...819e3cdec673


More information about the git mailing list