From: Samuel Just <sjust@redhat.com>
Date: Mon, 14 Mar 2016 22:37:25 +0000 (-0700)
Subject: features: deprecate CEPH_FEATURE_MON_SCRUB
X-Git-Tag: v10.1.0~17^2~7
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=fd3b67e7e9c4c9b783132fbf3ebae09fe4107799;p=ceph.git

features: deprecate CEPH_FEATURE_MON_SCRUB

Signed-off-by: Samuel Just <sjust@redhat.com>
---

diff --git a/src/include/ceph_features.h b/src/include/ceph_features.h
index a22a49b5acce..8cb74635a2a2 100755
--- a/src/include/ceph_features.h
+++ b/src/include/ceph_features.h
@@ -43,7 +43,7 @@
 #define CEPH_FEATURE_OSDHASHPSPOOL  (1ULL<<30)
 #define CEPH_FEATURE_MON_SINGLE_PAXOS (1ULL<<31)
 #define DEPRECATED_CEPH_FEATURE_OSD_SNAPMAPPER (1ULL<<32) // DEPRECATED: JEWEL
-#define CEPH_FEATURE_MON_SCRUB      (1ULL<<33)
+#define DEPRECATED_CEPH_FEATURE_MON_SCRUB      (1ULL<<33)
 #define CEPH_FEATURE_OSD_PACKED_RECOVERY (1ULL<<34)
 #define CEPH_FEATURE_OSD_CACHEPOOL (1ULL<<35)
 #define CEPH_FEATURE_CRUSH_V2      (1ULL<<36)  /* new indep; SET_* steps */
@@ -147,7 +147,7 @@ static inline unsigned long long ceph_sanitize_features(unsigned long long f) {
          CEPH_FEATURE_NEW_OSDOPREPLY_ENCODING | \
 	 CEPH_FEATURE_MON_SINGLE_PAXOS |    \
 	 DEPRECATED_CEPH_FEATURE_OSD_SNAPMAPPER |	    \
-	 CEPH_FEATURE_MON_SCRUB	|	    \
+	 DEPRECATED_CEPH_FEATURE_MON_SCRUB	|	    \
 	 CEPH_FEATURE_OSD_PACKED_RECOVERY | \
 	 CEPH_FEATURE_OSD_CACHEPOOL |	    \
 	 CEPH_FEATURE_CRUSH_V2 |	    \
diff --git a/src/mon/Monitor.cc b/src/mon/Monitor.cc
index 5911cdbf3256..cd315c9b6237 100644
--- a/src/mon/Monitor.cc
+++ b/src/mon/Monitor.cc
@@ -4480,11 +4480,6 @@ int Monitor::scrub_start()
   dout(10) << __func__ << dendl;
   assert(is_leader());
 
-  if ((get_quorum_features() & CEPH_FEATURE_MON_SCRUB) == 0) {
-    clog->warn() << "scrub not supported by entire quorum\n";
-    return -EOPNOTSUPP;
-  }
-
   if (!scrub_result.empty()) {
     clog->info() << "scrub already in progress\n";
     return -EBUSY;