]> git.apps.os.sepia.ceph.com Git - ceph.git/log
ceph.git
4 years agoMerge PR #39354 into master
Patrick Donnelly [Thu, 4 Mar 2021 21:28:59 +0000 (13:28 -0800)]
Merge PR #39354 into master

* refs/pull/39354/head:
mds/client: create cephfs errno aliases

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge PR #38684 into master
Patrick Donnelly [Thu, 4 Mar 2021 21:27:45 +0000 (13:27 -0800)]
Merge PR #38684 into master

* refs/pull/38684/head:
qa: add _check_scrub_status helper to simplify the code
qa: add run_scrub helper in filesystem class
qa: add get_scrub_status helper in filesystem class
qa: wait the scrub task to complete
qa: remove passed_validation check for test_damage
qa: move wait_until_scrub_complete helper to filesystem class
mds: simplify the C_MDS_EnqueueScrub finish code

Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge pull request #39835 from yuriw/wip-yuriw-crontab-master
Yuri Weinstein [Thu, 4 Mar 2021 21:23:20 +0000 (13:23 -0800)]
Merge pull request #39835 from yuriw/wip-yuriw-crontab-master

qa/tests: moved nautilus and octopus tests to run on gibba and -p 70 …

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
4 years agoMerge PR #39832 into master
Sage Weil [Thu, 4 Mar 2021 21:08:14 +0000 (16:08 -0500)]
Merge PR #39832 into master

* refs/pull/39832/head:
mgr/DaemonServer: osd ok-to-stop: return json when there are unknown PGs

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Sebastian Wagner <swagner@suse.com>
4 years agoMerge PR #39726 into master
Sage Weil [Thu, 4 Mar 2021 19:31:57 +0000 (14:31 -0500)]
Merge PR #39726 into master

* refs/pull/39726/head:
mgr/cephadm: document ok_to_stop output argument for clarity
mgr/DaemonServer: make warning language a bit friendlier
mgr/cephadm/upgrade: improve language a bit
mgr/cephadm/upgrade: restart multiple osds at once
mgr/cephadm: gather other osds that are safe to stop
mgr/cephadm: optional pass 'known' through to ok_to_stop
mgr/cephadm/upgrade: log start/stop/pause/resume

Reviewed-by: Sebastian Wagner <swagner@suse.com>
4 years agoMerge pull request #38741 from BryceCao/wip-standardize-request
Harish Munjulur [Thu, 4 Mar 2021 19:07:17 +0000 (11:07 -0800)]
Merge pull request #38741 from BryceCao/wip-standardize-request

rgw : add the check for bucket handler if http args exist object_exclusive sub_resource

4 years agoMerge pull request #39790 from yuvalif/wip-yuval-fix-49557
Harish Munjulur [Thu, 4 Mar 2021 19:05:30 +0000 (11:05 -0800)]
Merge pull request #39790 from yuvalif/wip-yuval-fix-49557

rgw/notification: fix persistent notification hang when ack-levl=none

4 years agoMerge pull request #39782 from yuvalif/wip-yuval-fix-49552
Harish Munjulur [Thu, 4 Mar 2021 19:05:14 +0000 (11:05 -0800)]
Merge pull request #39782 from yuvalif/wip-yuval-fix-49552

rgw/notification: fixing the "persistent=false" flag

4 years agoMerge pull request #39515 from CloudFerro/gc_max_objs
Harish Munjulur [Thu, 4 Mar 2021 19:03:56 +0000 (11:03 -0800)]
Merge pull request #39515 from CloudFerro/gc_max_objs

rgw: limit rgw_gc_max_objs to RGW_SHARDS_PRIME_1

4 years agoMerge pull request #39380 from leseb/rgw-svc-map-name
Harish Munjulur [Thu, 4 Mar 2021 19:03:19 +0000 (11:03 -0800)]
Merge pull request #39380 from leseb/rgw-svc-map-name

rgw: register daemon in service map with more details

4 years agoqa/tests: moved nautilus and octopus tests to run on gibba and -p 70 for smoke tests 39835/head
Yuri Weinstein [Thu, 4 Mar 2021 16:40:40 +0000 (08:40 -0800)]
qa/tests: moved nautilus and octopus tests to run on gibba and -p 70 for smoke tests

Signed-off-by: Yuri Weinstein <yweinste@redhat.com>
4 years agomgr/cephadm: document ok_to_stop output argument for clarity 39726/head
Sage Weil [Thu, 4 Mar 2021 17:01:28 +0000 (12:01 -0500)]
mgr/cephadm: document ok_to_stop output argument for clarity

