]> git.apps.os.sepia.ceph.com Git - ceph.git/log
ceph.git
4 years agomgr/volumes: Prevent subvolume recreate if trash is not-empty 36773/head
Shyamsundar Ranganathan [Wed, 26 Aug 2020 12:41:11 +0000 (08:41 -0400)]
mgr/volumes: Prevent subvolume recreate if trash is not-empty

Fixes: https://tracker.ceph.com/issues/47154
Signed-off-by: Shyamsundar Ranganathan <srangana@redhat.com>
4 years agomgr/volumes: Disallow subvolume group level snapshots
Shyamsundar Ranganathan [Tue, 25 Aug 2020 22:46:12 +0000 (18:46 -0400)]
mgr/volumes: Disallow subvolume group level snapshots

Fixes: https://tracker.ceph.com/issues/47154
Signed-off-by: Shyamsundar Ranganathan <srangana@redhat.com>
4 years agomgr/volumes: Add test case to ensure subvolume is marked
Shyamsundar Ranganathan [Tue, 25 Aug 2020 15:44:50 +0000 (11:44 -0400)]
mgr/volumes: Add test case to ensure subvolume is marked

with the vxattr ceph.dir.subvolume set to true.

Fixes: https://tracker.ceph.com/issues/47154
Signed-off-by: Shyamsundar Ranganathan <srangana@redhat.com>
4 years agomgr/volumes: handle idempotent subvolume marks
Venky Shankar [Tue, 25 Aug 2020 06:57:50 +0000 (02:57 -0400)]
mgr/volumes: handle idempotent subvolume marks

Fixes: https://tracker.ceph.com/issues/47154
Signed-off-by: Venky Shankar <vshankar@redhat.com>
4 years agomgr/volumes: Tests amended and added to ensure subvolume trash functionality
Shyamsundar Ranganathan [Mon, 24 Aug 2020 19:10:44 +0000 (15:10 -0400)]
mgr/volumes: Tests amended and added to ensure subvolume trash functionality

Amended a few test cases to ensure created subvolumes and snaps
are removed, and trash stays empty at the end of the test.

Further added one test case for create errors in a retained
v2 subvolume, to ensure metadata is sane, and created incarnation
is not present.

Fixes: https://tracker.ceph.com/issues/47154
Signed-off-by: Shyamsundar Ranganathan <srangana@redhat.com>
4 years agomgr/volumes: Mark subvolume root with the vxattr ceph.dir.subvolume
Shyamsundar Ranganathan [Mon, 24 Aug 2020 13:33:10 +0000 (09:33 -0400)]
mgr/volumes: Mark subvolume root with the vxattr ceph.dir.subvolume

This enables snapshot scaling of subvolumes, and hence prevents
renaming, hardlinking etc. outside the subvolume.

Fixes: https://tracker.ceph.com/issues/47154
Signed-off-by: Shyamsundar Ranganathan <srangana@redhat.com>
4 years agomgr/volumes: Move incarnations for v2 subvolumes, to subvolume trash
Shyamsundar Ranganathan [Sun, 23 Aug 2020 16:31:01 +0000 (12:31 -0400)]
mgr/volumes: Move incarnations for v2 subvolumes, to subvolume trash

For v2 subvolumes without any snapshots or pending purges
move the subvolume to trash, for the purge operation.

When removing only incarnations, leverage symlink based purge
job creation, to avoid renaming the incarnation out of the
subvolume.

For create errors, where the subvolume was retained, just remove
the created incarnation directory.

Fixes: https://tracker.ceph.com/issues/47154
Signed-off-by: Shyamsundar Ranganathan <srangana@redhat.com>
4 years agomgr/volumes: maintain per subvolume trash directory
Venky Shankar [Fri, 21 Aug 2020 14:07:37 +0000 (10:07 -0400)]
mgr/volumes: maintain per subvolume trash directory

PR https://github.com/ceph/ceph/pull/36472 introduces changes
that disallow nested nested snapshots in a subtree (subvolume)
and renames across subvolumes. This effect asynchronous purge
in mgr/volumes as subvolume are moved to a trash directory for
asynchronous deletion by purge threads.

To workaround this, start maintaining a subvolume specific
trash directory. Use the trash directory as an index to the
subvolume specific trash directory entry.

This changes subvolume deletion logic which currently relies
on `--retain-snapshots` flag to decide if the subvolume user
directory should get purged or the subvolume base directory
itself. Deleting a subvolume moves the user facing directory
to its specific trash directory. Purge threads take care of
deleting user facing directories (in trash) and the subvolume
base directory if required (when certain conditions are met).

Fixes: https://tracker.ceph.com/issues/47154
Signed-off-by: Venky Shankar <vshankar@redhat.com>
4 years agomgr/volumes: make subvolume_v2::_is_retained() object property
Venky Shankar [Fri, 21 Aug 2020 05:18:45 +0000 (01:18 -0400)]
mgr/volumes: make subvolume_v2::_is_retained() object property

