From 8b7ca687ded06fe0b67d98e81fa1dabbed440853 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 (cherry picked from commit 626de387e617db457d6d431c16327c275b0e8a34) --- 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 3c40ad3808526..336f508c56a05 100644 --- a/src/mon/Monitor.cc +++ b/src/mon/Monitor.cc @@ -1350,7 +1350,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.39.5