Signed-off-by: Sage Weil <sage@newdream.net>
4 years agoMerge PR #39823 into master
Sage Weil [Thu, 4 Mar 2021 16:41:28 +0000 (11:41 -0500)]
Merge PR #39823 into master

* refs/pull/39823/head:
qa/tasks/cephadm: remove mirror code

Reviewed-by: Sebastian Wagner <swagner@suse.com>
4 years agoMerge pull request #39822 from mgfritch/cephadm-custom-container-sample
Sebastian Wagner [Thu, 4 Mar 2021 15:23:26 +0000 (16:23 +0100)]
Merge pull request #39822 from mgfritch/cephadm-custom-container-sample

cephadm: split custom container args into argv

Reviewed-by: Sebastian Wagner <sebastian.wagner@suse.com>
4 years agoMerge pull request #39699 from mgfritch/cephadm-bin-flake8-quotes
Sebastian Wagner [Thu, 4 Mar 2021 14:59:07 +0000 (15:59 +0100)]
Merge pull request #39699 from mgfritch/cephadm-bin-flake8-quotes

cephadm: introduce flake8 and flake8-quotes

Reviewed-by: Juan Miguel Olmo Martínez <jolmomar@redhat.com>
4 years agomgr/DaemonServer: osd ok-to-stop: return json when there are unknown PGs 39832/head
Sage Weil [Thu, 4 Mar 2021 13:35:24 +0000 (08:35 -0500)]
mgr/DaemonServer: osd ok-to-stop: return json when there are unknown PGs

In 791952cc01201010f298033003ba52374cc0159f we switched to return JSON
both on success and fail to describe which PGs are affected or are blocking
the ability to stop/restart OSDs.  Do the same for the case where
some PG states are unknown (i.e., just after a mgr restart) so that
the cephadm upgrade process can unconditionally expect a JSON result.

Signed-off-by: Sage Weil <sage@newdream.net>
4 years agoqa/tasks/cephadm: remove mirror code 39823/head
Sage Weil [Wed, 3 Mar 2021 18:35:07 +0000 (12:35 -0600)]
qa/tasks/cephadm: remove mirror code

This is being done from ansible now.  Also, it breaks when
the conf file has unqualified-search-registries but not 'registry'
entries.

Signed-off-by: Sage Weil <sage@newdream.net>
4 years agoMerge pull request #39831 from xxhdx1985126/wip-crimson-io-sequencer-bug
Kefu Chai [Thu, 4 Mar 2021 13:10:36 +0000 (21:10 +0800)]
Merge pull request #39831 from xxhdx1985126/wip-crimson-io-sequencer-bug

crimson/osd: do not pass lvalue of the lambda to seastar::futurize_invoke

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge pull request #39656 from pritha-srivastava/wip-rgw-sts-user-fix
Matt Benjamin [Thu, 4 Mar 2021 12:57:58 +0000 (07:57 -0500)]
Merge pull request #39656 from pritha-srivastava/wip-rgw-sts-user-fix

rgw/sts: fix for encoding/decoding user namespace

4 years agoMerge pull request #39452 from rhcs-dashboard/wip-49283-master
Ernesto Puerta [Thu, 4 Mar 2021 12:53:06 +0000 (13:53 +0100)]
Merge pull request #39452 from rhcs-dashboard/wip-49283-master

mgr/dashboard: report mgr fsid

Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
4 years agocrimson/osd: do not pass lvalue of the lambda to seastar::futurize_invoke 39831/head
Xuehan Xu [Thu, 4 Mar 2021 12:29:57 +0000 (20:29 +0800)]
crimson/osd: do not pass lvalue of the lambda to seastar::futurize_invoke

Signed-off-by: Xuehan Xu <xxhdx1985126@gmail.com>
4 years agoMerge pull request #39815 from sebastian-philipp/github-cephadm-minor
Kefu Chai [Thu, 4 Mar 2021 11:06:08 +0000 (19:06 +0800)]
Merge pull request #39815 from sebastian-philipp/github-cephadm-minor

.github: Minor cephadm changes

Reviewed-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge pull request #39827 from xxhdx1985126/wip-crimson-minor-bugs
Kefu Chai [Thu, 4 Mar 2021 10:33:25 +0000 (18:33 +0800)]
Merge pull request #39827 from xxhdx1985126/wip-crimson-minor-bugs

crimson/osd: capture error_code by value in PG::handle_failed_op

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Reviewed-by: Xuehan Xu <xxhdx1985126@gmail.com>
4 years agoMerge pull request #39812 from rzarzynski/wip-crimson-admin-config-help
Kefu Chai [Thu, 4 Mar 2021 09:45:16 +0000 (17:45 +0800)]
Merge pull request #39812 from rzarzynski/wip-crimson-admin-config-help