... and drop the leading underscore.

Fixes: https://tracker.ceph.com/issues/47154
Signed-off-by: Venky Shankar <vshankar@redhat.com>
4 years agoMerge pull request #36521 from tchaikov/wip-rgw-optional
Kefu Chai [Sat, 22 Aug 2020 18:02:23 +0000 (02:02 +0800)]
Merge pull request #36521 from tchaikov/wip-rgw-optional

rgw: hold reloader using unique_ptr

Reviewed-by: Willem Jan Withagen <wjw@digiware.nl>
Reviewed-by: Daniel Gryniewicz <dang@redhat.com>
4 years agoMerge pull request #36707 from dillaman/wip-47007
Mykola Golub [Sat, 22 Aug 2020 10:12:05 +0000 (13:12 +0300)]
Merge pull request #36707 from dillaman/wip-47007

rbd-mirror: peer setup can still race and fail creation of peer

Reviewed-by: Mykola Golub <mgolub@suse.com>
4 years agoMerge pull request #36471 from dillaman/wip-41394
Mykola Golub [Sat, 22 Aug 2020 10:11:19 +0000 (13:11 +0300)]
Merge pull request #36471 from dillaman/wip-41394

librbd: using migration abort can result in the loss of data

Reviewed-by: Mykola Golub <mgolub@suse.com>
4 years agoMerge pull request #36380 from dillaman/wip-46754
Mykola Golub [Sat, 22 Aug 2020 10:10:17 +0000 (13:10 +0300)]
Merge pull request #36380 from dillaman/wip-46754

rbd: retrieve global config overrides from the MONs

Reviewed-by: Mykola Golub <mgolub@suse.com>
4 years agoMerge pull request #36751 from tchaikov/wip-redirect-stderr
Kefu Chai [Sat, 22 Aug 2020 08:59:14 +0000 (16:59 +0800)]
Merge pull request #36751 from tchaikov/wip-redirect-stderr

qa/tasks/ceph: create a log file before redirecting to it

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
4 years agoMerge pull request #36696 from tchaikov/wip-cmake-boost-zlib
Kefu Chai [Sat, 22 Aug 2020 04:49:15 +0000 (12:49 +0800)]
Merge pull request #36696 from tchaikov/wip-cmake-boost-zlib

cmake: link against ZLIB if statically linked against Boost

Reviewed-By: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
4 years agoMerge pull request #36710 from tchaikov/wip-ceph-pypath
Kefu Chai [Sat, 22 Aug 2020 04:48:34 +0000 (12:48 +0800)]
Merge pull request #36710 from tchaikov/wip-ceph-pypath

ceph: correct additional paths added to sys.path

Reviewed-By: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
4 years agoMerge pull request #36722 from tchaikov/wip-test-pybind
Kefu Chai [Sat, 22 Aug 2020 04:30:20 +0000 (12:30 +0800)]
Merge pull request #36722 from tchaikov/wip-test-pybind

test/pybind: label test_rados tests with attributes

Reviewed-By: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
4 years agoqa/tasks/ceph: create a log file before redirecting to it 36751/head
Kefu Chai [Fri, 21 Aug 2020 12:22:23 +0000 (20:22 +0800)]
qa/tasks/ceph: create a log file before redirecting to it

as it is shell who interprets ">>" and redirect the stderr to given
file, but the shell process is launched by ubuntu:ububunt without using
sudo, so the command fails with "Permission denied" failure. to address
this issue, in this change, a file with proper priviledges is created
beforehand using `install`, so shell is able to write to it.

also, instead of creating this file in `maybe_redirect_stderr()`, it
returns the command to create the log file.

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge PR #36560 into master
Patrick Donnelly [Sat, 22 Aug 2020 01:44:46 +0000 (18:44 -0700)]
Merge PR #36560 into master

* refs/pull/36560/head:
client: choose a random replica mds to send the request

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge PR #36681 into master
Patrick Donnelly [Sat, 22 Aug 2020 01:44:06 +0000 (18:44 -0700)]
Merge PR #36681 into master

* refs/pull/36681/head:
mds: don't track change of config 'mds_replay_unsafe_with_closed_session'
mds: fix 'forward loop' when forward_all_requests_to_auth is set

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge PR #36673 into master
Patrick Donnelly [Sat, 22 Aug 2020 01:42:50 +0000 (18:42 -0700)]
Merge PR #36673 into master

* refs/pull/36673/head:
mds: don't recover files after normal session close

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge PR #36632 into master
Patrick Donnelly [Sat, 22 Aug 2020 01:41:35 +0000 (18:41 -0700)]
Merge PR #36632 into master

* refs/pull/36632/head:
mds: standy-replay mds remained in the "resolve" state after restart the active mds

