From 626de387e617db457d6d431c16327c275b0e8a34 Mon Sep 17 00:00:00 2001 From: Joao Eduardo Luis Date: Thu, 30 May 2013 18:17:28 +0100 Subject: [PATCH] mon: Monitor: backup monmap using all ceph features instead of quorum's When a monitor is freshly created and for some reason its initial sync is aborted, it will end up with an incorrect backup monmap. This monmap is incorrect in the sense that it will not contain the monitor's names as it will expect on the next run. This results from us being using the quorum features to encode the monmap when backing it up, instead of CEPH_FEATURES_ALL. Fixes: #5203 Signed-off-by: Joao Eduardo Luis --- src/mon/Monitor.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mon/Monitor.cc b/src/mon/Monitor.cc index acfeb65da672..b8dc9580b9f4 100644 --- a/src/mon/Monitor.cc +++ b/src/mon/Monitor.cc @@ -1351,7 +1351,7 @@ void Monitor::sync_store_init() return; // this is moot } else { dout(10) << __func__ << " backup current monmap" << dendl; - monmap->encode(latest_monmap, get_quorum_features()); + monmap->encode(latest_monmap, CEPH_FEATURES_ALL); } } -- 2.47.3