]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
qa: drop leftover of FileStore
authorRadoslaw Zarzynski <rzarzyns@redhat.com>
Tue, 27 Jan 2026 15:59:42 +0000 (15:59 +0000)
committerRadoslaw Zarzynski <rzarzyns@redhat.com>
Tue, 27 Jan 2026 16:00:41 +0000 (16:00 +0000)
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
17 files changed:
qa/cephfs/tasks/cfuse_workunit_suites_ffsb.yaml
qa/standalone/ceph-helpers.sh
qa/standalone/osd/osd-dup.sh
qa/standalone/scrub/osd-scrub-repair.sh
qa/suites/rados/basic/tasks/repair_test.yaml
qa/suites/rados/perf/settings/optimized.yaml
qa/suites/rados/thrash-old-clients/thrashers/morepggrow.yaml
qa/suites/rados/thrash-old-clients/thrashers/pggrow.yaml
qa/suites/rados/thrash/thrashers/morepggrow.yaml
qa/suites/rados/thrash/thrashers/pggrow.yaml
qa/suites/rados/thrash/thrashers/pggrow_host.yaml
qa/tasks/ceph.conf.template
qa/tasks/ceph_deploy.py
qa/tasks/ceph_manager.py
qa/tasks/repair_test.py
qa/workunits/cephtool/test.sh
qa/workunits/rados/test_alloc_hint.sh

index 6a2b35a185589714e0503b6ebdf12e866855d074..680ccf481b4f5db1418544d33d5ec4c7051c8640 100644 (file)
@@ -3,9 +3,6 @@ overrides:
     log-ignorelist:
     - SLOW_OPS
     - slow request
-    conf:
-      osd:
-        filestore flush min: 0
 tasks:
 - check-counter:
     counters:
index ccf5f63ee874c4ebbc9654d4ed14a7ba0eab8c58..d3de6c1d634c217f4ffc61ba454bb63772cef4f5 100755 (executable)
@@ -417,7 +417,6 @@ function get_unused_port() {
 # CEPH_ARGS="--fsid=$(uuidgen) "
 # CEPH_ARGS+="--mon-host=127.0.0.1:7018 "
 # run_mon $dir a # spawn a mon and bind port 7018
-# run_mon $dir a --debug-filestore=20 # spawn with filestore debugging
 #
 # If mon_initial_members is not set, the default rbd pool is deleted
 # and replaced with a replicated pool with less placement groups to
@@ -676,61 +675,6 @@ EOF
 
 }
 