Reviewed-by: Zheng Yan <zyan@redhat.com>
4 years agoMerge PR #36131 into master
Patrick Donnelly [Sat, 22 Aug 2020 01:40:42 +0000 (18:40 -0700)]
Merge PR #36131 into master

* refs/pull/36131/head:
doc: document cephfs mirroring dev work
test: add tests for `ceph fs mirror` family of commands
mds: track filesystem mirror peers in fsmap

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge PR #36459 into master
Patrick Donnelly [Sat, 22 Aug 2020 01:39:01 +0000 (18:39 -0700)]
Merge PR #36459 into master

* refs/pull/36459/head:
mds: dump granular cap info in mds_sessions
common: fix validate type CephBool cause 'invalid command json'

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge PR #36598 into master
Patrick Donnelly [Sat, 22 Aug 2020 01:38:09 +0000 (18:38 -0700)]
Merge PR #36598 into master

* refs/pull/36598/head:
mds: drop decoding as MDSMap in FSMap::decode()

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge PR #36687 into master
Patrick Donnelly [Sat, 22 Aug 2020 01:37:21 +0000 (18:37 -0700)]
Merge PR #36687 into master

* refs/pull/36687/head:
client: use ccap_string to show caps in more user friendly

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge PR #36697 into master
Patrick Donnelly [Sat, 22 Aug 2020 01:35:56 +0000 (18:35 -0700)]
Merge PR #36697 into master

* refs/pull/36697/head:
messages,mds: Fix decoding of enum types on big-endian systems

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge PR #36472 into master
Patrick Donnelly [Fri, 21 Aug 2020 23:12:32 +0000 (16:12 -0700)]
Merge PR #36472 into master

* refs/pull/36472/head:
qa/workunits/fs: add test for subvolume
mds: don't move inode with nlink > 1 to global snaprealm if it's in subvolume
mds: disallow hardlink across subvolume
mds: disallow across subvolume rename
mds: disallow creating snapshot on descendent directory of subvolume
mds: add vxattr that marks/clears subvolume flag

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge pull request #35789 from Reperator/fix-ceph-objectstore-tool-docs
Kefu Chai [Fri, 21 Aug 2020 10:31:20 +0000 (18:31 +0800)]
Merge pull request #35789 from Reperator/fix-ceph-objectstore-tool-docs

doc/man/8/ceph-objectstore-tool.rst: remove duplicate lines

Reviewed-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge pull request #36742 from hulksmaaash/patch-2
Kefu Chai [Fri, 21 Aug 2020 10:26:46 +0000 (18:26 +0800)]
Merge pull request #36742 from hulksmaaash/patch-2

doc/rados: remove statement that ISA plugin only works on Intel chips

Reviewed-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge pull request #36496 from Daniel-Pivonka/cephadm-44926
Sebastian Wagner [Fri, 21 Aug 2020 09:40:17 +0000 (11:40 +0200)]
Merge pull request #36496 from Daniel-Pivonka/cephadm-44926

mgr/cephadm: rgw update period after realm, zone creation

Reviewed-by: Sebastian Wagner <sebastian.wagner@suse.com>
4 years agoMerge pull request #36716 from jmolmo/dashboard_ports_2
Sebastian Wagner [Fri, 21 Aug 2020 09:39:52 +0000 (11:39 +0200)]
Merge pull request #36716 from jmolmo/dashboard_ports_2

cephadm: Allow users to use a custom dashboard ssl port

Reviewed-by: Sebastian Wagner <sebastian.wagner@suse.com>
4 years agoMerge pull request #36721 from ceph/47036-sass-colors-typescript
Lenz Grimmer [Fri, 21 Aug 2020 09:12:33 +0000 (11:12 +0200)]
Merge pull request #36721 from ceph/47036-sass-colors-typescript

mgr/dashboard: use SASS colors in TypeScript

Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Sebastian Krah <skrah@suse.com>
4 years agoMerge pull request #36735 from rhcs-dashboard/fix-this-week-endpoint
Lenz Grimmer [Fri, 21 Aug 2020 09:10:18 +0000 (11:10 +0200)]
Merge pull request #36735 from rhcs-dashboard/fix-this-week-endpoint

mgr/dashboard: remove "This week/month/year" and "Today" time stamps of Grafana in dashboard

Reviewed-by: Alfonso Martínez <almartin@redhat.com>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
4 years agotest/pybind: remove object for testing after done with it 36722/head
Kefu Chai [Fri, 21 Aug 2020 05:30:24 +0000 (13:30 +0800)]
test/pybind: remove object for testing after done with it

* add vertical spacing for better readability
* remove object for testing after done with it

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agotest/pybind: label test_rados tests with attributes
Kefu Chai [Wed, 19 Aug 2020 17:03:41 +0000 (01:03 +0800)]
test/pybind: label test_rados tests with attributes

