diff options
| author | Joao Eduardo Luis <joao.luis@inktank.com> | 2013-07-16 16:45:39 +0100 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2013-07-16 10:31:46 -0700 |
| commit | 036e6739a4e873863bae3d7d00f310c015dfcdb3 (patch) | |
| tree | b25119d5013e4a398440021293519fe4c622ae84 | |
| parent | 38691e7f957a819325c50da0503c086cd96172d1 (diff) | |
| download | ceph-036e6739a4e873863bae3d7d00f310c015dfcdb3.tar.gz | |
mon: Monitor: do not reopen MonitorDBStore during conversion
We already open the store on ceph_mon.cc, before we start the conversion.
Given we are unable to reproduce this every time a conversion is triggered,
we are led to believe that this causes a race in leveldb that will lead
to 'store.db/LOCK' being locked upon the open this patch removes.
Regardless, reopening the db here is pointless as we already did it when
we reach Monitor::StoreConverter::convert().
Fixes: #5640
Backport: cuttlefish
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>
| -rw-r--r-- | src/mon/Monitor.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/mon/Monitor.cc b/src/mon/Monitor.cc index 1282cb359ca..5cba9d4f80a 100644 --- a/src/mon/Monitor.cc +++ b/src/mon/Monitor.cc @@ -3851,7 +3851,6 @@ out: int Monitor::StoreConverter::convert() { _init(); - assert(!db->create_and_open(std::cerr)); assert(!store->mount()); if (db->exists("mon_convert", "on_going")) { dout(0) << __func__ << " found a mon store in mid-convertion; abort!" |