crimson/admin: add support for 'config help'

Reviewed-by: Kefu Chai <kchai@redhat.com>
4 years agocrimson/osd: capture error_code by value in PG::handle_failed_op 39827/head
Xuehan Xu [Thu, 4 Mar 2021 09:20:28 +0000 (17:20 +0800)]
crimson/osd: capture error_code by value in PG::handle_failed_op

Signed-off-by: Xuehan Xu <xxhdx1985126@gmail.com>
4 years agoMerge pull request #39801 from rhcs-dashboard/49574-fix-pyjwt
Ernesto Puerta [Thu, 4 Mar 2021 08:53:42 +0000 (09:53 +0100)]
Merge pull request #39801 from rhcs-dashboard/49574-fix-pyjwt

mgr/dashboard: fix issues related with PyJWT versions >=2.0.0

Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
4 years agoMerge pull request #39755 from smithfarm/wip-49550
Nathan Cutler [Thu, 4 Mar 2021 08:29:47 +0000 (09:29 +0100)]
Merge pull request #39755 from smithfarm/wip-49550

rpm: use PMDK system libraries on SUSE

Reviewed-by: Jason Dillaman <dillaman@redhat.com>
Reviewed-by: Mykola Golub <mgolub@suse.com>
4 years agoMerge PR #39799 into master
Patrick Donnelly [Thu, 4 Mar 2021 04:05:22 +0000 (20:05 -0800)]
Merge PR #39799 into master

* refs/pull/39799/head:
doc: update affiliation for myoungwon

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
4 years agodoc: update affiliation for myoungwon 39799/head
myoungwon oh [Wed, 3 Mar 2021 06:05:05 +0000 (15:05 +0900)]
doc: update affiliation for myoungwon

Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
4 years agocephadm: split custom container args into argv 39822/head
Michael Fritch [Wed, 3 Mar 2021 20:00:32 +0000 (13:00 -0700)]
cephadm: split custom container args into argv

introduced by usage of `shlex` in 3ea514c5

Fixes: https://tracker.ceph.com/issues/49590
Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agorpm: use PMDK system libraries on SUSE 39755/head
Nathan Cutler [Mon, 1 Mar 2021 11:07:29 +0000 (12:07 +0100)]
rpm: use PMDK system libraries on SUSE

As of a49d1dbb32e2436ff2836a85b2fa84418f0a5fff, when the rbd_rwl_cache and
rbd_ssd_cache bconds are enabled and WITH_SYSTEM_PMDK is disabled (as it is by
default), the RPM build attempts to

    git clone https://github.com/ceph/pmdk.git

but of course that won't work in the OBS, where the build workers have no
Internet connectivity.

Fortunately, the openSUSE/SLE versions targeted by Ceph master and pacific ship
the necessary PMDK libraries as RPM packages.

Fixes: a49d1dbb32e2436ff2836a85b2fa84418f0a5fff
Fixes: https://tracker.ceph.com/issues/49550
Signed-off-by: Nathan Cutler <ncutler@suse.com>
4 years agoMerge pull request #39765 from smithfarm/wip-ncpus-for-obs
Nathan Cutler [Wed, 3 Mar 2021 19:44:20 +0000 (20:44 +0100)]
Merge pull request #39765 from smithfarm/wip-ncpus-for-obs

rpm: set build parallelism from memory on SUSE

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: David Galloway <dgallowa@redhat.com>
4 years agoMerge PR #39682 into master
Patrick Donnelly [Wed, 3 Mar 2021 18:05:35 +0000 (10:05 -0800)]
Merge PR #39682 into master

* refs/pull/39682/head:
vstart_runner: remove duplicate methods from LocalCephManager

Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
4 years agoMerge PR #39780 into master
Patrick Donnelly [Wed, 3 Mar 2021 18:04:23 +0000 (10:04 -0800)]
Merge PR #39780 into master

* refs/pull/39780/head:
qa/vstart_runner: dont log "not Ceph bin" msg too often

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Xiubo Li <xiubli@redhat.com>
4 years agoMerge PR #39681 into master
Patrick Donnelly [Wed, 3 Mar 2021 18:02:41 +0000 (10:02 -0800)]
Merge PR #39681 into master

* refs/pull/39681/head:
vstart_runner: define path to ceph binary and use it

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Xiubo Li <xiubli@redhat.com>
4 years agoMerge pull request #39382 from gauravsitlani/master
Casey Bodley [Wed, 3 Mar 2021 17:13:16 +0000 (12:13 -0500)]
Merge pull request #39382 from gauravsitlani/master