so we can pick the tests selectively. currently, some features are not
supported by crimson. but we will cover more and more of them. so
attaching different labels to tests cases helps us to run these tests
selectively.

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agoqa/workunits/rados: pass options down to test_rados.py
Kefu Chai [Wed, 19 Aug 2020 17:06:10 +0000 (01:06 +0800)]
qa/workunits/rados: pass options down to test_rados.py

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge pull request #36740 from varshar16/wip-fix-nfs-bug-47009 36749/head
Sebastian Wagner [Fri, 21 Aug 2020 07:58:12 +0000 (09:58 +0200)]
Merge pull request #36740 from varshar16/wip-fix-nfs-bug-47009

qa/tasks/nfs: Change the mds name and delete created nfs clusters

Reviewed-by: Sebastian Wagner <sebastian.wagner@suse.
4 years agoMerge pull request #36694 from shun-s/fix-linger-op-stray
Kefu Chai [Fri, 21 Aug 2020 07:33:14 +0000 (15:33 +0800)]
Merge pull request #36694 from shun-s/fix-linger-op-stray

osdc: fix lingerOp stray

Reviewed-By: Josh Durgin <jdurgin@redhat.com>
4 years agomgr/dashboard: remove "This week/month/year" and "Today" time stamps of Grafana in... 36735/head
Avan Thakkar [Thu, 20 Aug 2020 09:32:33 +0000 (15:02 +0530)]
mgr/dashboard: remove "This week/month/year" and "Today" time stamps of Grafana in dashboard

Fixes: https://tracker.ceph.com/issues/47049
Signed-off-by: Avan Thakkar <athakkar@redhat.com>
4 years agoMerge pull request #36465 from majianpeng/bluefs-avoid-flush-much-data
Kefu Chai [Fri, 21 Aug 2020 01:57:27 +0000 (09:57 +0800)]
Merge pull request #36465 from majianpeng/bluefs-avoid-flush-much-data

os/bluestore/BlueRocksEnv: Avoid flushing too much data at once.

Reviewed-by: Mark Nelson <mnelson@redhat.com>
Reviewed-by: Adam Kupczyk <akucpzyk@redhat.com>
4 years agoMerge pull request #36593 from gregsfortytwo/wip-46914
Kefu Chai [Fri, 21 Aug 2020 01:53:25 +0000 (09:53 +0800)]
Merge pull request #36593 from gregsfortytwo/wip-46914

mon: mark pgtemp messages as no_reply more consistenly in preprocess_…

Reviewed-by: Neha Ojha <nojha@redhat.com>
4 years agoMerge pull request #36675 from tchaikov/wip-fix-option-typo
Kefu Chai [Fri, 21 Aug 2020 01:52:40 +0000 (09:52 +0800)]
Merge pull request #36675 from tchaikov/wip-fix-option-typo

common/options: fix a typo in option description

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-By: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
4 years agoMerge pull request #35613 from changchengx/str_list
Kefu Chai [Fri, 21 Aug 2020 01:51:28 +0000 (09:51 +0800)]
Merge pull request #35613 from changchengx/str_list

use for_each_substr to avoid redundant operation by calling get_str_set

Reviewed-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge pull request #36573 from changchengx/osdmonitor
Kefu Chai [Fri, 21 Aug 2020 01:50:11 +0000 (09:50 +0800)]
Merge pull request #36573 from changchengx/osdmonitor

mon: use string_veiw to glimpse string and remove unused variable

Reviewed-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge pull request #36680 from tchaikov/wip-mgr-disk-predict
Kefu Chai [Fri, 21 Aug 2020 01:46:54 +0000 (09:46 +0800)]
Merge pull request #36680 from tchaikov/wip-mgr-disk-predict

pybind/mgr/diskprediction_cloud: remove mgr-diskprediction-cloud

Reviewed-By: Josh Durgin <jdurgin@redhat.com>
Reviewed-By: Zac Dover <zac.dover@gmail.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
Reviewed-by: Laura Paduano <lpaduano@suse.com>
4 years agoMerge pull request #36534 from changchengx/formatter
Kefu Chai [Fri, 21 Aug 2020 01:45:03 +0000 (09:45 +0800)]
Merge pull request #36534 from changchengx/formatter

common/Formatter: don't construct string if string_view work

Reviewed-By: Kefu Chai <kchai@redhat.com>
4 years agoMerge pull request #36420 from qwren/qat-compressor
Kefu Chai [Fri, 21 Aug 2020 01:42:18 +0000 (09:42 +0800)]
Merge pull request #36420 from qwren/qat-compressor

compressor: fix compile error for QAT

Reviewed-By: Kefu Chai <kchai@redhat.com>
4 years agodoc/rados: remove statement that ISA plugin only works on Intel chips 36742/head
Robbie Williamson [Thu, 20 Aug 2020 20:45:36 +0000 (15:45 -0500)]
doc/rados: remove statement that ISA plugin only works on Intel chips

