]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph_test_rados_api_tier_cxx: remove luminous check
authorSage Weil <sage@redhat.com>
Sat, 16 Mar 2019 20:21:37 +0000 (15:21 -0500)
committerSage Weil <sage@redhat.com>
Sat, 16 Mar 2019 20:21:37 +0000 (15:21 -0500)
It was a flawed check, since it would fail for anything newer than
luminous.  And in any case, we don't need it anymore.

Signed-off-by: Sage Weil <sage@redhat.com>
src/test/librados/tier_cxx.cc

index e6acd304771026f6d419633c8d4777eaf63d5cc9..0d4fde535705a7ed384cdeba71cc80bb94711dba 100644 (file)
@@ -2900,19 +2900,6 @@ TEST_F(LibRadosTwoPoolsPP, CachePin) {
 }
 
 TEST_F(LibRadosTwoPoolsPP, SetRedirectRead) {
-  // skip test if not yet luminous
-  {
-    bufferlist inbl, outbl;
-    ASSERT_EQ(0, cluster.mon_command(
-               "{\"prefix\": \"osd dump\"}",
-               inbl, &outbl, NULL));
-    string s(outbl.c_str(), outbl.length());
-    if (s.find("luminous") == std::string::npos) {
-      cout << "cluster is not yet luminous, skipping test" << std::endl;
-      return;
-    }
-  }
-
   // create object
   {
     bufferlist bl;
@@ -6181,19 +6168,6 @@ TEST_F(LibRadosTwoPoolsECPP, CachePin) {
   cluster.wait_for_latest_osdmap();
 }
 TEST_F(LibRadosTwoPoolsECPP, SetRedirectRead) {
-  // skip test if not yet luminous
-  {
-    bufferlist inbl, outbl;
-    ASSERT_EQ(0, cluster.mon_command(
-               "{\"prefix\": \"osd dump\"}",
-               inbl, &outbl, NULL));
-    string s(outbl.c_str(), outbl.length());
-    if (s.find("luminous") == std::string::npos) {
-      cout << "cluster is not yet luminous, skipping test" << std::endl;
-      return;
-    }
-  }
-
   // create object
   {
     bufferlist bl;