Re: [ovs-dev] [PATCH] ovs-lib: dont't purge corrupted DB

2017-11-03 Thread Ben Pfaff
On Wed, Sep 27, 2017 at 09:32:52PM +, Mark Michelson wrote: > LGTM > > On Wed, Sep 27, 2017 at 8:46 AM Matteo Croce wrote: > > > In ovs-lib there is a function named upgrade_db which tries to convert a > > database after OVS {up,down}grades. This function uses ovsdb-tool

Re: [ovs-dev] [PATCH] ovs-lib: dont't purge corrupted DB

2017-09-27 Thread Mark Michelson
LGTM On Wed, Sep 27, 2017 at 8:46 AM Matteo Croce wrote: > In ovs-lib there is a function named upgrade_db which tries to convert a > database after OVS {up,down}grades. This function uses ovsdb-tool to > check if the DB needs to be upgraded. If the upgrade fails, > it purges

[ovs-dev] [PATCH] ovs-lib: dont't purge corrupted DB

2017-09-27 Thread Matteo Croce
In ovs-lib there is a function named upgrade_db which tries to convert a database after OVS {up,down}grades. This function uses ovsdb-tool to check if the DB needs to be upgraded. If the upgrade fails, it purges the DB and create an empty one. ovsdb-tool returns "yes" or "no" to indicate if the DB