This option is now filtered out automatically, so these codes are unaccessable.
./bin/ceph osd unset sortbitwise
Invalid command: sortbitwise not in full|pause|noup|nodown|noout|noin|nobackfill|norebalance|norecover|noscrub|nodeep-scrub|notieragent
osd unset full|pause|noup|nodown|noout|noin|nobackfill|norebalance|norecover|noscrub|nodeep-scrub|notieragent : unset <key>
Error EINVAL: invalid command
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
On Linux systems a socket listening on :: also listens on
all IPv4 addresses.
This makes sure the API is usable over IPv6 as well.
Signed-off-by: Wido den Hollander <wido@42on.com>
Fixes the Coverity Scan Report:
CID 1412776 (#1 of 1): Explicit null dereferenced (FORWARD_NULL)19. var_deref_model: Passing null pointer option_name to operator <<, which dereferences it.
Fixed the review comments too in this commit.
Signed-off-by: Jos Collin <jcollin@redhat.com>
large zero object has a large compression rate,
even 4M compressed data can decompress several GB data.
Handle so much data in single process lead strange issue.
Fixes: http://tracker.ceph.com/issues/20098
Signed-off-by: fang yuxiang fang.yuxiang@eisoo.com
* `cct` is not globally avaiable, use cct() wrapper instead
* libzfs.h is always avaiable here. so remove the guard
Signed-off-by: Kefu Chai <kchai@redhat.com>
* silence clang analyzer's warning of: "Value stored to 'r' is never
read"
* replace the "goto" statement with early return to improve the
readability
Signed-off-by: Kefu Chai <kchai@redhat.com>
* silence the warning of: Value stored to 'r' is never read
* update the gtest assertions to be semantically more correct.
Signed-off-by: Kefu Chai <kchai@redhat.com>
The ".." form only works within the teuthology repo. With swift.py now in the
Ceph repo, we have to be explicit.
Error message was: "ValueError: Attempted relative import beyond toplevel
package
Signed-off-by: Nathan Cutler <ncutler@suse.com>
Fixes the Coverity Scan Report:
CID 1411820 (#1 of 1): Division or modulo by zero (DIVIDE_BY_ZERO)
9. divide_by_zero: In expression bl.length() * i / sum, division by expression sum which may be zero has undefined behavior.
Signed-off-by: Jos Collin <jcollin@redhat.com>
osd: unlock sdata_op_ordering_lock with sdata_lock hold to avoid miss…
Reviewed-by: Sage Weil <sage@redhat.com>
Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
Fixes The Coverity Scan Report:
CID 1412577 (#1 of 1): Division or modulo by float zero (DIVIDE_BY_ZERO)
35. divide_by_zero: In expression (float)mk / k, division by expression k which may be zero has undefined behavior.
Signed-off-by: Jos Collin <jcollin@redhat.com>
9164804416 introduced a build error:
ceph-common-12.1.0+git.1498286248.2fcedc7b3d-1.1.x86_64.rpm: directories not
owned by a package:
- /usr/share/doc/packages/ceph
The %docdir directive is a way of flagging anything in that directory as being
documentation. It does not actually package the directory. And we don't need
it because we're not dumping a large number of files into this directory.
For more information, see the "Directory-related Directives" section of
http://ftp.rpm.org/max-rpm/s1-rpm-inside-files-list-directives.html
Signed-off-by: Nathan Cutler <ncutler@suse.com>
In the front of this file, '__func__' has been added to dout_prefix as below:
" #define dout_prefix *_dout << "mgr.server " << __func__ << " " "
Signed-off-by: Yan Jun <yan.jun8@zte.com.cn>
If the underlying disk is missing, the current logic won't check the actual
reason why Get/Set failed, it will result to client get a empty key/value
pair which is not expected. The correct logic should be abort
right now. Otherwise, it will leads to undefined behavior.
Signed-off-by: Haomai Wang <haomai@xsky.com>
