bridge: Set ofport column in every database transaction.
authorBen Pfaff <blp@nicira.com>
Fri, 31 Jan 2014 00:57:16 +0000 (16:57 -0800)
committerBen Pfaff <blp@nicira.com>
Fri, 31 Jan 2014 00:57:48 +0000 (16:57 -0800)
commit4366871536e3fef825f886cf917953d7a3af76ba
treefc645e8abfa72eb805e2d57a12ea5ac487340958
parentccf59c9b9a738bcb43ba018768098be665ddb5db
bridge: Set ofport column in every database transaction.

Database transactions can occasionally fail due to concurrent changes in
the database.  When that happens, the next transaction should repeat the
changes that ovs-vswitchd tried to make the first time (adjusted for the
changes to the database).

The code to report the OpenFlow port number in use didn't do that.  It set
the ofport field once when it created the port and never set it again, even
if the transaction to set it failed.  This commit fixes the problem.

Bug #23047.
Reported-by: Suganya Ramachandran <suganyar@vmware.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Justin Pettit <jpettit@nicira.com>
AUTHORS
vswitchd/bridge.c