rgw: added missing documentation on "pubsub" in rgw_enable_apis

Reviewed-by: Yuval Lifshitz <ylifshit@redhat.com>
Reviewed-by: Casey Bodley <cbodley@redhat.com>
4 years agoMerge pull request #39762 from smithfarm/wip-support-leap-15.3
Nathan Cutler [Wed, 3 Mar 2021 17:05:07 +0000 (18:05 +0100)]
Merge pull request #39762 from smithfarm/wip-support-leap-15.3

rpm/luarocks: simplify conditional and support Leap 15.3

Reviewed-by: Yuval Lifshitz <ylifshit@redhat.com>
Reviewed-by: Kyr Shatskyy <kyrylo.shatskyy@suse.de>
4 years agocephadm: fixup `alrady` -> `already` 39699/head
Michael Fritch [Thu, 25 Feb 2021 15:12:46 +0000 (08:12 -0700)]
cephadm: fixup `alrady` -> `already`

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: Change outer quotes to avoid escaping inner quotes (Q003)
Michael Fritch [Thu, 25 Feb 2021 15:12:37 +0000 (08:12 -0700)]
cephadm: Change outer quotes to avoid escaping inner quotes (Q003)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: Remove bad quotes from multiline string (Q001)
Michael Fritch [Thu, 25 Feb 2021 15:12:24 +0000 (08:12 -0700)]
cephadm: Remove bad quotes from multiline string (Q001)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: Remove bad quotes (Q000)
Michael Fritch [Wed, 24 Feb 2021 20:50:33 +0000 (13:50 -0700)]
cephadm: Remove bad quotes (Q000)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: introduce flake8-quotes
Michael Fritch [Wed, 24 Feb 2021 20:48:37 +0000 (13:48 -0700)]
cephadm: introduce flake8-quotes

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: line break after binary operator (W504)
Michael Fritch [Fri, 19 Feb 2021 15:09:49 +0000 (08:09 -0700)]
cephadm: line break after binary operator (W504)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: blank line contains whitespace (W293)
Michael Fritch [Fri, 19 Feb 2021 15:09:40 +0000 (08:09 -0700)]
cephadm: blank line contains whitespace (W293)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: trailing whitespace (W291)
Michael Fritch [Fri, 19 Feb 2021 15:09:31 +0000 (08:09 -0700)]
cephadm: trailing whitespace (W291)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: local variable 'e' is assigned to but never used (F841)
Michael Fritch [Fri, 19 Feb 2021 15:09:22 +0000 (08:09 -0700)]
cephadm: local variable 'e' is assigned to but never used (F841)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: 'select' imported but unused (F401)
Michael Fritch [Fri, 19 Feb 2021 15:09:08 +0000 (08:09 -0700)]
cephadm: 'select' imported but unused (F401)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: ambiguous variable name 'l' (E741)
Michael Fritch [Thu, 18 Feb 2021 22:30:02 +0000 (15:30 -0700)]
cephadm: ambiguous variable name 'l' (E741)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: do not use bare 'except' (E722)
Michael Fritch [Thu, 18 Feb 2021 22:25:27 +0000 (15:25 -0700)]
cephadm: do not use bare 'except' (E722)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: statement ends with a semicolon (E703)
Michael Fritch [Thu, 18 Feb 2021 22:24:03 +0000 (15:24 -0700)]
cephadm: statement ends with a semicolon (E703)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: module level import not at top of file (E402)
Michael Fritch [Thu, 18 Feb 2021 22:19:30 +0000 (15:19 -0700)]
cephadm: module level import not at top of file (E402)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: expected 1 blank line before a nested definition (E306)
Michael Fritch [Thu, 18 Feb 2021 22:15:48 +0000 (15:15 -0700)]
cephadm: expected 1 blank line before a nested definition (E306)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: expected 2 blank lines after end of function or class (E305)
Michael Fritch [Thu, 18 Feb 2021 22:13:57 +0000 (15:13 -0700)]
cephadm: expected 2 blank lines after end of function or class (E305)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: too many blank lines (E303)
Michael Fritch [Thu, 18 Feb 2021 22:12:50 +0000 (15:12 -0700)]
cephadm: too many blank lines (E303)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: expected 2 blank lines, found 1 (E302)
Michael Fritch [Thu, 18 Feb 2021 22:10:41 +0000 (15:10 -0700)]
cephadm: expected 2 blank lines, found 1 (E302)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: expected 1 blank line, found 0 (E301)
Michael Fritch [Thu, 18 Feb 2021 22:04:42 +0000 (15:04 -0700)]
cephadm: expected 1 blank line, found 0 (E301)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: too many leading '#' for block comment (E266)
Michael Fritch [Thu, 18 Feb 2021 22:02:31 +0000 (15:02 -0700)]
cephadm: too many leading '#' for block comment (E266)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: block comment should start with '# ' (E265)
Michael Fritch [Wed, 17 Feb 2021 15:23:44 +0000 (08:23 -0700)]
cephadm: block comment should start with '# ' (E265)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: at least two spaces before inline comment (E261)
Michael Fritch [Wed, 17 Feb 2021 15:18:31 +0000 (08:18 -0700)]
cephadm: at least two spaces before inline comment (E261)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: unexpected spaces around keyword / parameter equals (E251)
Michael Fritch [Wed, 17 Feb 2021 04:10:36 +0000 (21:10 -0700)]
cephadm: unexpected spaces around keyword / parameter equals (E251)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: multiple spaces after ',' (E241)
Michael Fritch [Wed, 17 Feb 2021 04:08:39 +0000 (21:08 -0700)]
cephadm: multiple spaces after ',' (E241)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: missing whitespace after ':' (E231)
Michael Fritch [Wed, 17 Feb 2021 04:05:04 +0000 (21:05 -0700)]
cephadm: missing whitespace after ':' (E231)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: missing whitespace around arithmetic operator (E226)
Michael Fritch [Wed, 17 Feb 2021 03:56:03 +0000 (20:56 -0700)]
cephadm: missing whitespace around arithmetic operator (E226)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: missing whitespace around operator (E225)
Michael Fritch [Wed, 17 Feb 2021 03:53:24 +0000 (20:53 -0700)]
cephadm: missing whitespace around operator (E225)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: whitespace before ':' (E203)
Michael Fritch [Wed, 17 Feb 2021 03:50:10 +0000 (20:50 -0700)]
cephadm: whitespace before ':' (E203)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: whitespace after '{' (E201)
Michael Fritch [Wed, 17 Feb 2021 03:47:39 +0000 (20:47 -0700)]
cephadm: whitespace after '{' (E201)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: continuation line unaligned for hanging indent (E131)
Michael Fritch [Wed, 17 Feb 2021 03:46:19 +0000 (20:46 -0700)]
cephadm: continuation line unaligned for hanging indent (E131)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: continuation line under-indented for visual indent (E128)
Michael Fritch [Wed, 17 Feb 2021 03:38:37 +0000 (20:38 -0700)]
cephadm: continuation line under-indented for visual indent (E128)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: continuation line over-indented for visual indent (E127)
Michael Fritch [Wed, 17 Feb 2021 03:18:50 +0000 (20:18 -0700)]
cephadm: continuation line over-indented for visual indent (E127)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: continuation line over-indented for hanging indent (E126)
Michael Fritch [Fri, 12 Feb 2021 16:26:26 +0000 (09:26 -0700)]
cephadm: continuation line over-indented for hanging indent (E126)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: continuation line with same indent as next logical line (E125)
Michael Fritch [Tue, 2 Mar 2021 20:50:10 +0000 (13:50 -0700)]
cephadm: continuation line with same indent as next logical line (E125)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: closing bracket does not match visual indentation (E124)
Michael Fritch [Fri, 12 Feb 2021 16:20:49 +0000 (09:20 -0700)]
cephadm: closing bracket does not match visual indentation (E124)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: ... does not match indentation of opening bracket's line (E123)
Michael Fritch [Fri, 12 Feb 2021 16:11:38 +0000 (09:11 -0700)]
cephadm: ... does not match indentation of opening bracket's line (E123)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: continuation line missing indentation or outdented (E122)
Michael Fritch [Fri, 12 Feb 2021 16:09:47 +0000 (09:09 -0700)]
cephadm: continuation line missing indentation or outdented (E122)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: continuation line under-indented for hanging indent (E121)
Michael Fritch [Fri, 12 Feb 2021 16:03:30 +0000 (09:03 -0700)]
cephadm: continuation line under-indented for hanging indent (E121)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: over-indented (E117)
Michael Fritch [Fri, 12 Feb 2021 16:02:17 +0000 (09:02 -0700)]
cephadm: over-indented (E117)

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agocephadm: introduce flake8
Michael Fritch [Fri, 12 Feb 2021 15:59:07 +0000 (08:59 -0700)]
cephadm: introduce flake8

