[ovs-git] Open vSwitch: ovs-vsctl: Add tests for database commands. (next)

Fri Jan 29 00:19:20 UTC 2010

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, next has been updated
       via  5c414a2ed8c21762292776ecee294a11e0b5468b (commit)
       via  870aeb4aa05cf68b90e3e138dfe45b787e29ec55 (commit)
       via  c29a8ba8868d55026e9f4b0f88582fd85da48621 (commit)
       via  492eb7a8ec68cf48ab1dc5f88c4973d91740c9da (commit)
       via  3ef917b5699874c46131c1d8731fafc5246c0dd1 (commit)
       via  3da1c516ba4556970040747c9d4778b87986da52 (commit)
       via  f8ff4bc4f32c8d20c899f6a9b1959a46e9f62526 (commit)
       via  baa8f41b24179704944f278b5ae00231b1ca8246 (commit)
       via  63e60b866ffee6895e1772da2c48591ab2767aa7 (commit)
       via  69490970b3555fd45f844b4f6304bc2dc6a01b0b (commit)
       via  e83fd213300e19ad1da4e1ddbc049d4b858d34e8 (commit)
       via  9397a13f60a477a4244a305fd88845fd5dc36b22 (commit)
       via  cb4562170bdb37e1bb33f291a85fe275380af785 (commit)
       via  54a687fd7b3ae2004638e289ae20a912cf4c03dd (commit)
      from  0e17074a1a43edadde14266715c614a8313c6cbb (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 5c414a2ed8c21762292776ecee294a11e0b5468b
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=5c414a2ed8c21762292776ecee294a11e0b5468b
Author: Ben Pfaff <blp at nicira.com>

ovs-vsctl: Add tests for database commands.

Bug #2396.

commit 870aeb4aa05cf68b90e3e138dfe45b787e29ec55
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=870aeb4aa05cf68b90e3e138dfe45b787e29ec55
Author: Ben Pfaff <blp at nicira.com>

ovs-vsctl: Add --if-exists option to "get" command, for map columns.

commit c29a8ba8868d55026e9f4b0f88582fd85da48621
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=c29a8ba8868d55026e9f4b0f88582fd85da48621
Author: Ben Pfaff <blp at nicira.com>

ovs-vsctl: Improve error reporting.

commit 492eb7a8ec68cf48ab1dc5f88c4973d91740c9da
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=492eb7a8ec68cf48ab1dc5f88c4973d91740c9da
Author: Ben Pfaff <blp at nicira.com>

ovs-vsctl: Alphabetize lists of database column names.

This makes the "list" command print the database columns in alphabetical
order, which seems more sensible than the semi-random order we had before.

commit 3ef917b5699874c46131c1d8731fafc5246c0dd1
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=3ef917b5699874c46131c1d8731fafc5246c0dd1
Author: Ben Pfaff <blp at nicira.com>

ovs-vsctl: Make "create" command print new row's UUID.

Also, document limitations of other commands regarding UUIDs of new rows.

commit 3da1c516ba4556970040747c9d4778b87986da52
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=3da1c516ba4556970040747c9d4778b87986da52
Author: Ben Pfaff <blp at nicira.com>

ovs-vsctl: Add postprocess step.

Nothing uses this new feature yet, so this commit by itself has no visible
effect.

This is a step toward printing the UUIDs of newly inserted rows.

commit f8ff4bc4f32c8d20c899f6a9b1959a46e9f62526
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=f8ff4bc4f32c8d20c899f6a9b1959a46e9f62526
Author: Ben Pfaff <blp at nicira.com>

ovs-vsctl: Refactor in preparation for adding "postprocess" step.

Until now, all the ovs-vsctl commands have had no need to look at the
results of the database transaction.  However, to print the UUID of newly
inserted database records, they do need to do this.

This commit refactors ovs-vsctl in preparation for adding this ability.

commit baa8f41b24179704944f278b5ae00231b1ca8246
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=baa8f41b24179704944f278b5ae00231b1ca8246
Author: Ben Pfaff <blp at nicira.com>

New functions hmap_moved(), shash_moved().

To be used in ovs-vsctl in an upcoming commit.

commit 63e60b866ffee6895e1772da2c48591ab2767aa7
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=63e60b866ffee6895e1772da2c48591ab2767aa7
Author: Ben Pfaff <blp at nicira.com>

hmap: Rename hmap_moved() to hmap_node_moved().

This prepares for adding a new function that deals with a "struct hmap"
moving, as opposed to a "struct hmap_node".

Since there was only a single call to this in the whole tree, and its
caller didn't have any callers of its own at all, also move this function
from hmap.h to hmap.c.

commit 69490970b3555fd45f844b4f6304bc2dc6a01b0b
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=69490970b3555fd45f844b4f6304bc2dc6a01b0b
Author: Ben Pfaff <blp at nicira.com>

ovsdb-idl: Add interface to find out the permanent IDL of an inserted row.

The ovs-vsctl "create" command, and perhaps other commands, should print
the UUID of the newly created database row, but until now the IDL has not
provided a way to find that out.  This commit adds the ability.

commit e83fd213300e19ad1da4e1ddbc049d4b858d34e8
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e83fd213300e19ad1da4e1ddbc049d4b858d34e8
Author: Ben Pfaff <blp at nicira.com>

dynamic-string: New function ds_swap().

commit 9397a13f60a477a4244a305fd88845fd5dc36b22
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=9397a13f60a477a4244a305fd88845fd5dc36b22
Author: Ben Pfaff <blp at nicira.com>

ovsdb-data: Allow spaces around '=' in key-value pairs.

This allows a = b, a= b, a =b, etc. whereas before only a=b was accepted.

commit cb4562170bdb37e1bb33f291a85fe275380af785
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=cb4562170bdb37e1bb33f291a85fe275380af785
Author: Ben Pfaff <blp at nicira.com>

ovsdb-data: Allow arbitrary white space as string data delimiters.

commit 54a687fd7b3ae2004638e289ae20a912cf4c03dd
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=54a687fd7b3ae2004638e289ae20a912cf4c03dd
Author: Ben Pfaff <blp at nicira.com>

ovsdb-data: Make string parsing of negative 0 match JSON parsing.

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

Summary of changes:
 lib/classifier.c         |    5 +-
 lib/dynamic-string.c     |   11 +-
 lib/dynamic-string.h     |    3 +-
 lib/hmap.c               |   32 ++-
 lib/hmap.h               |   21 +--
 lib/ovsdb-data.c         |   12 +-
 lib/ovsdb-idl.c          |  120 +++++++++-
 lib/ovsdb-idl.h          |    5 +-
 lib/shash.c              |    6 +
 lib/shash.h              |    1 +
 tests/ovs-vsctl.at       |  180 ++++++++++++++
 tests/ovsdb-data.at      |  270 +++++++++++++++++++---
 tests/test-ovsdb.c       |   73 ++++++-
 utilities/ovs-vsctl.8.in |   12 +-
 utilities/ovs-vsctl.c    |  597 ++++++++++++++++++++++++++--------------------
 15 files changed, 1010 insertions(+), 338 deletions(-)

hooks/post-receive
-- 
Open vSwitch