1
0
mirror of https://github.com/ceph/ceph synced 2025-04-01 23:02:17 +00:00
Commit Graph

105998 Commits

Author SHA1 Message Date
Patrick Donnelly
c755740376
Merge PR into master
* refs/pull/32467/head:
	qa: fix various py3 cephfs qa bugs

Reviewed-by: Thomas Bechtold <tbechtold@suse.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
2020-01-06 12:25:34 -08:00
Patrick Donnelly
0997d24ddd
Merge PR into master
* refs/pull/32164/head:
	mount.ceph: give a hint message when no mds is up or cluster is laggy

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Jeff Layton <jlayton@redhat.com>
2020-01-06 12:19:01 -08:00
Patrick Donnelly
02b3883dd0
Merge PR into master
* refs/pull/32363/head:
	qa: add .qa link

Reviewed-by: Sage Weil <sage@redhat.com>
2020-01-06 12:18:12 -08:00
Kefu Chai
13acfe0491
Merge pull request from tchaikov/wip-qa-test_envlibrados_for_rocksdb.sh
qa: enable CRB repo for RHEL8

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
2020-01-07 01:46:14 +08:00
Kefu Chai
f05696ebeb qa: enable CRB repo for RHEL8
instead of enabling PowerTools repo, we need to enable CodeReady Builder
repo for RHEL8

also, since we are moving to RHEL8, there is no need to install cmake3
specifically for CentOS. CentOS 8 comes with cmake3.

Signed-off-by: Kefu Chai <kchai@redhat.com>
2020-01-06 23:41:23 +08:00
Jan Fajerski
69df8573bf
Merge pull request from rishabh-d-dave/cv-methods-for-lvm-opt-S
ceph-volume: add methods to pass filters to pvs, vgs and lvs commands
2020-01-06 15:53:52 +01:00
Rishabh Dave
a4f2fcefb8 ceph-volume: add methods to pass filters to pvs, vgs and lvs commands
Filters can be passed to these commands by using option '-S'.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
2020-01-06 19:27:42 +05:30
Kefu Chai
fd0cbcfee4
Merge pull request from tchaikov/wip-clang
common,crimson: fixes to compile with clang and libc++

Reviewed-by: Ronen Friedman <rfriedma@redhat.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
2020-01-06 14:43:13 +08:00
Kefu Chai
dba2b80897
Merge pull request from luis-henrix/copy-from-notrunc
osd: add a 'copy-from2' operation that includes truncate_{seq,size} parameters

Reviewed-by: Jeff Layton <jlayton@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
Reviewed-by: Greg Farnum <gfarnum@redhat.com>
2020-01-06 10:07:45 +08:00
Kefu Chai
3535e24181
Merge pull request from ronen-fr/clang_3
crimson: CLANG-related fixes to errorator.h

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
2020-01-06 10:00:14 +08:00
Kefu Chai
ccc59a64c2
Merge pull request from rzarzynski/wip-crimson-missing_err_include
crimson: add missing include in common/errorator.h

Reviewed-by: Kefu Chai <kchai@redhat.com>
2020-01-06 09:57:46 +08:00
Radoslaw Zarzynski
9c51035545 crimson: add missing include in common/errorator.h
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
2020-01-05 22:17:10 +01:00
Ronen Friedman
f81054d5ce crimson: fix non-standard C++ constructs in errorator.h
Fixing use patterns accepted by gcc, but frowned upon by Clang:

Put noexcept specifier after attributes specifiers (as
per the C++ standard $11.3.5)

Limit the use of a not-completely-defined class names to
within function bodies, per the standard ("A class is considered
a completely-defined object type (or complete type) at the
closing } of the class-specifier.
Within the class member-specification, the class is regarded as
complete within function bodies, default arguments, noexcept-specifiers,
and default member initializers (including such things in nested classes).
Otherwise it is regarded as incomplete within its own class member-specification.")

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
2020-01-05 19:50:52 +02:00
Radoslaw Zarzynski
12aafcd769 crimson: fix FTBFS on Clang induced by errorator.
Reported-by Ronen Friedman <rfriedma@redhat.com>
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
2020-01-05 18:00:37 +02:00
Kefu Chai
598c37296c common: add missing #include
clang++-9.0.1 and libc++ require them to build