Signed-off-by: Michael Fritch <mfritch@suse.com>
4 years agoMerge pull request #39789 from zdover23/wip-doc-cephadm-convert--bash-shells-2021...
Sebastian Wagner [Wed, 3 Mar 2021 15:45:00 +0000 (16:45 +0100)]
Merge pull request #39789 from zdover23/wip-doc-cephadm-convert--bash-shells-2021-Mar-03

doc/cephadm: add prompts to adoption.rst

Reviewed-by: Sebastian Wagner <sebastian.wagner@suse.com>
4 years agoMerge pull request #39794 from cbodley/wip-qa-rgw-valgrind-supp-librados-delete
Casey Bodley [Wed, 3 Mar 2021 15:38:36 +0000 (10:38 -0500)]
Merge pull request #39794 from cbodley/wip-qa-rgw-valgrind-supp-librados-delete

valgrind: suppress operator delete[] for tcmalloc

Reviewed-by: Daniel Gryniewicz <dang@redhat.com>
4 years ago.github: Minor cephadm changes 39815/head
Sebastian Wagner [Wed, 3 Mar 2021 15:34:01 +0000 (16:34 +0100)]
.github: Minor cephadm changes

Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
4 years agoMerge pull request #39558 from sebastian-philipp/doc-rados-rm-toctree-cephadm
Sebastian Wagner [Wed, 3 Mar 2021 15:17:51 +0000 (16:17 +0100)]
Merge pull request #39558 from sebastian-philipp/doc-rados-rm-toctree-cephadm

