[ovs-dev] [PATCH 4/4] ovn-ctl: Ability to upgrade databases.

Ben Pfaff blp at nicira.com
Fri Oct 2 13:21:38 UTC 2015


On Thu, Oct 01, 2015 at 03:09:56PM -0700, Gurucharan Shetty wrote:
> Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>

In upgrade_ovn_dbs, I'd consider upgrading both databases even if they
don't appear in the list of databases.  It seems more robust to me,
especially given this comment in start_northd:

 # We expect ovn-northd to be co-located with ovsdb-server handling both the
 # OVN_Northbound and OVN_Southbound dbs.

I'm not sure about error handling.  If an upgrade to one database fails,
it might be better to try to upgrade the other database instead of
exiting immediately.

Acked-by: Ben Pfaff <blp at nicira.com>



More information about the dev mailing list