[ovs-git] Open vSwitch: test: add ofproto OpenFlow1.2 tests (master)

dev at openvswitch.org dev at openvswitch.org
Thu Nov 29 16:11:59 UTC 2012


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, master has been updated
       via  9dcd1c005864d6395fbfe090be1a1427f73169fe (commit)
       via  f02b12c4ff762e4cb38150c5d2c757ee847a9ecb (commit)
       via  be721d87f4978f7625b9948591c7ffb079262fd9 (commit)
       via  37923ac734a8b8bdb4da2fe77a24d77d487dc056 (commit)
       via  4f13da568d49b4910cedf45e6c54c43ea318dd18 (commit)
       via  e71bff1bef0252209d01e618a33d80d59e366829 (commit)
       via  6d2c051a33bccc913259baa14134dd0356530b26 (commit)
       via  a53a8efa2f88d1e1d44617b6c037144c6eadd2c7 (commit)
       via  b060d38039759f71391c27ba251b663404163283 (commit)
      from  e65361d6b6fa0c3fb383b37cd259a24846ded7d5 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 9dcd1c005864d6395fbfe090be1a1427f73169fe
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=9dcd1c005864d6395fbfe090be1a1427f73169fe
Author: Simon Horman <horms at verge.net.au>
		
test: add ofproto OpenFlow1.2 tests
		
These were useful in isolating a number of bugs.

Signed-off-by: Simon Horman <horms at verge.net.au>
Signed-off-by: Ben Pfaff <blp at nicira.com>


commit f02b12c4ff762e4cb38150c5d2c757ee847a9ecb
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=f02b12c4ff762e4cb38150c5d2c757ee847a9ecb
Author: Simon Horman <horms at verge.net.au>
		
ovs-ofctl: By default, do not set flow format for OpenFlow 1.1+
		
Only set the default format for ovs-ofctl monitor if
OpenFlow 1.0 is the prevailing version. IMHO that is
the only case where it makes sense.

Signed-off-by: Simon Horman <horms at verge.net.au>
Signed-off-by: Ben Pfaff <blp at nicira.com>


commit be721d87f4978f7625b9948591c7ffb079262fd9
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=be721d87f4978f7625b9948591c7ffb079262fd9
Author: Simon Horman <horms at verge.net.au>
		
ovs-controller: Allow setting of allowed OpenFlow versions
		
--protocols allows configuration of the versions
that may be used when establishing an OpenFlow connection.

The default is 'OpenFlow10' which is consistent with
the behaviour prior to this patch.

The useful values at this time are:
'OpenFlow10', 'OpenFlow12', 'OpenFlow13',
Values may be combined in a comma delimited list.

e.g.: --protocols 'OpenFlow10,OpenFlow12,OpenFlow13'

Signed-off-by: Simon Horman <horms at verge.net.au>
Signed-off-by: Ben Pfaff <blp at nicira.com>


commit 37923ac734a8b8bdb4da2fe77a24d77d487dc056
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=37923ac734a8b8bdb4da2fe77a24d77d487dc056
Author: Ben Pfaff <blp at nicira.com>
		
ovs-ofctl: Reject impossible protocols configurations at startup.
		
The -O and -F options interact, so that it's possible to select only
flow formats that are not supported on a given OpenFlow version.  It seems
best to report these problems up front rather than failing in a more
mysterious way later.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Simon Horman <horms at verge.net.au>


commit 4f13da568d49b4910cedf45e6c54c43ea318dd18
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=4f13da568d49b4910cedf45e6c54c43ea318dd18
Author: Ben Pfaff <blp at nicira.com>
		
ofp-util: New functions for converting protocols to and from versions.
		
The values of enum ofputil_protocol have meanings that overlap with
OpenFlow version numbers.  These new functions provide ways to figure out
these overlaps.

These functions will be used for the first time in an upcoming commit.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Simon Horman <horms at verge.net.au>


commit e71bff1bef0252209d01e618a33d80d59e366829
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e71bff1bef0252209d01e618a33d80d59e366829
Author: Ben Pfaff <blp at nicira.com>
		
ofp-util: Make "OXM" flow format name include both OpenFlow 1.2 and 1.3.
		
To my mind, it makes sense, when a user requests OXM as the flow format,
to allow any protocol based on the OXM flow format to be used.  Until this
commit, however, "OXM" was specifically OpenFlow 1.2, and "OpenFlow13" was
required to request OXM on OpenFlow 1.3.

This doesn't affect the behavior of any released version of Open vSwitch.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Simon Horman <horms at verge.net.au>


commit 6d2c051a33bccc913259baa14134dd0356530b26
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=6d2c051a33bccc913259baa14134dd0356530b26
Author: Ben Pfaff <blp at nicira.com>
		
ofp-util: Style fixes.
		
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Simon Horman <horms at verge.net.au>


commit a53a8efa2f88d1e1d44617b6c037144c6eadd2c7
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=a53a8efa2f88d1e1d44617b6c037144c6eadd2c7
Author: Simon Horman <horms at verge.net.au>
		
ovs-ofctl: Add option to set allowed OpenFlow versions
		
--protocols allows configuration of the versions
that may be used when establishing an OpenFlow connection.

The default is 'OpenFlow10' which is consistent with
the behaviour prior to this patch.

The useful values at this time are:
'OpenFlow10', 'OpenFlow12', 'OpenFlow13',
Values may be combined in a comma delimited list.

e.g.: --protocols 'OpenFlow10,OpenFlow12,OpenFlow13'

Signed-off-by: Simon Horman <horms at verge.net.au>
Signed-off-by: Ben Pfaff <blp at nicira.com>


commit b060d38039759f71391c27ba251b663404163283
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=b060d38039759f71391c27ba251b663404163283
Author: Ben Pfaff <blp at nicira.com>
		
ofp-version-opt: Fix wrong information in usage message.
		
This is my fault; I introduced this inconsistency when I modified Simon's
correct patch.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Simon Horman <horms at verge.net.au>


-----------------------------------------------------------------------

Summary of changes:
 lib/ofp-util.c                |   63 ++++-
 lib/ofp-util.h                |   13 +-
 lib/ofp-version-opt.c         |    8 +-
 lib/ofp-version-opt.h         |    1 +
 manpages.mk                   |    2 +
 tests/learn.at                |    2 +-
 tests/ofproto-macros.at       |    2 +-
 tests/ofproto.at              |  595 ++++++++++++++++++++++++++++++++++++++---
 tests/ovs-ofctl.at            |   33 ++-
 utilities/ovs-controller.8.in |    1 +
 utilities/ovs-controller.c    |   20 +-
 utilities/ovs-ofctl.8.in      |    9 +
 utilities/ovs-ofctl.c         |  110 +++++---
 13 files changed, 756 insertions(+), 103 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list