ISA-L supports aarch64 (Arm), not just Intel processors, as does Ceph (see #32171).

Signed-off-by: Robbie Williamson <robbie.williamson@arm.com>
4 years agomon: mark pgtemp messages as no_reply more consistently in preprocess_pgtemp 36593/head
Greg Farnum [Wed, 12 Aug 2020 23:44:11 +0000 (23:44 +0000)]
mon: mark pgtemp messages as no_reply more consistently in preprocess_pgtemp

If a message is forwarded, it's conceivable the leader's and peon's evaluation
will disagree about whether the message is useful or not, which could result
in the leader ignoring it and the peon having a dangling forwarded message.
Fix this by marking the op as no_reply whenever ignoring it.

Fixes: https://tracker.ceph.com/issues/46914
Signed-off-by: Greg Farnum <gfarnum@redhat.com>
4 years agoqa/tasks/nfs: Instead of 'orch ls' use 'orch ps' 36740/head
Varsha Rao [Thu, 20 Aug 2020 14:36:47 +0000 (14:36 +0000)]
qa/tasks/nfs: Instead of 'orch ls' use 'orch ps'

Signed-off-by: Varsha Rao <varao@redhat.com>
4 years agorbd: delay parsing of default kernel map options 36380/head
Jason Dillaman [Tue, 4 Aug 2020 18:51:29 +0000 (14:51 -0400)]
rbd: delay parsing of default kernel map options

It now connects to the cluster to retrieve any global config
overrides before processing the default map options.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
4 years agorbd: 'list' action should connect to cluster before querying config
Jason Dillaman [Tue, 4 Aug 2020 18:39:36 +0000 (14:39 -0400)]
rbd: 'list' action should connect to cluster before querying config

This will allow global config overrides for the max concurrent ops to
be applied prior to listing the images.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
4 years agorbd: drop querying of RBD default features
Jason Dillaman [Tue, 4 Aug 2020 18:28:21 +0000 (14:28 -0400)]
rbd: drop querying of RBD default features

These will be computed internally by librbd using any potential
global configuration overrides.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
4 years agorbd: delay determination of default pool name
Jason Dillaman [Tue, 4 Aug 2020 18:03:59 +0000 (14:03 -0400)]
rbd: delay determination of default pool name

Wait until after the CLI has connected to the cluster and therefore
has received all global configuration overrides.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
4 years agorbd: let librados finish initializing the global context
Jason Dillaman [Thu, 30 Jul 2020 17:12:20 +0000 (13:12 -0400)]
rbd: let librados finish initializing the global context

librados will retrieve a bootstrap config update from the MONs
after connecting and then finish initializing the context. In
general, non-primitive config options cannot be updated after
initialization is complete. This prevents the rbd CLI from
supporting non-primitive config overrides from the MON.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
4 years agomon/MonClient: apply MON config before finishing bootstrap
Jason Dillaman [Thu, 30 Jul 2020 17:00:22 +0000 (13:00 -0400)]
mon/MonClient: apply MON config before finishing bootstrap

In general, non-primitive config options cannot be updated after
the service threads have been marked as running. In librados, this
happens after the MonClient bootstrap process. However, the configs
received from the MON were applied in a separate thread which can
race with librados marking the service threads as running.

Now the received MON configs will be applied within the context of
the bootstrap function call instead of deferring to a separate
thread.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
4 years agoqa/suites/rados/test_orch_cli: Change mds name
Varsha Rao [Thu, 20 Aug 2020 14:28:57 +0000 (14:28 +0000)]
qa/suites/rados/test_orch_cli: Change mds name

As MDS name should not start with numeric digit.

Signed-off-by: Varsha Rao <varao@redhat.com>
4 years agoqa/tasks/nfs: Delete nfs cluster after exports are created
Varsha Rao [Thu, 20 Aug 2020 12:09:33 +0000 (12:09 +0000)]
qa/tasks/nfs: Delete nfs cluster after exports are created

Fixes: https://tracker.ceph.com/issues/47009
Signed-off-by: Varsha Rao <varao@redhat.com>
4 years agoMerge pull request #36477 from p-se/fix-downstream-alertmanager
Sebastian Wagner [Thu, 20 Aug 2020 12:24:43 +0000 (14:24 +0200)]
Merge pull request #36477 from p-se/fix-downstream-alertmanager

cephadm: Support custom alertmanager container

Reviewed-by: Sebastian Wagner <sebastian.wagner@suse.com>
4 years agocephadm: Allow users to use a custom dashboard ssl port 36716/head
Juan Miguel Olmo Martínez [Wed, 19 Aug 2020 10:11:28 +0000 (12:11 +0200)]
cephadm: Allow users to use a custom dashboard ssl port

This modification allows the user to create a new bootstrap cluster using a predefined SSl port for the dashboard.
If firewall is enabled, any new manager daemon deployed in new hosts will take care of open the required ports for all the services enabled in the manager.

Two new parameters for cephadm tool (aka binary or standalone):
Command bootstrap:
--ssl-dashboard-port SSL_DASHBOARD_PORT
                      Port number used to connect with dashboard using SSL
Command deploy:
--tcp-ports TCP_PORTS
                      List of tcp ports to open in the host firewall

Signed-off-by: Juan Miguel Olmo Martínez <jolmomar@redhat.com>
4 years agoMerge pull request #35756 from ShyamsundarR/tr-46163
Ramana Raja [Thu, 20 Aug 2020 11:23:11 +0000 (16:53 +0530)]
Merge pull request #35756 from ShyamsundarR/tr-46163

mgr/volumes: Use snapshot root directory attrs when creating clone root

Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Ramana Raja <rraja@redhat.com>
Reviewed-by: Kotresh Hiremath Ravishankar <khiremat@redhat.com>
4 years agoMerge pull request #36485 from adk3798/cephadm-44252
Sebastian Wagner [Thu, 20 Aug 2020 09:18:26 +0000 (11:18 +0200)]
Merge pull request #36485 from adk3798/cephadm-44252

mgr/cephadm: mgr or mds scale-down should prefer non-active daemons

Reviewed-by: Sebastian Wagner <sebastian.wagner@suse.com>
4 years agoqa/workunits/fs: add test for subvolume 36472/head
Yan, Zheng [Wed, 5 Aug 2020 15:31:03 +0000 (23:31 +0800)]
qa/workunits/fs: add test for subvolume

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
4 years agomds: don't move inode with nlink > 1 to global snaprealm if it's in subvolume
Yan, Zheng [Wed, 5 Aug 2020 09:31:57 +0000 (17:31 +0800)]
mds: don't move inode with nlink > 1 to global snaprealm if it's in subvolume

Fixes: https://tracker.ceph.com/issues/46074
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
4 years agomds: disallow hardlink across subvolume
Yan, Zheng [Wed, 5 Aug 2020 08:04:30 +0000 (16:04 +0800)]
mds: disallow hardlink across subvolume

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
4 years agomds: disallow across subvolume rename
Yan, Zheng [Wed, 5 Aug 2020 07:37:20 +0000 (15:37 +0800)]
mds: disallow across subvolume rename

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
4 years agomds: disallow creating snapshot on descendent directory of subvolume
Yan, Zheng [Wed, 5 Aug 2020 07:33:10 +0000 (15:33 +0800)]
mds: disallow creating snapshot on descendent directory of subvolume

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
4 years agomds: add vxattr that marks/clears subvolume flag
Yan, Zheng [Tue, 4 Aug 2020 13:26:26 +0000 (21:26 +0800)]
mds: add vxattr that marks/clears subvolume flag

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
4 years agoMerge pull request #36723 from tchaikov/wip-osd-writesame
Kefu Chai [Thu, 20 Aug 2020 03:03:32 +0000 (11:03 +0800)]
Merge pull request #36723 from tchaikov/wip-osd-writesame

crimson/osd: support writesame op

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
4 years agoMerge PR #36711 into master
Patrick Donnelly [Thu, 20 Aug 2020 02:10:54 +0000 (19:10 -0700)]
Merge PR #36711 into master

* refs/pull/36711/head:
Revert "vstart_runner: rotate logs between testcases"

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoRevert "vstart_runner: rotate logs between testcases" 36711/head
Ernesto Puerta [Wed, 19 Aug 2020 08:36:05 +0000 (10:36 +0200)]
Revert "vstart_runner: rotate logs between testcases"

Fixes: https://tracker.ceph.com/issues/47032
Signed-off-by: Ernesto Puerta <epuertat@redhat.com>
4 years agocrimson/osd: write log when sending "osd_failure" msg 36723/head
Kefu Chai [Wed, 19 Aug 2020 17:02:14 +0000 (01:02 +0800)]
crimson/osd: write log when sending "osd_failure" msg

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agocrimson/osd: support writesame op
Kefu Chai [Wed, 19 Aug 2020 16:59:37 +0000 (00:59 +0800)]
crimson/osd: support writesame op

it is tested by TestIoctx.test_aio_writesame. so let's implement it as
well.

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agomgr/dashboard: use SASS colors in TypeScript 36721/head
Alfonso Martínez [Wed, 19 Aug 2020 14:39:53 +0000 (16:39 +0200)]
mgr/dashboard: use SASS colors in TypeScript

NOTE: Enabled TS compiler option esModuleInterop for the unit tests to pass, which is also highly recommended:
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-7.html#support-for-import-d-from-cjs-from-commonjs-modules-with---esmoduleinterop
This implied updating lodash & momentjs imports (unneeded ones have been removed).

Fixes: https://tracker.ceph.com/issues/47036
Signed-off-by: Alfonso Martínez <almartin@redhat.com>
4 years agoMerge pull request #36719 from ofriedma/wip-reshard-logs
ofriedma [Wed, 19 Aug 2020 14:37:18 +0000 (17:37 +0300)]
Merge pull request #36719 from ofriedma/wip-reshard-logs

rgw: log resharding events at level 1 (formerly 20)

4 years agorgw: log resharding events at level 1 (formerly 20) 36719/head
Or Friedmann [Wed, 19 Aug 2020 13:07:19 +0000 (13:07 +0000)]
rgw: log resharding events at level 1 (formerly 20)
log resharding events at level 1 (formerly 20)
Signed-off-by: Or Friedmann <ofriedma@redhat.com>
4 years agoMerge pull request #36381 from dang/wip-dang-zipper-7
Daniel Gryniewicz [Wed, 19 Aug 2020 13:50:58 +0000 (09:50 -0400)]
Merge pull request #36381 from dang/wip-dang-zipper-7

Zipper 7 - chop list items

4 years agoMerge pull request #36703 from tchaikov/wip-crimson-errorator-omap
Kefu Chai [Wed, 19 Aug 2020 11:45:44 +0000 (19:45 +0800)]
Merge pull request #36703 from tchaikov/wip-crimson-errorator-omap

crimson/osd: errorator omap_get_values() ops

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
4 years agoMerge pull request #36702 from tchaikov/wip-doc-crimson
Kefu Chai [Wed, 19 Aug 2020 10:57:02 +0000 (18:57 +0800)]
Merge pull request #36702 from tchaikov/wip-doc-crimson

doc/dev/crimson: fix a typo

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
4 years agoMerge pull request #36699 from ceph/47008-table-footer-size
Lenz Grimmer [Wed, 19 Aug 2020 09:23:14 +0000 (11:23 +0200)]
Merge pull request #36699 from ceph/47008-table-footer-size

mgr/dashboard: restore table footer size

Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Laura Paduano <lpaduano@suse.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
4 years agoMerge pull request #36706 from Daniel-Pivonka/cephadm-rootuserdocfix
Sebastian Wagner [Wed, 19 Aug 2020 07:45:36 +0000 (09:45 +0200)]
Merge pull request #36706 from Daniel-Pivonka/cephadm-rootuserdocfix

doc/cephadm: remove ssh user limitation troubleshooting

Reviewed-by: Sebastian Wagner <sebastian.wagner@suse.com>
4 years agocrimson/osd: errorator omap_get_values() ops 36703/head
Kefu Chai [Tue, 18 Aug 2020 16:57:58 +0000 (00:57 +0800)]
crimson/osd: errorator omap_get_values() ops

* os/alienstore: do not panic if retval is less than 0. instead,
  translate ENOENT to enoent and other erros to input_output_error
* os/FuturizedStore: errorator omap_get_values() variants, so
  they return errorator::future<> instead of plain seastar::future<>
* os/seastore: update accordingly.
* osd/pg_backend: use enodata for OI without omap in it. better
  performance, and better readability this way.

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agoceph: correct additional paths added to sys.path 36710/head
Kefu Chai [Wed, 19 Aug 2020 04:54:41 +0000 (12:54 +0800)]
ceph: correct additional paths added to sys.path

pybind_path and pythonlib_path are already absolute paths, they are
prepared before being passed to `respawn_in_path()`. so let's drop
path components prepended to them.

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agodoc/mgr/diskprediction: add a configuration section 36680/head
Kefu Chai [Mon, 17 Aug 2020 08:36:09 +0000 (16:36 +0800)]
doc/mgr/diskprediction: add a configuration section

also link to monitoring, as diskprediction cannot function without
enabling monitoring.

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agopybind/mgr/devicehealth: drop the cloud model of disk prediction
Kefu Chai [Mon, 17 Aug 2020 07:44:44 +0000 (15:44 +0800)]
pybind/mgr/devicehealth: drop the cloud model of disk prediction

as the service offered by https://www.diskprophet.com/ is not accessible
anymore, the cloud model of disk prediction is not available. so let's
drop the code handling "cloud" model.

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agopybind/mgr/dashboard: drop test for diskprediction_cloud
Kefu Chai [Mon, 17 Aug 2020 07:42:46 +0000 (15:42 +0800)]
pybind/mgr/dashboard: drop test for diskprediction_cloud

this module is removed, so no need to test its settings

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agoqa/tasks/mgr: drop commented code
Kefu Chai [Mon, 17 Aug 2020 07:41:27 +0000 (15:41 +0800)]
qa/tasks/mgr: drop commented code

the test for diskprediction_cloud is never enabled, and the used
cloud-based service is not reachable anymore. let's just remove the dead
code.

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agoqa: stop referencing diskprediction_cloud
Kefu Chai [Mon, 17 Aug 2020 07:40:01 +0000 (15:40 +0800)]
qa: stop referencing diskprediction_cloud

diskprediction_cloud is removed from master, so no need to have it in
the exclude list anymore

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agodoc/mgr/diskprediction.rst: remove references to diskprediction_cloud
Kefu Chai [Mon, 17 Aug 2020 07:37:34 +0000 (15:37 +0800)]
doc/mgr/diskprediction.rst: remove references to diskprediction_cloud

since https://www.diskprophet.com is not accessible anymore, and this
feature is not available at the time of writing. let's drop the related
document.

Fixes: https://tracker.ceph.com/issues/46863
Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agopybind/mgr/diskprediction_cloud: remove mgr-diskprediction-cloud
Kefu Chai [Mon, 17 Aug 2020 07:35:25 +0000 (15:35 +0800)]
pybind/mgr/diskprediction_cloud: remove mgr-diskprediction-cloud

the service offered by https://www.diskprophet.com is not accessible
anymore. before we have a replacement of it, let's drop
mgr-diskprediction-cloud. and remove its packaging.

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge pull request #36531 from tchaikov/wip-alpine
Kefu Chai [Wed, 19 Aug 2020 03:05:59 +0000 (11:05 +0800)]
Merge pull request #36531 from tchaikov/wip-alpine

alpine: remove stale Alpine build recipe

Reviewed-by: Neha Ojha <nojha@redhat.com>
4 years agoMerge pull request #36688 from coreycb/master
Jason Dillaman [Tue, 18 Aug 2020 21:23:25 +0000 (17:23 -0400)]
Merge pull request #36688 from coreycb/master

librbd: Align rbd_write_zeroes declarations

Reviewed-by: Jason Dillaman <dillaman@redhat.com>
4 years agoqa/workunits/rbd: retrieve peer uuid when attempting to delete duplicate 36707/head
Jason Dillaman [Tue, 18 Aug 2020 21:18:25 +0000 (17:18 -0400)]
qa/workunits/rbd: retrieve peer uuid when attempting to delete duplicate

Previously, the peer uuid variable was empty which resulted in the failure
to remove the duplicate peer.

Fixes: https://tracker.ceph.com/issues/47007
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
4 years agodoc/cephadm: remove ssh user limitation troubleshooting 36706/head
Daniel-Pivonka [Tue, 18 Aug 2020 20:46:35 +0000 (16:46 -0400)]
doc/cephadm: remove ssh user limitation troubleshooting

Signed-off-by: Daniel-Pivonka <dpivonka@redhat.com>
4 years agorbd: re-use helper method for adding mirror peer site-name optional
Jason Dillaman [Tue, 18 Aug 2020 20:29:20 +0000 (16:29 -0400)]
rbd: re-use helper method for adding mirror peer site-name optional

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
4 years agomgr/cephadm: rgw update period after realm, zone creation 36496/head
Daniel-Pivonka [Wed, 5 Aug 2020 21:50:12 +0000 (17:50 -0400)]
mgr/cephadm: rgw update period after realm, zone creation

Signed-off-by: Daniel-Pivonka <dpivonka@redhat.com>
4 years agoMerge PR #35824 into master
Patrick Donnelly [Tue, 18 Aug 2020 16:27:49 +0000 (09:27 -0700)]
Merge PR #35824 into master

* refs/pull/35824/head:
vstart_runner.py: rotate logs between testcases
vstart.sh: write logrotate conf and rotate logs
vstart_runner: run one test method at a time...

Reviewed-by: Sidharth Anupkrishnan <sanupkri@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agomessages,mds: Fix decoding of enum types on big-endian systems 36697/head
Ulrich Weigand [Tue, 18 Aug 2020 07:51:22 +0000 (09:51 +0200)]
messages,mds: Fix decoding of enum types on big-endian systems

When a struct member that has enum type needs to be encoded or
decoded, we need to use an explicit integer type, since there
are no encode routines for the enum type.  (This is probably
to avoid introducing dependencies on implementation-defined
choices by the compiler to use a particular underlying type.)

This leads to code sequences along the lines of:
  encode((int32_t)state, bl);
and
  decode((int32_t&)(state), bl);

The encode line is actually fine, but the decode line is
incorrect on big-endian systems if the underlying type of
the enum differs from the explicitly chosen integer type.

This is because this performs in effect a pointer cast,
and will write the decoded int32_t value into the memory
backing the "state" member variable.  If the sizes differ,
the value is written into the wrong bytes of "state" on
big-endian systems.

This patch fixes the problem by decoding into an intermediate
variable of the integer type first, and then casting the result
while assigning to the struct member of enum type.

This bug showed up initially as invalid health-status values
causing Ceph daemon aborts on s390x.  I've tried to find and
fix all other instances of the same enum decode pattern as well.

Fixes: https://tracker.ceph.com/issues/47015
Signed-off-by: Ulrich Weigand <ulrich.weigand@de.ibm.com>
4 years agodoc/dev/crimson: fix a typo 36702/head
Kefu Chai [Tue, 18 Aug 2020 16:13:04 +0000 (00:13 +0800)]
doc/dev/crimson: fix a typo

Signed-off-by: Kefu Chai <kchai@redhat.com>