]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
Merge pull request #58052 from jamiepryde/isal-default
authorYuri Weinstein <yweinste@redhat.com>
Tue, 21 Jan 2025 18:54:52 +0000 (10:54 -0800)
committerGitHub <noreply@github.com>
Tue, 21 Jan 2025 18:54:52 +0000 (10:54 -0800)
core: Change the default plugin for Ceph erasure coded pools from Jerasure to ISA-L

Reviewed-by: Afreen Misbah <afreen23.git@gmail.com>
Reviewed-by: Mark Nelson <mnelson@redhat.com>
1  2 
doc/rados/operations/erasure-code-jerasure.rst
doc/rados/operations/erasure-code.rst
qa/standalone/mon/osd-erasure-code-profile.sh
src/common/options/global.yaml.in
src/test/erasure-code/ceph_erasure_code_benchmark.cc
src/vstart.sh

Simple merge
index e09e298bc9f6f2c01a160a879a8d2f1fe4231d8c,79df86ae24f42b64f4ad18a734641c8973f238e4..f0f02e1500fae777e5fb19d1f1e44d1b291803a0
@@@ -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
Simple merge
index 00054bf4784d8c4581f258f220fb022f81cf9be5,4e98406befd3905accbc697bb340609fc063a0db..2849542a177788dd2590e5c27925e3a6688419a3
@@@ -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<int>()->default_value(1024 * 1024),
 +    ("size,s", po::value<int>()->default_value(80 * 1024 * 1024),
       "size of the buffer to be encoded")
 -    ("iterations,i", po::value<int>()->default_value(1),
 +    ("iterations,i", po::value<int>()->default_value(100),
       "number of encode/decode runs")
-     ("plugin,p", po::value<string>()->default_value("jerasure"),
+     ("plugin,p", po::value<string>()->default_value("isa"),
       "erasure code plugin name")
      ("workload,w", po::value<string>()->default_value("encode"),
       "run either encode or decode")
diff --cc src/vstart.sh
Simple merge