Signed-off-by: Kefu Chai <kchai@redhat.com>
2020-01-05 22:53:48 +08:00
Kefu Chai
eb24936cfc crimson: do not use auto in param list of function prototype
src/crimson/net/Socket.cc:114:57: error: 'auto' not allowed in function
prototype
static inline seastar::future<> close_and_handle_errors(auto& out) {
                                                        ^~~~

Signed-off-by: Kefu Chai <kchai@redhat.com>
2020-01-05 22:27:03 +08:00
Kefu Chai
ba0fc1fef3 crimson: do not capture unused variables
to silence warnings like

src/crimson/net/Socket.cc:93:41: warning: lambda capture 'this' is not
used [-Wunused-lambda-capture]
    return in.read_exactly(bytes).then([this](auto buf) {
                                        ^~~~

Signed-off-by: Kefu Chai <kchai@redhat.com>
2020-01-05 22:27:03 +08:00
Kefu Chai
acd4de57c4
Merge pull request from kalebskeithley/fedora32
do_cmake.sh: fedora-32 (rawhide) build with python-3.8

Reviewed-by: Kefu Chai <kchai@redhat.com>
2020-01-04 23:17:56 +08:00
Kefu Chai
f8cab0eed2
Merge pull request from tchaikov/wip=admin-build-doc-python3
admin/build-doc: always install python3-* for build deps

Reviewed-by: Brad Hubbard <bhubbard@redhat.com>
2020-01-04 22:16:16 +08:00
Kefu Chai
b87781c120 admin/build-doc: always install python3-* for build deps
since we've dropped the support of python36-*, i.e. python packages
provided by EPEL7 before RHEL7/CentOS7 included python3. as before the
inclusion of python3 as supported python3, python packages are named
python36-*. and they don't provide python3-*. so we had to install
python36-* explicitly. now that we are able to use the python3-*
packages, we can just install python3-*.

Signed-off-by: Kefu Chai <kchai@redhat.com>
2020-01-04 17:05:06 +08:00
Patrick Donnelly
6ea2730ea6
qa: fix various py3 cephfs qa bugs
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
2020-01-03 16:42:13 -08:00
Neha Ojha
bd24ccf90d
Merge pull request from rs-fabrica/doc_rados_operations_crush-map-edits
doc/rados/operations/crush-map-edits: recompile and set instructions

Reviewed-by: Neha Ojha <nojha@redhat.com>
2020-01-03 15:20:33 -08:00
Patrick Donnelly
dacdb85e81
Merge PR into master
* refs/pull/32442/head:
	doc/developer: fix dev mailing list address

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
2020-01-03 14:43:22 -08:00
Willem Jan Withagen
867fb0cfb0 doc/developer: fix dev mailing list address
Signed-off-by: Willem Jan Withagen <wjw@digiware.nl>
2020-01-03 17:52:20 +01:00
Kaleb S. KEITHLEY
5c80bb8f78 do_cmake.sh: fedora-32 (rawhide) build with python-3.8
Fedora-32 now has python-3.8.

Fedora-29 is EOL. All non-EOL versions of Fedora have python-3.7, except
Fedora-32

(Unclear to me why PYBUILD="3" isn't used, like Ubuntu and FreeBSD. That
would eliminate breakage when pyhon versions change in Rawhide.)

Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com>
2020-01-03 08:34:37 -05:00
Patrick Donnelly
aafef601f8
Merge PR into master
* refs/pull/32457/head:
	doc: fix Getting Started with CephFS

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
2020-01-02 18:26:03 -08:00
Casey Bodley
d21598b816
Merge pull request from matthewoliver/rgw_swift_auth_note
doc: Add a RGW swift auth note

Reviewed-by: Casey Bodley <cbodley@redhat.com>
2020-01-02 15:59:36 -05:00
Casey Bodley
d48e6f57dd
Merge pull request from zhangsw/rgw-cleanup-cls-reshard-const
rgw:cleanup the "magic string" usage in cls_rgw_client.cc

Reviewed-by: Yuval Lifshitz <yuvalif@yahoo.com>
2020-01-02 15:33:02 -05:00
Casey Bodley
1787b7c880
Merge pull request from zhangsw/rgw-cleanup-cls-guard-bucket-resharding
rgw: correct some error log about reshard in cls_rgw.cc.

Reviewed-by: Casey Bodley <cbodley@redhat.com>
2020-01-02 10:56:54 -05:00
Yuval Lifshitz
c46ea53a55
Merge pull request from zhangsw/rgw-cleanup-deletelc
rgw:cleanup:remove un-used class member in RGWDeleteLC
2020-01-02 17:56:27 +02:00
Casey Bodley
8e41c4b97f
Merge pull request from zhangsw/rgw-cleanup-rgw-admin-check-instance
rgw:cleanup:remove un-used create_new_bucket_instance in rgw_admin.cc

Reviewed-by: Casey Bodley <cbodley@redhat.com>
2020-01-02 10:51:23 -05:00
Jan Fajerski
fe80000491
Merge pull request from mykaul/osd_fsid
ceph-volume/lvm/activate.py: clarify error message: fsid refers to osd_fsid
2020-01-02 15:32:55 +01:00
Nathan Cutler
0e9b4f5bc8
Merge pull request from smithfarm/wip-make-dist
make-dist: drop Python 2/3 autoselect

Reviewed-by: Kefu Chai <kchai@redhat.com>
2020-01-02 14:14:17 +01:00
Yaniv Kaul
ff3ba9200a lvm/activate.py: clarify error message: fsid refers to osd_fsid
Users complained[1] the error message isn't clear, and they thought
it referred to the cluster fsid instead of the osd_fsid.
Made it clearer.

[1] 

Fixes: https://tracker.ceph.com/issues/43442
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>
2020-01-02 10:58:33 +02:00
Jos Collin
9f4a20e653
doc: fix Getting Started with CephFS
Signed-off-by: Jos Collin <jcollin@redhat.com>
2020-01-02 12:04:50 +05:30
Matthew Oliver
9f40b38d3e doc: Add a RGW swift auth note
Swift accounts are not like normal accounts, they are more akin to a
bank account that multile people could share. Or in the case of a cloud
it is usually mapped to the tenant.

Radosgw deals with this with a user and subuser, which is great, but a
little confusing. So this patch adds a note to those used to the Swift
API to make it more clear.

Signed-off-by: Matthew Oliver <moliver@suse.com>
2020-01-02 00:32:01 +00:00
Rodrigo Severo
442878f229 doc/rados/operations/crush-map-edits: including recompile and set crushmapo instructions
Including missing recompile and set crushmap sections in crush-map-edits

Signed-off-by: Rodrigo Severo <rodrigo@fabricadeideias.com>
2019-12-30 18:13:25 -03:00
Kefu Chai
2ca2221ddb
Merge pull request from tchaikov/wip-43414
rgw: svc.bucket: assign to optional<> using "="

Reviewed-by: Matt Benjamin <mbenjamin@redhat.com>
2019-12-30 13:44:43 +08:00
Sage Weil
f2e6d5df6b Merge PR into master
* refs/pull/32415/head:
	include/cpp-btree: use inline variable to define btree_is_key_compare_to
	include/cpp-btree: use std::conditional_t<>
	include/cpp-btree: use static_assert()
	include/interval_set: use interval_set when appropriate
	include/interval_set: rename some types
	include/cpp-btree: add emplace_hint()

Reviewed-by: Sage Weil <sage@redhat.com>
2019-12-28 20:31:33 -06:00
Mykola Golub
f3b25725fe
Merge pull request from dillaman/wip-41556-2
rbd-mirror: continue to isolate journal replay logic

Reviewed-by: Mykola Golub <mgolub@suse.com>
2019-12-28 09:03:56 +02:00
Kefu Chai
f12dbaeaa3
Merge pull request from tchaikov/qa-devices-schema-with-path
qa: add "path" to "device" output schema

Reviewed-by: Sage Weil <sage@redhat.com>
2019-12-28 12:18:15 +08:00
Kefu Chai
07f04541f2
Merge pull request from tchaikov/wip-rgw-rgw_log_entry
rgw: initialize member variables of rgw_log_entry

Reviewed-by: Sage Weil <sage@redhat.com>
2019-12-28 12:18:05 +08:00
Kefu Chai
cda2c50f55
Merge pull request from tchaikov/wip-vstart-runner-py3
qa/tasks/vstart_runner: write string to StringIO

Reviewed-by: Sage Weil <sage@redhat.com>
2019-12-28 12:16:55 +08:00
Kefu Chai
5fe27c2267 qa/tasks/vstart_runner: write string to StringIO
we use `six.StringIO` for stdout when writing output messages from
`self.subproc.communicate()`, and `six.StringIO` accepts strings, so we
need to decode the output before sending them to `six.StringIO`.

Signed-off-by: Kefu Chai <kchai@redhat.com>
2019-12-28 01:11:54 +08:00
Nathan Cutler
249bf81a32 make-dist: drop Python 2/3 autoselect
virtualenv will prefer Python 3 over Python 2 if both are available.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
2019-12-27 10:55:57 +01:00
Kefu Chai
71cb0fdc64 rgw: svc.bucket: assign to optional<> using "="
instead of using

*optional_val = value;

we should use:

optional_val = value;

otherwise we will have following runtime assertion failure like

/usr/include/c++/8/optional:945: constexpr _Tp& std::_Optional_base<_Tp, true, true>::_M_get() [with _Tp = RGWBucketInfo*]: Assertion 'this->_M_is_engaged()' failed.

when compiled with the `_GLIBCXX_ASSERTIONS` macro defined. and recent
`redhat-rpm-config` adds `-D_GLIBCXX_ASSERTIONS` to
`__global_compiler_flags`, which is in turn included by `optflags`. and
`optflags` is pratically used everywhere.

Fixes: https://tracker.ceph.com/issues/43414
Signed-off-by: Kefu Chai <kchai@redhat.com>
2019-12-27 11:38:08 +08:00
Kefu Chai
c1da125974 rgw: svc.bucket: move _orig_info out of "if" block
so `*orig_info` is valid out of the `if` block if it's set to
`&_orig_info`.

Signed-off-by: Kefu Chai <kchai@redhat.com>
2019-12-27 11:36:05 +08:00
Kefu Chai
570f18f4c8 qa: add "path" to "device" output schema
"path" was added by 2c0fd7d868, so update
the test accordingly

Signed-off-by: Kefu Chai <kchai@redhat.com>
2019-12-27 00:25:51 +08:00
Jason Dillaman
b89a16f786 rbd-mirror: set proper state after journal replay shut down
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
2019-12-26 08:58:58 -05:00
Kefu Chai
19f1c55541 rgw: initialize member variables of rgw_log_entry
`check-generated.sh` tests the dencoder of a class by:

ceph-dencoder type rgw_log_entry select_test 2 dump_json > /tmp/foo
ceph-dencoder type rgw_log_entry select_test 2 encode decode dump_json > /tmp/bar
cmp /tmp/foo /tmp/bar

but the second test instance created by
`rgw_log_entry::generate_test_instances` is created using the default
constructor of `rgw_log_entry`. if any of the member variable is created
using random bits in the heap, there is chance that the test fails like:

2 rgw_log_entry
/tmp/typ-l5omDhmDf /tmp/typ-4aoRVkoiV differ: char 315, line 15
**** rgw_log_entry test 2 dump_json check failed ****
   ceph-dencoder type rgw_log_entry select_test 2 dump_json > /tmp/typ-l5omDhmDf
   ceph-dencoder type rgw_log_entry select_test 2 encode decode dump_json > /tmp/typ-4aoRVkoiV
15c15
<     "obj_size": 809908640,
---
>     "obj_size": 1122974112,

Signed-off-by: Kefu Chai <kchai@redhat.com>
2019-12-26 21:42:20 +08:00