crush/CrushTester: drop --ruleset

Signed-off-by: Sage Weil <sage@newdream.net>

diff --git a/src/crush/CrushTester.cc b/src/crush/CrushTester.cc
index d65cc5bba17ce55c05b5e444bc3253d58b07d1c1..07a40b5c2c0880c8fe04585c1a8332b76d5c0084 100644 (file)
--- a/src/crush/CrushTester.cc
+++ b/src/crush/CrushTester.cc
@@ -486,10 +486,6 @@ int CrushTester::test()
         err << "rule " << r << " dne" << std::endl;
       continue;
     }
-    if (ruleset >= 0 &&
-       crush.get_rule_mask_ruleset(r) != ruleset) {
-      continue;
-    }
     int minr = min_rep, maxr = max_rep;
     if (min_rep < 0 || max_rep < 0) {
       minr = crush.get_rule_mask_min_size(r);
@@ -732,10 +728,6 @@ int CrushTester::compare(CrushWrapper& crush2)
         err << "rule " << r << " dne" << std::endl;
       continue;
     }
-    if (ruleset >= 0 &&
-       crush.get_rule_mask_ruleset(r) != ruleset) {
-      continue;
-    }
     int minr = min_rep, maxr = max_rep;
     if (min_rep < 0 || max_rep < 0) {
       minr = crush.get_rule_mask_min_size(r);

diff --git a/src/crush/CrushTester.h b/src/crush/CrushTester.h
index b83b18782226fc7b8318efb3e1529abc41d61e6e..aa3d904b6222fbc3735bd861840a5247a1052904 100644 (file)
--- a/src/crush/CrushTester.h
+++ b/src/crush/CrushTester.h
@@ -14,7 +14,6 @@ class CrushTester {
 
   std::map<int, int> device_weight;
   int min_rule, max_rule;
-  int ruleset;
   int min_x, max_x;
   int min_rep, max_rep;
   int64_t pool_id;
@@ -169,7 +168,6 @@ public:
   CrushTester(CrushWrapper& c, std::ostream& eo)
     : crush(c), err(eo),
       min_rule(-1), max_rule(-1),
-      ruleset(-1),
       min_x(-1), max_x(-1),
       min_rep(-1), max_rep(-1),
       pool_id(-1),
@@ -341,10 +339,6 @@ public:
     min_rule = max_rule = rule;
   }
 
-  void set_ruleset(int rs) {
-    ruleset = rs;
-  }
-
   /**
    * check if any bucket/nodes is referencing an unknown name or type
    * @param max_id rejects any non-bucket items with id less than this number,

diff --git a/src/test/cli/crushtool/help.t b/src/test/cli/crushtool/help.t
index f233b9ad49e174c9732b1a2b75cd0aadeafcc2e8..d0b41a7863d095c35770c8877b4fab5447f95cb0 100644 (file)
--- a/src/test/cli/crushtool/help.t
+++ b/src/test/cli/crushtool/help.t
@@ -91,7 +91,7 @@
                            show location for given device id
      -i mapfn --test       test a range of inputs on the map
         [--min-x x] [--max-x x] [--x x]
-        [--min-rule r] [--max-rule r] [--rule r] [--ruleset rs]
+        [--min-rule r] [--max-rule r] [--rule r]
         [--num-rep n]
         [--pool-id n]      specifies pool id
         [--batches b]      split the CRUSH mapping into b > 1 rounds

diff --git a/src/tools/crushtool.cc b/src/tools/crushtool.cc
index 205af9d22d6c2d0e6df3bd561114efa258399351..df8d2cf45b35414bbaa52a4e1b96914f88b23500 100644 (file)
--- a/src/tools/crushtool.cc
+++ b/src/tools/crushtool.cc
@@ -199,7 +199,7 @@ void usage()
   cout << "                         show location for given device id\n";
   cout << "   -i mapfn --test       test a range of inputs on the map\n";
   cout << "      [--min-x x] [--max-x x] [--x x]\n";
-  cout << "      [--min-rule r] [--max-rule r] [--rule r] [--ruleset rs]\n";
+  cout << "      [--min-rule r] [--max-rule r] [--rule r]\n";
   cout << "      [--num-rep n]\n";
   cout << "      [--pool-id n]      specifies pool id\n";
   cout << "      [--batches b]      split the CRUSH mapping into b > 1 rounds\n";
@@ -783,12 +783,6 @@ int main(int argc, const char **argv)
        return EXIT_FAILURE;
       }
       tester.set_rule(x);
-    } else if (ceph_argparse_witharg(args, i, &x, err, "--ruleset", (char*)NULL)) {
-      if (!err.str().empty()) {
-       cerr << err.str() << std::endl;
-       return EXIT_FAILURE;
-      }
-      tester.set_ruleset(x);
     } else if (ceph_argparse_witharg(args, i, &x, err, "--batches", (char*)NULL)) {
       if (!err.str().empty()) {
        cerr << err.str() << std::endl;