doc/rados: avoid including cephadm into the toctree

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
Reviewed-by: Zac Dover <zac.dover@gmail.com>
4 years agorpm: drop old SUSE-specific OOM fix 39765/head
Nathan Cutler [Wed, 3 Mar 2021 12:57:31 +0000 (13:57 +0100)]
rpm: drop old SUSE-specific OOM fix

This fix was needed some years ago, but no longer.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
4 years agorpm: limit build jobs by system memory on SUSE
Nathan Cutler [Tue, 2 Mar 2021 16:00:53 +0000 (17:00 +0100)]
rpm: limit build jobs by system memory on SUSE

43b441f9a3bc907c17d52385251001ffcd5d3ff9 removed a bunch of code which the SUSE
builds were relying on to avoid OOM. This commit brings back that code in
a much-streamlined form: the SUSE-specific %limit_build macro.

This also has the advantage of not breaking the build on older RPMs which only
know about %_smp_mflags, and not the newer %_smp_build_ncpus etc. macros.

Fixes: 43b441f9a3bc907c17d52385251001ffcd5d3ff9
Fixes: https://tracker.ceph.com/issues/49556
Signed-off-by: Nathan Cutler <ncutler@suse.com>
4 years agocrimson/admin: add support for 'config help' 39812/head
Radoslaw Zarzynski [Wed, 3 Mar 2021 14:21:28 +0000 (14:21 +0000)]
crimson/admin: add support for 'config help'

This commit solves a problem unveiled in teuthology testing
where `ceph --admin-daemon ... config help` were constantly
returning `EINVAL`:

```
2021-03-03T02:43:01.566 DEBUG:tasks.admin_socket:Testing config help with config {'branch': 'master'}
2021-03-03T02:43:01.567 DEBUG:teuthology.orchestra.run.smithi083:> sudo adjust-ulimits ceph-coverage /home/ubuntu/cephtest/archive/coverage ceph --admin-daemon /var/run/ceph/ceph-osd.0.asok config help
2021-03-03T02:43:01.711 INFO:teuthology.orchestra.run.smithi083.stderr:no valid command found; 3 closest matches:
2021-03-03T02:43:01.712 INFO:teuthology.orchestra.run.smithi083.stderr:config get <var>
2021-03-03T02:43:01.712 INFO:teuthology.orchestra.run.smithi083.stderr:config set <var> <val>...
2021-03-03T02:43:01.712 INFO:teuthology.orchestra.run.smithi083.stderr:config show
2021-03-03T02:43:01.713 INFO:teuthology.orchestra.run.smithi083.stderr:admin_socket: invalid command
2021-03-03T02:43:01.714 DEBUG:teuthology.orchestra.run:got remote process result: 22
2021-03-03T02:43:01.714 INFO:tasks.admin_socket:ceph cli returned an error, command not registered yet?
2021-03-03T02:43:01.715 INFO:tasks.admin_socket:sleeping and retrying ...
```

The root cause is no support for `config help` in crimson.
As shown below this command hasn't been registered at all:

