From: Yuri Weinstein Date: Tue, 21 Jan 2025 18:54:52 +0000 (-0800) Subject: Merge pull request #58052 from jamiepryde/isal-default X-Git-Tag: v20.0.0~349 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=a87a77c63fd6b14b197ff414f5b92b6ffb61b808;p=ceph.git Merge pull request #58052 from jamiepryde/isal-default core: Change the default plugin for Ceph erasure coded pools from Jerasure to ISA-L Reviewed-by: Afreen Misbah Reviewed-by: Mark Nelson --- a87a77c63fd6b14b197ff414f5b92b6ffb61b808 diff --cc qa/standalone/mon/osd-erasure-code-profile.sh index e09e298bc9f,79df86ae24f..f0f02e1500f --- a/qa/standalone/mon/osd-erasure-code-profile.sh +++ b/qa/standalone/mon/osd-erasure-code-profile.sh @@@ -52,12 -52,11 +52,12 @@@ function TEST_set() # key=value pairs override the default # ceph osd erasure-code-profile set $profile \ - key=value plugin=isa || return 1 + key=value plugin=jerasure || return 1 ceph osd erasure-code-profile get $profile | \ - grep -e key=value -e plugin=isa || return 1 + grep -e key=value -e plugin=jerasure || return 1 # - # --force is required to override an existing profile + # --force & --yes-i-really-mean-it are required to override + # an existing profile # ! ceph osd erasure-code-profile set $profile > $dir/out 2>&1 || return 1 grep 'will not override' $dir/out || return 1 diff --cc src/test/erasure-code/ceph_erasure_code_benchmark.cc index 00054bf4784,4e98406befd..2849542a177 --- a/src/test/erasure-code/ceph_erasure_code_benchmark.cc +++ b/src/test/erasure-code/ceph_erasure_code_benchmark.cc @@@ -52,11 -52,11 +52,11 @@@ int ErasureCodeBench::setup(int argc, c desc.add_options() ("help,h", "produce help message") ("verbose,v", "explain what happens") - ("size,s", po::value()->default_value(1024 * 1024), + ("size,s", po::value()->default_value(80 * 1024 * 1024), "size of the buffer to be encoded") - ("iterations,i", po::value()->default_value(1), + ("iterations,i", po::value()->default_value(100), "number of encode/decode runs") - ("plugin,p", po::value()->default_value("jerasure"), + ("plugin,p", po::value()->default_value("isa"), "erasure code plugin name") ("workload,w", po::value()->default_value("encode"), "run either encode or decode")