Merge pull request ceph#61034 from aclamk/wip-aclamk-jenkins-reduced-unittest-bluefs

os/bluestore: Make some unittest_bluefs runs lighter on jenkins

Author: aclamk

qa/suites/rados/objectstore/backends/ceph_test_bluefs.yaml

@@ -0,0 +1,8 @@
+roles:
+- [mon.a, mgr.x, osd.0, osd.1, client.0]
+tasks:
+- install:
+- exec:
+    client.0:
+      - mkdir $TESTDIR/ceph_test_bluefs && cd $TESTDIR/ceph_test_bluefs && ceph_test_bluefs --log-file $TESTDIR/archive/ceph_test_bluefs.log --debug-bluefs 5/20 --gtest_catch_exceptions=0
+      - rm -rf $TESTDIR/ceph_test_bluefs

src/test/CMakeLists.txt

@@ -591,6 +591,7 @@ if(NOT WIN32)
   ceph_snappy
   cls_lock
   ceph_test_objectstore
+  ceph_test_bluefs
   ceph_erasure_code_non_regression
   cython_modules
   crushtool

src/test/objectstore/CMakeLists.txt

@@ -48,6 +48,18 @@ add_executable(unittest_rocksdb_option
 add_ceph_unittest(unittest_rocksdb_option)
 target_link_libraries(unittest_rocksdb_option global os ${BLKID_LIBRARIES})
 
+# ceph_test_bluefs (a clone of unittest_bluefs)
+add_executable(ceph_test_bluefs
+  test_bluefs.cc
+  )
+target_link_libraries(ceph_test_bluefs
+  os
+  global
+  ${UNITTEST_LIBS}
+  )
+install(TARGETS ceph_test_bluefs
+  DESTINATION ${CMAKE_INSTALL_BINDIR})
+
 if(WITH_EVENTTRACE)
   add_dependencies(os eventtrace_tp)
 endif()

src/test/objectstore/test_bluefs.cc

@@ -23,6 +23,11 @@
 
 using namespace std;
 
+// some test should not be executed on jenkins make check
+#define SKIP_JENKINS() \
+  if (getenv("JENKINS_HOME") != nullptr) GTEST_SKIP_("test disabled on jenkins");
+
+
 std::unique_ptr<char[]> gen_buffer(uint64_t size)
 {
     std::unique_ptr<char[]> buffer = std::make_unique<char[]>(size);
@@ -174,6 +179,7 @@ TEST(BlueFS, small_appends) {
 }
 
 TEST(BlueFS, very_large_write) {
+  SKIP_JENKINS();
   // we'll write a ~5G file, so allocate more than that for the whole fs
   uint64_t size = 1048576 * 1024 * 6ull;
   TempBdev bdev{size};
@@ -248,6 +254,7 @@ TEST(BlueFS, very_large_write) {
 }
 
 TEST(BlueFS, very_large_write2) {
+  SKIP_JENKINS();
   // we'll write a ~5G file, so allocate more than that for the whole fs
   uint64_t size_full = 1048576 * 1024 * 6ull;
   uint64_t size = 1048576 * 1024 * 5ull;