```
DEBUG 2021-03-03 02:42:48,242 [shard 0] osd - start: asok socket path=/var/run/ceph/ceph-osd.0.asok
INFO  2021-03-03 02:42:48,242 [shard 0] osd - register_command(): mark_unfound_lost)
INFO  2021-03-03 02:42:48,242 [shard 0] osd - register_command(): query)
INFO  2021-03-03 02:42:48,242 [shard 0] osd - register_command(): perf dump_seastar)
INFO  2021-03-03 02:42:48,242 [shard 0] osd - register_command(): dump_pgstate_history)
INFO  2021-03-03 02:42:48,242 [shard 0] osd - register_command(): flush_pg_stats)
INFO  2021-03-03 02:42:48,243 [shard 0] osd - register_command(): send_beacon)
INFO  2021-03-03 02:42:48,243 [shard 0] osd - register_command(): status)
INFO  2021-03-03 02:42:48,243 [shard 0] osd - register_command(): injectargs)
INFO  2021-03-03 02:42:48,243 [shard 0] osd - register_command(): config show)
INFO  2021-03-03 02:42:48,243 [shard 0] osd - register_command(): config set)
INFO  2021-03-03 02:42:48,243 [shard 0] osd - register_command(): config get)
INFO  2021-03-03 02:42:48,243 [shard 0] osd - register_command(): get_command_descriptions)
INFO  2021-03-03 02:42:48,243 [shard 0] osd - register_command(): help)
INFO  2021-03-03 02:42:48,243 [shard 0] osd - register_command(): git_version)
INFO  2021-03-03 02:42:48,243 [shard 0] osd - register_command(): version)
```

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
4 years agomgr/dashboard: fix issues related with PyJWT versions >=2.0.0 39801/head
Alfonso Martínez [Wed, 3 Mar 2021 14:36:06 +0000 (15:36 +0100)]
mgr/dashboard: fix issues related with PyJWT versions >=2.0.0

Fixes: https://tracker.ceph.com/issues/49574
Signed-off-by: Alfonso Martínez <almartin@redhat.com>
4 years agoMerge pull request #39399 from dang/wip-dang-zipper-10
Daniel Gryniewicz [Wed, 3 Mar 2021 13:56:04 +0000 (08:56 -0500)]
Merge pull request #39399 from dang/wip-dang-zipper-10

RGW - Zipper 10: The Great Zippening

4 years agoMerge PR #38909 into master
Sage Weil [Wed, 3 Mar 2021 13:38:58 +0000 (08:38 -0500)]
Merge PR #38909 into master

* refs/pull/38909/head:
PendingReleaseNotes: document option osd_fast_shutdown_notify_mon
osd: add osd_fast_shutdown_notify_mon option (default false)

Reviewed-by: Neha Ojha <nojha@redhat.com>
4 years agoMerge PR #39654 into master
Sage Weil [Wed, 3 Mar 2021 13:38:20 +0000 (08:38 -0500)]
Merge PR #39654 into master

* refs/pull/39654/head:
common/options: drop ms_async_max_op_threads
msg/async: drop Stack::num_workers
msg/async: s/num_workers/workers.size()/
msg/async: use range-based loop in NetworkStack
msg/async: do not pass worker id to Stack::spawn_worker()
async/Stack: pass Worker* to NetworkStack::add_thread()
async/rdma: do not reference worker id in RDMAStack::spawn_worker()
async/dpdk: do not use worker id when creating worker
async/PosixStack: do not reference worker id in ctor
async/rdma: initialize worker in RDMAStack::create_worker()
async/rdma: move RDMAStack::create_worker() to .cc

Reviewed-by: luo runbing <luo.runbing@zte.com.cn>
Reviewed-by: Haomai Wang <haomai@xsky.com>
4 years agoMerge PR #39665 into master
Sage Weil [Wed, 3 Mar 2021 13:35:51 +0000 (08:35 -0500)]
Merge PR #39665 into master

* refs/pull/39665/head:
qa/rados/dashbord use "random" objectore

Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Laura Paduano <lpaduano@suse.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
4 years agoMerge PR #39689 into master
Sage Weil [Wed, 3 Mar 2021 13:33:28 +0000 (08:33 -0500)]
Merge PR #39689 into master

* refs/pull/39689/head:
src/global/signal_handler.h: fix preprocessor logic for alpine

Reviewed-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge PR #39750 into master
Sage Weil [Wed, 3 Mar 2021 13:32:41 +0000 (08:32 -0500)]
Merge PR #39750 into master

* refs/pull/39750/head:
mgr: create client messenger using ms_public_type

