Bug #9579
closedDefault parameters are not getting initialized for EC profile using isa EC plugin
0%
Description
When created an EC profile using erasure code plugin "isa", default values for parameters k, m and technique are not getting initialized.
But, for erasure code plugin "jerasure", parameters are getting assigned with default values.
It's expected that it should initialize default values for EC profile parameters created with "isa" erasure code plug in.
ems@rack6-client-6:~$ sudo ceph osd erasure-code-profile set isaprofile plugin=isa
ems@rack6-client-6:~$
ems@rack6-client-6:~$ sudo ceph osd erasure-code-profile get isaprofile
directory=/usr/lib/ceph/erasure-code
plugin=isa
ems@rack6-client-6:~$
ems@rack6-client-6:~$ sudo ceph osd erasure-code-profile set jerasureprofile plugin=jerasure
ems@rack6-client-6:~$ sudo ceph osd erasure-code-profile get jerasureprofile
directory=/usr/lib/ceph/erasure-code
k=2
m=1
plugin=jerasure
technique=reed_sol_van
ems@rack6-client-6:~$
This issue is been observed on 0.84 CEPH.
Files
Updated by Loïc Dachary over 9 years ago
- Status changed from New to Won't Fix
- Assignee changed from Sage Weil to Loïc Dachary
There are no defaults for k/m for the isa plugin, the parameters need to be set explicitly as documented at http://ceph.com/docs/master/rados/operations/erasure-code-isa/
Updated by Mallikarjun Biradar over 9 years ago
From the code, it seems default value of k & m for "isa" profile are 7 & 3 respectively.
class ErasureCodeIsaDefault : public ErasureCodeIsa {
int matrixtype;
public:
static const int DEFAULT_K = 7;
static const int DEFAULT_M = 3;
static const int g_decode_tbls_lru_length=2516; // caches up to 12+4 completely
unsigned char* a; // encoding coefficient
unsigned char* g_encode_tbls; // encoding table
Updated by Loïc Dachary over 9 years ago
This is confusing and I added http://tracker.ceph.com/issues/9589 to work on improving the user experience. Thanks for your understanding.
Updated by Loïc Dachary over 9 years ago
Documentation update on k/m, good catch ! https://github.com/ceph/ceph/pull/2571