-function run_osd_filestore() {
-    local dir=$1
-    shift
-    local id=$1
-    shift
-    local osd_data=$dir/$id
-
-    local ceph_args="$CEPH_ARGS"
-    ceph_args+=" --osd-failsafe-full-ratio=.99"
-    ceph_args+=" --osd-journal-size=100"
-    ceph_args+=" --osd-scrub-load-threshold=2000"
-    ceph_args+=" --osd-data=$osd_data"
-    ceph_args+=" --osd-journal=${osd_data}/journal"
-    ceph_args+=" --chdir="
-    ceph_args+=$EXTRA_OPTS
-    ceph_args+=" --run-dir=$dir"
-    ceph_args+=" --admin-socket=$(get_asok_path)"
-    ceph_args+=" --debug-osd=20"
-    ceph_args+=" --debug-ms=1"
-    ceph_args+=" --debug-monc=20"
-    ceph_args+=" --log-file=$dir/\$name.log"
-    ceph_args+=" --pid-file=$dir/\$name.pid"
-    ceph_args+=" --osd-max-object-name-len=460"
-    ceph_args+=" --osd-max-object-namespace-len=64"
-    ceph_args+=" --enable-experimental-unrecoverable-data-corrupting-features=*"
-    ceph_args+=" "
-    ceph_args+="$@"
-    mkdir -p $osd_data
-
-    local uuid=`uuidgen`
-    echo "add osd$osd $uuid"
-    OSD_SECRET=$(ceph-authtool --gen-print-key)
-    echo "{\"cephx_secret\": \"$OSD_SECRET\"}" > $osd_data/new.json
-    ceph osd new $uuid -i $osd_data/new.json
-    rm $osd_data/new.json
-    ceph-osd -i $id $ceph_args --mkfs --key $OSD_SECRET --osd-uuid $uuid --osd-objectstore=filestore
-
-    local key_fn=$osd_data/keyring
-    cat > $key_fn<<EOF
-[osd.$osd]
-key = $OSD_SECRET
-EOF
-    echo adding osd$id key to auth repository
-    ceph -i "$key_fn" auth add osd.$id osd "allow *" mon "allow profile osd" mgr "allow profile osd"
-    echo start osd.$id
-    ceph-osd -i $id $ceph_args &
-
-    # If noup is set, then can't wait for this osd
-    if ceph osd dump --format=json | jq '.flags_set[]' | grep -q '"noup"' ; then
-      return 0
-    fi
-    wait_for_osd up $id || return 1
-
-
-}
 
 function test_run_osd() {
     local dir=$1
index ab442c53812b4a309489557e4f334f5d84ed52b8..4dd020584e12d91d2179026743d76f8bf39bedd4 100755 (executable)
@@ -12,9 +12,6 @@ function run() {
     export CEPH_ARGS
     CEPH_ARGS+="--fsid=$(uuidgen) --auth-supported=none "
     CEPH_ARGS+="--mon-host=$CEPH_MON "
-    # avoid running out of fds in rados bench
-    CEPH_ARGS+="--filestore_wbthrottle_xfs_ios_hard_limit=900 "
-    CEPH_ARGS+="--filestore_wbthrottle_btrfs_ios_hard_limit=900 "
     local funcs=${@:-$(set | sed -n -e 's/^\(TEST_[0-9a-z_]*\) .*/\1/p')}
     for func in $funcs ; do
         setup $dir || return 1
index 4296a0f72cab91b601ba6150f2db6ea9723598f1..a3f1083bc5875cb389ca4b333bc1ddaeecd23ee4 100755 (executable)
@@ -20,7 +20,6 @@ source $CEPH_ROOT/qa/standalone/scrub/scrub-helpers.sh
 
 if [ `uname` = FreeBSD ]; then
     # erasure coding overwrites are only tested on Bluestore
-    # erasure coding on filestore is unsafe
     # http://docs.ceph.com/en/latest/rados/operations/erasure-code/#erasure-coding-with-overwrites
     use_ec_overwrite=false
 else
index b1acc90a5b52049632e6d47fdf780f9a23eb1177..9f15a2236c95bbaed0e6335e105f96f0396daf27 100644 (file)
@@ -29,7 +29,6 @@ overrides:
       - or freeform for custom applications
     conf:
       osd:
-        filestore debug inject read err: true
         bluestore debug inject read err: true
 tasks:
 - repair_test:
index 46351c0d578aa4c8916879e3ba3848b170661d73..59150694722ba1707184c15b24230a6016b83ad2 100644 (file)
@@ -6,7 +6,6 @@ overrides:
         debug ms: "0/0"
         debug paxos: "0/0"
       osd:
-        debug filestore: "0/0"
         debug journal: "0/0"
         debug ms: "0/0"
         debug osd: "0/0"
@@ -40,7 +39,6 @@ overrides:
         debug osd: "0/0"
         debug optracker: "0/0"
         debug objclass: "0/0"
-        debug filestore: "0/0"
         debug journal: "0/0"
         debug ms: "0/0"
         debug mon: "0/0"
index bb65d6a600cb466e9a276d8a524c06de16ecf33f..cc1dbfe6dae20313d24a7b2f0e76ee7f54749fce 100644 (file)
@@ -6,8 +6,6 @@ overrides:
         osd scrub max interval: 120
         journal throttle high multiple: 2
         journal throttle max multiple: 10
-        filestore queue throttle high multiple: 2
-        filestore queue throttle max multiple: 10
         osd max backfills: 9
     log-ignorelist:
     - but it is still running
index 000550bd865d79e83487d388b365d6e31455f297..50e96b1db441426619ee91d0daf66ba471e56321 100644 (file)
@@ -8,7 +8,6 @@ overrides:
       osd:
         osd scrub min interval: 60
         osd scrub max interval: 120
-        filestore odsync write: true
         osd max backfills: 2
         osd snap trim sleep: .5
       mon:
index d07f051db4fa54c15d0481b3ed6bea2bbcffd57f..2d28ab28d2ca7a547a0f9cdf4e93bdbdf6c9e528 100644 (file)
@@ -6,8 +6,6 @@ overrides:
         osd scrub max interval: 120
         journal throttle high multiple: 2
         journal throttle max multiple: 10
-        filestore queue throttle high multiple: 2
-        filestore queue throttle max multiple: 10
         osd max backfills: 9
     log-ignorelist:
     - but it is still running
index 79f96fd0a8d2781a06dad18ada9883387ab0617c..b57739be43f97fd4e8e743613f9886c5c79781f2 100644 (file)
@@ -7,7 +7,6 @@ overrides:
       osd:
         osd scrub min interval: 60
         osd scrub max interval: 120
-        filestore odsync write: true
         osd max backfills: 2
         osd snap trim sleep: .5
       mon:
index d3c5a63d0d2228932304c462b1fa9f05588e2a97..bff6c981ec7364d4f6d2a4879b4710649cf448b9 100644 (file)
@@ -7,7 +7,6 @@ overrides:
       osd:
         osd scrub min interval: 60
         osd scrub max interval: 120
-        filestore odsync write: true
         osd max backfills: 2
         osd snap trim sleep: .5
       mon:
index 9fad8ecc6e749ff58fd92044755027c4ab5e92e8..a499a1dff26d1af89b86b07dd724430c9d80d3df 100644 (file)
@@ -3,8 +3,6 @@
        pid file = /var/run/ceph/$cluster-$name.pid
         auth supported = cephx
 
-       filestore xattr use omap = true
-
        mon clock drift allowed = 1.000
 
        osd crush chooseleaf type = 0
@@ -66,9 +64,6 @@
 
         journal zero on create = true
 
-       filestore ondisk finisher threads = 3
-       filestore apply finisher threads = 3
-
        bdev debug aio = true
        osd debug misdirected ops = true
 
        # 1m isn't quite enough
        mon_down_mkfs_grace = 2m
 
-       mon_warn_on_filestore_osds = false
-
 [client]
        rgw cache enabled = true
        rgw enable ops log = true
index 832e2a1f4b3b58b8211d4a8707c6f4a43b544a8b..8f4d5d0bf0929c8a6bd9cefd37272817aa857ca3 100644 (file)
@@ -244,10 +244,7 @@ def build_ceph_cluster(ctx, config):
                 if estatus != 0:
                     raise RuntimeError("ceph-deploy: Failed to zap osds")
             osd_create_cmd = './ceph-deploy osd create '
-            # first check for filestore, default is bluestore with ceph-deploy
-            if config.get('filestore') is not None:
-                osd_create_cmd += '--filestore '
-            elif config.get('bluestore') is not None:
+            if config.get('bluestore') is not None:
                 osd_create_cmd += '--bluestore '
             if config.get('dmcrypt') is not None:
                 osd_create_cmd += '--dmcrypt '
@@ -266,7 +263,7 @@ def build_ceph_cluster(ctx, config):
         for remote in osds.remotes.keys():
             # all devs should be lvm
             osd_create_cmd = './ceph-deploy osd create --debug ' + remote.shortname + ' '
-            # default is bluestore so we just need config item for filestore
+            # default is bluestore
             roles = ctx.cluster.remotes[remote]
             dev_needed = len([role for role in roles
                               if role.startswith('osd')])
@@ -282,19 +279,7 @@ def build_ceph_cluster(ctx, config):
             for device in devs:
                 device_split = device.split('/')
                 lv_device = device_split[-2] + '/' + device_split[-1]
-                if config.get('filestore') is not None:
-                    osd_create_cmd += '--filestore --data ' + lv_device + ' '
-                    # filestore with ceph-volume also needs journal disk
-                    try:
-                        jdevice = all_devs.pop(jdevs)
-                    except IndexError:
-                        raise RuntimeError("No device available for \
-                                            journal configuration")
-                    jdevice_split = jdevice.split('/')
-                    j_lv = jdevice_split[-2] + '/' + jdevice_split[-1]
-                    osd_create_cmd += '--journal ' + j_lv
-                else:
-                    osd_create_cmd += ' --data ' + lv_device
+                osd_create_cmd += ' --data ' + lv_device
                 estatus_osd = execute_ceph_deploy(osd_create_cmd)
                 if estatus_osd == 0:
                     log.info('successfully created osd')
@@ -860,10 +845,7 @@ def task(ctx, config):
              ceph-deploy-branch: my-ceph-deploy-branch
              only_mon: true
              keep_running: true
-             # either choose bluestore or filestore, default is bluestore
              bluestore: True
-             # or
-             filestore: True
              # skip install of mgr for old release using below flag
              skip-mgr: True  ( default is False )
              # to use ceph-volume instead of ceph-disk
index 0f7e92c5c2fb98af3c1dacbc43656e7fe53545e1..23b3675818e5408f46acb0fdca5c1c4b1b8197b0 100644 (file)
@@ -1334,7 +1334,7 @@ class OSDThrasher(Thrasher):
         if self.chance_trim_stale_osdmaps > 0:
             actions.append((self.trim_stale_osdmaps, self.chance_trim_stale_osdmaps))
 
-        for key in ['heartbeat_inject_failure', 'filestore_inject_stall']:
+        for key in ['heartbeat_inject_failure']:
             for scenario in [
                 (lambda:
                  self.inject_pause(key,
index cfd6ef79186a84ffd318eb0457178ee6227b34f3..0fef62997e74ec1de44d6e7add7a842fc1fc5a99 100644 (file)
@@ -246,7 +246,6 @@ def task(ctx, config):
     The config should be as follows:
 
       Must include the log-ignorelist below
-      Must enable filestore_debug_inject_read_err config
 
     example:
 
@@ -271,9 +270,6 @@ def task(ctx, config):
           - 'attr name mismatch'
           - 'Regular scrub request, deep-scrub details will be lost'
           - 'candidate size [0-9]+ info size [0-9]+ mismatch'
-        conf:
-          osd:
-            filestore debug inject read err: true
     - repair_test:
 
     """
index 57ec1fa0a09cc05492fd69c6fae86ab36237a177..70cad64a76b7d8b690095d27a65b50018c572b22 100755 (executable)
@@ -2681,12 +2681,6 @@ function test_osd_bench()
   ceph tell osd.0 bench 104857600 2097152
 }
 
-function test_osd_negative_filestore_merge_threshold()
-{
-  $SUDO ceph daemon osd.0 config set filestore_merge_threshold -1
-  expect_config_value "osd.0" "filestore_merge_threshold" -1
-}
-
 function test_mon_tell()
 {
   for m in mon.a mon.b; do
@@ -2993,7 +2987,6 @@ MON_TESTS+=" mon_stdin_stdout"
 MON_TESTS+=" mon_messenger_dump"
 
 OSD_TESTS+=" osd_bench"
-OSD_TESTS+=" osd_negative_filestore_merge_threshold"
 OSD_TESTS+=" tiering_agent"
 OSD_TESTS+=" admin_heap_profiler"
 OSD_TESTS+=" osd_tell_help_command"
index 535201ca3c455ddf474a6caabec80131f86c1a52..a6c44e4a3ccbd2bed715d96066bb53b3f1b581bf 100755 (executable)
@@ -51,7 +51,7 @@ function setup_pgid() {
 }
 
 function expect_alloc_hint_eq() {
-    export CEPH_ARGS="--osd-objectstore=filestore"
+    # This is running with filestore. Is it even run with BlueStore?
     local expected_extsize="$1"
 
     for (( i = 0 ; i < "${NUM_OSDS}" ; i++ )); do
@@ -96,8 +96,7 @@ NUM_OSDS="$((EC_K + EC_M))"
 NUM_PG="12"
 NUM_PGP="${NUM_PG}"
 
-LOW_CAP="$(get_conf_val "osd.0" "filestore_max_alloc_hint_size")"
-HIGH_CAP="$((LOW_CAP * 10))" # 10M, assuming 1M default cap
+LOW_CAP="$((1 * 1024 * 1024))" # 1M
 SMALL_HINT="$((LOW_CAP / 4))" # 256K, assuming 1M default cap
 BIG_HINT="$((LOW_CAP * 6))" # 6M, assuming 1M default cap
 
@@ -108,7 +107,7 @@ setup_osd_data
 #
 
 POOL="alloc_hint-rep"
-ceph osd pool create "${POOL}" "${NUM_PG}"
+ceph osd pool create "${POOL}" "${NUM_PG}" # 1M
 ceph osd pool set "${POOL}" size "${NUM_OSDS}" --yes-i-really-mean-it
 ceph osd pool application enable "${POOL}" rados
 
@@ -124,15 +123,10 @@ expect_alloc_hint_eq "${SMALL_HINT}"
 rados -p "${POOL}" set-alloc-hint "${OBJ}" "${BIG_HINT}" "${BIG_HINT}"
 expect_alloc_hint_eq "${LOW_CAP}"
 
-# Bump the cap to HIGH_CAP
-ceph tell 'osd.*' injectargs "--filestore_max_alloc_hint_size ${HIGH_CAP}"
-
-# Try changing to BIG_HINT (2) - expect BIG_HINT (BIG_HINT < HIGH_CAP)
+# Try changing to BIG_HINT (2) - expect BIG_HINT
 rados -p "${POOL}" set-alloc-hint "${OBJ}" "${BIG_HINT}" "${BIG_HINT}"
 expect_alloc_hint_eq "${BIG_HINT}"
 
-ceph tell 'osd.*' injectargs "--filestore_max_alloc_hint_size ${LOW_CAP}"
-
 # Populate object with some data
 rados -p "${POOL}" put "${OBJ}" /etc/passwd