Reviewed-by: Kefu Chai <kchai@redhat.com>
4 years agoMerge pull request #39770 from liewegas/bug-46745
Sebastian Wagner [Wed, 3 Mar 2021 09:51:47 +0000 (10:51 +0100)]
Merge pull request #39770 from liewegas/bug-46745

cephadm: add docker.service dependency in systemd units

Reviewed-by: Sebastian Wagner <sebastian.wagner@suse.com>
4 years agoMerge pull request #39795 from dillaman/wip-migration-import-docs
Mykola Golub [Wed, 3 Mar 2021 08:45:35 +0000 (10:45 +0200)]
Merge pull request #39795 from dillaman/wip-migration-import-docs

doc/rbd: clarify QCOW2 migration support

Reviewed-by: Mykola Golub <mgolub@suse.com>
4 years agoMerge pull request #39555 from tchaikov/wip-drop-pre-nautilus
Kefu Chai [Wed, 3 Mar 2021 08:39:46 +0000 (16:39 +0800)]
Merge pull request #39555 from tchaikov/wip-drop-pre-nautilus

*: drop backward compatibility with pre-nautilus versions

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
4 years agoMerge pull request #39798 from tchaikov/wip-crimson-monc-copy-sent
Kefu Chai [Wed, 3 Mar 2021 04:31:39 +0000 (12:31 +0800)]
Merge pull request #39798 from tchaikov/wip-crimson-monc-copy-sent

crimson/mon: keep a copy of sent MMonCommand messages

Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
4 years agocrimson/mon: keep a copy of sent MMonCommand messages 39798/head
Kefu Chai [Wed, 3 Mar 2021 03:39:36 +0000 (11:39 +0800)]
crimson/mon: keep a copy of sent MMonCommand messages

as per Yingxin Cheng,

> The send process can be asynchronous (there is a conn.out_q, or if the
> underlying socket lives in a different core in the m:n model to be
> ordered there). If user really wants to reuse a message, they must be
> careful not to modify it because it may result in modifing the pending
> messages.
>
> I think the best way is to copy the message if user want to resend it,
> and keep the ceph_assert(!msg->get_seq()). It may looks good to reuse a
> message under lossy policy, but the correctness is now up to user not to
> modify it inplace.

see also https://github.com/ceph/ceph/pull/39786

Signed-off-by: Kefu Chai <kchai@redhat.com>
4 years agodoc/rbd: clarify QCOW2 migration support 39795/head
Jason Dillaman [Tue, 2 Mar 2021 22:29:29 +0000 (17:29 -0500)]
doc/rbd: clarify QCOW2 migration support

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
4 years agovalgrind: suppress operator delete[] for tcmalloc 39794/head
Casey Bodley [Tue, 2 Mar 2021 14:31:08 +0000 (09:31 -0500)]
valgrind: suppress operator delete[] for tcmalloc

suppresses a mismatched operator delete[] from the librados client's
destructor:

<error>
  <unique>0xeef7eb</unique>
  <tid>1</tid>
  <kind>MismatchedFree</kind>
  <what>Mismatched free() / delete / delete []</what>
  <stack>
    <frame>
      <ip>0x4C32EA0</ip>
      <obj>/usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so</obj>
      <fn>operator delete[](void*, unsigned long)</fn>
      <dir>/builddir/build/BUILD/valgrind-3.16.0/coregrind/m_replacemalloc</dir>
      <file>vg_replace_malloc.c</file>
      <line>660</line>
    </frame>
    <frame>
      <ip>0x5FBEB86</ip>
      <obj>/usr/lib64/librados.so.2.0.0</obj>
    </frame>
    <frame>
      <ip>0x5FBEF2C</ip>
      <obj>/usr/lib64/librados.so.2.0.0</obj>
    </frame>
    <frame>
      <ip>0x5F7F7C5</ip>
      <obj>/usr/lib64/librados.so.2.0.0</obj>
      <fn>librados::v14_2_0::Rados::shutdown()</fn>
    </frame>

Fixes: https://tracker.ceph.com/issues/49501
Signed-off-by: Casey Bodley <cbodley@redhat.com>
4 years agomgr/DaemonServer: make warning language a bit friendlier
Sage Weil [Fri, 26 Feb 2021 17:14:09 +0000 (12:14 -0500)]
mgr/DaemonServer: make warning language a bit friendlier

This message is shown during the upgrade process.

Signed-off-by: Sage Weil <sage@newdream.net>
4 years agomgr/cephadm/upgrade: improve language a bit
Sage Weil [Fri, 26 Feb 2021 17:12:24 +0000 (12:12 -0500)]
mgr/cephadm/upgrade: improve language a bit

Signed-off-by: Sage Weil <sage@newdream.net>