]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
mon: fix kraken requirement to use FEATUREMASK, HAVE_FEATURE macro
authorSage Weil <sage@redhat.com>
Mon, 6 Feb 2017 16:58:35 +0000 (11:58 -0500)
committerSage Weil <sage@redhat.com>
Tue, 7 Feb 2017 15:36:32 +0000 (10:36 -0500)
The KRAKEN_SERVER bit was reused.

Signed-off-by: Sage Weil <sage@redhat.com>
src/mon/Monitor.cc

index 117f5cae39e7230a80fdf8b59753b6f37ee5a046..b951f0675df154e9814f7b3fae18f3670bd38ca0 100644 (file)
@@ -2188,7 +2188,7 @@ void Monitor::apply_monmap_to_compatset_features()
     assert(ceph::features::mon::get_persistent().contains_all(
            ceph::features::mon::FEATURE_KRAKEN));
     // this feature should only ever be set if the quorum supports it.
-    assert(quorum_con_features & CEPH_FEATURE_SERVER_KRAKEN);
+    assert(HAVE_FEATURE(quorum_con_features, SERVER_KRAKEN));
     new_features.incompat.insert(CEPH_MON_FEATURE_INCOMPAT_KRAKEN);
   }
   dout(5) << __func__ << dendl;
@@ -2211,7 +2211,7 @@ void Monitor::apply_compatset_features_to_quorum_requirements()
     required_features |= CEPH_FEATURE_ERASURE_CODE_PLUGINS_V3;
   }
   if (features.incompat.contains(CEPH_MON_FEATURE_INCOMPAT_KRAKEN)) {
-    required_features |= CEPH_FEATURE_SERVER_KRAKEN;
+    required_features |= CEPH_FEATUREMASK_SERVER_KRAKEN;
   }
   dout(10) << __func__ << " required_features " << required_features << dendl;
 }