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:16 +0000 (16:57 -0800)
commit59804c80a9821e3aca208012c1a496f3c26bfe9a
tree15080525d2d1eeb0c42cf07dedeb66b85c95438c
parent0a8763fcb31bfca0d8d854c235c531005088fcb9
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