]> git.apps.os.sepia.ceph.com Git - ceph.git/log
ceph.git
3 years agorgw: Fix data race in ChangeStatus 47196/head
Adam C. Emerson [Tue, 8 Feb 2022 18:47:49 +0000 (13:47 -0500)]
rgw: Fix data race in ChangeStatus

Fixes: https://tracker.ceph.com/issues/54208
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
(cherry picked from commit 27f5ba9e5f649d8767c8ab44d56404e0186f6fc1)
Fixes: https://tracker.ceph.com/issues/54491
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
3 years agoMerge pull request #47095 from aaSharma14/wip-56559-pacific
Ernesto Puerta [Wed, 20 Jul 2022 18:28:31 +0000 (20:28 +0200)]
Merge pull request #47095 from aaSharma14/wip-56559-pacific

pacific: os/bluestore: update perf counter priorities

Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>
Reviewed-by: neha-ojha <NOT@FOUND>
Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Yuri Weinstein <yweins@redhat.com>
3 years agoos/bluestore: update priorities and nicks of bluestore perf counters 47095/head
Laura Flores [Mon, 4 Oct 2021 04:41:10 +0000 (04:41 +0000)]
os/bluestore: update priorities and nicks of bluestore perf counters

These perf counters do not show up in telemetry unless they are set to a "useful" priority or higher. Fetching these counters in telemetry may help to diagnose problems with RocksDB / BlueFS prefetching / insufficient cache sizes.

Signed-off-by: Laura Flores <lflores@redhat.com>
(cherry picked from commit 8790f04c5469d86daebb3bbe1eb780a86b7404ce)

Note: This backport (and the original PR) exposed an extra 34 perf counters/OSD to Prometheus. Given Pacific is a stable release and not to add that much extra load we are adapting this backport and only exposing the 2 required perf-coun>

3 years agoMerge pull request #46553 from adk3798/pacific-preserve-cephadm-user
Kefu Chai [Tue, 19 Jul 2022 13:34:34 +0000 (21:34 +0800)]
Merge pull request #46553 from adk3798/pacific-preserve-cephadm-user

pacific: cephadm: preserve cephadm user during RPM upgrade

Reviewed-by: Kefu Chai <tchaikov@gmail.com>
3 years agoMerge pull request #47093 from rhcs-dashboard/wip-56546-pacific
Pedro Gonzalez Gomez [Tue, 19 Jul 2022 06:51:51 +0000 (08:51 +0200)]
Merge pull request #47093 from rhcs-dashboard/wip-56546-pacific

pacific: mgr/dashboard: don't log tracebacks on 404s

Reviewed-by: Pegonzal <pegonzal@redhat.com>
Reviewed-by: Avan Thakkar <athakkar@redhat.com>
3 years agoMerge pull request #47027 from tchaikov/pacific-56466
David Galloway [Mon, 18 Jul 2022 16:36:26 +0000 (12:36 -0400)]
Merge pull request #47027 from tchaikov/pacific-56466

pacific: make-dist: patch boost source to support python 3.10  …

3 years agoMerge pull request #47145 from s0nea/wip-56594-pacific
Pedro Gonzalez Gomez [Mon, 18 Jul 2022 13:35:10 +0000 (15:35 +0200)]
Merge pull request #47145 from s0nea/wip-56594-pacific

pacific: mgr/dashboard: prevent alert redirect

3 years agoMerge pull request #46695 from votdev/dashboard_pull_i18n_transifex
Nizamudeen A [Mon, 18 Jul 2022 12:33:57 +0000 (18:03 +0530)]
Merge pull request #46695 from votdev/dashboard_pull_i18n_transifex

mgr/dashboard: Pull latest languages from Transifex

Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Tatjana Dehler <tdehler@suse.com>
3 years agomgr/dashboard: prevent alert redirect 47145/head
Tatjana Dehler [Thu, 7 Jul 2022 15:21:14 +0000 (17:21 +0200)]
mgr/dashboard: prevent alert redirect

Prevent Alertmanager alerts from being redirected to the active mgr
dashboard instance. There are two reasons for it:

1. It doesn't bring any additional benefit. The Alertmanager config
   includes all available mgr instances - active and passive ones. In
   case of an alert, it will be sent to all of them. It ensures that
   the active mgr dashboard will receive the alert in any case.
2. The redirect URL includes the mgr IP and NOT the FQDN. This leads
   to issues in environments where an SSL certificate is configured and
   matches the FQDNs, only.

Fixes: https://tracker.ceph.com/issues/56401
Signed-off-by: Tatjana Dehler <tdehler@suse.com>
(cherry picked from commit 965005e0789e566ccadce7a326b0e197ab8d7f5f)

3 years agoMerge pull request #47008 from rhcs-dashboard/wip-56125-pacific
Nizamudeen A [Mon, 18 Jul 2022 07:52:39 +0000 (13:22 +0530)]
Merge pull request #47008 from rhcs-dashboard/wip-56125-pacific

pacific: mgr/dashboard: display helpfull message when the iframe-embedded Grafana dashboard failed to load

Reviewed-by: Aashish Sharma <aasharma@redhat.com>
Reviewed-by: nSedrickm <NOT@FOUND>
3 years agoMerge pull request #46969 from guits/wip-56471-2-pacific
Guillaume Abrioux [Mon, 18 Jul 2022 07:18:41 +0000 (09:18 +0200)]
Merge pull request #46969 from guits/wip-56471-2-pacific

pacific: ceph-volume: avoid unnecessary subprocess calls

3 years agoMerge pull request #47122 from zdover23/wip-doc-2022-07-16-backport-47109-to-pacific
zdover23 [Sat, 16 Jul 2022 18:45:42 +0000 (04:45 +1000)]
Merge pull request #47122 from zdover23/wip-doc-2022-07-16-backport-47109-to-pacific

pacific: doc/start: update hardware recs

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
3 years agoMerge pull request #47084 from rhcs-dashboard/wip-56544-pacific
Nizamudeen A [Sat, 16 Jul 2022 13:58:50 +0000 (19:28 +0530)]
Merge pull request #47084 from rhcs-dashboard/wip-56544-pacific

pacific: mgr/dashboard: ingress backend service should list all supported services

Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
3 years agocmake: patch boost source to support python 3.10 47027/head
Kefu Chai [Sat, 9 Jul 2022 01:58:32 +0000 (21:58 -0400)]
cmake: patch boost source to support python 3.10

Python 3.10 doesn't include the _Py_fopen() function.  Boost
1.75.0 includes a patch which switches to using fopen() for
python versions >= 3.1, but Pacific is using boost 1.73.0,
which still uses _Py_fopen().  This commit adds the boost
1.75.0 patch to `make-dist`, so it's applied to our copy of
the boost source which is then used when building RPM packages.

the included patch comes from
https://github.com/boostorg/python/commit/d9f06052e28873037db7f98629bce72182a42410

Fixes: https://tracker.ceph.com/issues/56466
please note, this change is not cherry-picked from the
"main" branch. as we use boost 1.75 already in that branch,
but to minimize the risk of switching boost from 1.73 to
1.75 in a LTS branch like pacific, we just add a fix to
address this particular issue in boost 1.73.

Signed-off-by: Tim Serong <tserong@suse.com>
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
3 years agodoc/start: update hardware recs 47122/head
Zac Dover [Thu, 14 Jul 2022 19:29:11 +0000 (05:29 +1000)]
doc/start: update hardware recs

This PR picks up the parts of
https://github.com/ceph/ceph/pull/44466
that were not merged back in January, when that
pull request was raised.

Matters added here:
* improved organzation of matter
* emphasis of IOPs per core over cores per OSD

Signed-off-by: Zac Dover <zac.dover@gmail.com>
(cherry picked from commit 29c44a3bd6919f7fc23083d75d22e84c8b51e1c1)

3 years agoMerge pull request #47103 from adk3798/pacific-osd-removal-docs-update
Adam King [Thu, 14 Jul 2022 17:27:57 +0000 (13:27 -0400)]
Merge pull request #47103 from adk3798/pacific-osd-removal-docs-update

pacific: doc/cephadm: add note about OSDs being recreated to OSD removal section

Reviewed-by: <Anthony D'Atri anthonyeleven@users.noreply.github.com>
3 years agodoc/cephadm: add note about OSDs being recreated to OSD removal section 47103/head
Adam King [Tue, 12 Jul 2022 20:54:19 +0000 (16:54 -0400)]
doc/cephadm: add note about OSDs being recreated to OSD removal section

Signed-off-by: Adam King <adking@redhat.com>
(cherry picked from commit d4a39cd046b93cb7bb5b7ce0311139d9f6552802)

3 years agomgr/dashboard: don't log tracebacks on 404s 47093/head
Ernesto Puerta [Wed, 29 Jun 2022 18:09:21 +0000 (20:09 +0200)]
mgr/dashboard: don't log tracebacks on 404s

Currently we're logging tracebacks on expected request errors as 404,
401, etc. This may cause confusion to users, which might think that
something really wrong was happening there.

This should be extended to many DashboardExceptions that simply return
expected HTTP error codes (403, 415, 3xx, etc).

Fixes: https://tracker.ceph.com/issues/55720
Signed-off-by: Ernesto Puerta <epuertat@redhat.com>
(cherry picked from commit 3928fe91a4b8c313a52e38cab9ba31e54ff1ae51)

3 years agomgr/dashboard: ingress backend service should list all supported services 47084/head
Avan Thakkar [Wed, 6 Jul 2022 09:24:26 +0000 (14:54 +0530)]
mgr/dashboard: ingress backend service should list all supported services

Fixes: https://tracker.ceph.com/issues/56478
Signed-off-by: Avan Thakkar <athakkar@redhat.com>
(cherry picked from commit 32118522cb71722c3cc20a1e1a9ca9ffdf7897e4)

3 years agoMerge pull request #46675 from rhcs-dashboard/wip-56038-pacific
Nizamudeen A [Wed, 13 Jul 2022 07:37:13 +0000 (13:07 +0530)]
Merge pull request #46675 from rhcs-dashboard/wip-56038-pacific

pacific: mgr/dashboard: stop polling when page is not visible

Reviewed-by: Aashish Sharma <aasharma@redhat.com>
Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
3 years agoMerge pull request #46953 from idryomov/wip-56387-pacific
Yuri Weinstein [Tue, 12 Jul 2022 13:31:05 +0000 (06:31 -0700)]
Merge pull request #46953 from idryomov/wip-56387-pacific

pacific: rbd-fuse: librados will filter out -r option from command-line

Reviewed-by: Mykola Golub <mgolub@suse.com>
Reviewed-by: Christopher Hoffman <choffman@redhat.com>
3 years agoMerge pull request #46767 from ronen-fr/wip-rf-42780-pacific
Yuri Weinstein [Tue, 12 Jul 2022 13:30:10 +0000 (06:30 -0700)]
Merge pull request #46767 from ronen-fr/wip-rf-42780-pacific

pacific: osd/scrub: mark PG as being scrubbed, from scrub initiation to Inacti…

Reviewed-by: Neha Ojha <nojha@redhat.com>
3 years agoMerge pull request #47049 from zdover23/wip-doc-2022-07-12-backport-47045-to-pacific
zdover23 [Mon, 11 Jul 2022 21:00:27 +0000 (07:00 +1000)]
Merge pull request #47049 from zdover23/wip-doc-2022-07-12-backport-47045-to-pacific

pacific: doc/dev: s/github/GitHub/ in essentials.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
3 years agoMerge pull request #47050 from zdover23/wip-doc-2022-07-12-backport-47031-to-pacific
zdover23 [Mon, 11 Jul 2022 20:55:21 +0000 (06:55 +1000)]
Merge pull request #47050 from zdover23/wip-doc-2022-07-12-backport-47031-to-pacific

pacific: doc/dev: edit delayed-delete.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
3 years agodoc/dev: s/github/GitHub/ in essentials.rst 47049/head
Zac Dover [Mon, 11 Jul 2022 16:58:18 +0000 (02:58 +1000)]
doc/dev: s/github/GitHub/ in essentials.rst

This addresses a typo that Anthony D'Atri pointed out in
PR#47032. It didn't get addressed there, so it's being
addressed here.

Signed-off-by: Zac Dover <zac.dover@gmail.com>
(cherry picked from commit f7581e05b08d6ab55446088b0fb9f29b1dd6441b)

3 years agodoc/dev: edit delayed-delete.rst 47050/head
Zac Dover [Sat, 9 Jul 2022 16:10:31 +0000 (02:10 +1000)]
doc/dev: edit delayed-delete.rst

This PR improves the English in the "CephFS Delayed
Deletion" chapter of the Ceph Internals Guide (which
is stored in the doc/dev/ directory).

There was no pressing need to rewrite this. I just saw
that it could be tightened up, and I had the fifteen
minutes I needed to do it.

Co-author: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@gmail.com>
(cherry picked from commit c186c5ec6cb2ea1999cb14500aaa64d6aa72dddd)

3 years agoMerge pull request #47043 from zdover23/wip-doc-2022-07-12-backport-47032-to-pacific
zdover23 [Mon, 11 Jul 2022 18:15:19 +0000 (04:15 +1000)]
Merge pull request #47043 from zdover23/wip-doc-2022-07-12-backport-47032-to-pacific

pacific: doc/dev: add Dependabot section to essentials.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
3 years agodoc/dev: add Dependabot section to essentials.rst 47043/head
Zac Dover [Sat, 9 Jul 2022 18:05:47 +0000 (04:05 +1000)]
doc/dev: add Dependabot section to essentials.rst

This PR adds a section to the Developer Guide chapter
"Essentials" that explains what Dependabot is. This
section is adapted from an email from Ernesto Puerta
to the CLT that was sent on 08 Jul 2022.

Co-authored-by: Ernesto Puerta <epuertat@redhat.com>>
Signed-off-by: Zac Dover <zac.dover@gmail.com>
(cherry picked from commit 2711e8de80edea121503e022cca37683f8abdfa9)

3 years agoMerge pull request #46921 from ceph/revert-46429-wip-55309-pacific
Yuri Weinstein [Mon, 11 Jul 2022 13:36:20 +0000 (06:36 -0700)]
Merge pull request #46921 from ceph/revert-46429-wip-55309-pacific

Revert "pacific: mgr, mgr/prometheus: Fix regression with prometheus metrics"

Reviewed-by: Laura Flores <lflores@redhat.com>
3 years agoMerge pull request #46920 from ceph/revert-46427-wip-55308-pacific
Yuri Weinstein [Mon, 11 Jul 2022 13:35:50 +0000 (06:35 -0700)]
Merge pull request #46920 from ceph/revert-46427-wip-55308-pacific

Revert "pacific: mgr, mon: Keep upto date metadata with mgr for MONs"

Reviewed-by: Laura Flores <lflores@redhat.com>
3 years agoMerge pull request #46845 from ronen-fr/wip-rf-46727-pacific
Yuri Weinstein [Mon, 11 Jul 2022 13:34:46 +0000 (06:34 -0700)]
Merge pull request #46845 from ronen-fr/wip-rf-46727-pacific

Pacific: osd/scrubber/pg_scrubber.cc: fix bug where scrub machine gets stuck

Reviewed-by: Neha Ojha <nojha@redhat.com>
3 years agoMerge pull request #46806 from nmshelke/wip-56104-pacific
Yuri Weinstein [Mon, 11 Jul 2022 13:34:03 +0000 (06:34 -0700)]
Merge pull request #46806 from nmshelke/wip-56104-pacific

pacific: mgr/volumes: subvolume ls command crashes if groupname as '_nogroup'

Reviewed-by: Venky Shankar <vshankar@redhat.com>
3 years agoMerge pull request #46803 from nmshelke/wip-56107-pacific
Yuri Weinstein [Mon, 11 Jul 2022 13:33:18 +0000 (06:33 -0700)]
Merge pull request #46803 from nmshelke/wip-56107-pacific

pacific: mgr/volumes: remove incorrect 'size' from output of 'snapshot info'

Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
3 years agoMerge pull request #46677 from ljflores/wip-51287-pacific
Yuri Weinstein [Mon, 11 Jul 2022 13:31:11 +0000 (06:31 -0700)]
Merge pull request #46677 from ljflores/wip-51287-pacific

pacific: ceph_test_librados_service: wait longer for servicemap to update

Reviewed-by: Kefu Chai <kchai@redhat.com>
3 years agoMerge pull request #46939 from zdover23/wip-doc-2022-07-03-backport-46938-to-pacific
zdover23 [Sat, 9 Jul 2022 16:12:27 +0000 (02:12 +1000)]
Merge pull request #46939 from zdover23/wip-doc-2022-07-03-backport-46938-to-pacific

pacific: doc/dev: add IRC registration instructions

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
3 years agodoc/dev: add IRC registration instructions 46939/head
Zac Dover [Sat, 2 Jul 2022 15:47:02 +0000 (01:47 +1000)]
doc/dev: add IRC registration instructions

This PR adds to the Developer Guide a link to the OFTC's IRC nick
registration instructions.

(cherry picked from commit 893e68a8f57b22cc025c82586ad0b42da2a6477c)

3 years agoMerge pull request #46340 from zdover23/wip-doc-pr-45209-backport-pacific
zdover23 [Fri, 8 Jul 2022 17:49:58 +0000 (03:49 +1000)]
Merge pull request #46340 from zdover23/wip-doc-pr-45209-backport-pacific

pacific: doc/start: alphabetize hardware-recs links

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
3 years agoMerge pull request #46347 from zdover23/wip-min-hardware-typo-pacific-backport-2022...
zdover23 [Fri, 8 Jul 2022 17:36:45 +0000 (03:36 +1000)]
Merge pull request #46347 from zdover23/wip-min-hardware-typo-pacific-backport-2022-05-19

pacific: Wip min hardware typo pacific backport 2022 05 19

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
3 years agoMerge pull request #46976 from adk3798/wip-56436-pacific
Adam King [Fri, 8 Jul 2022 12:23:31 +0000 (08:23 -0400)]
Merge pull request #46976 from adk3798/wip-56436-pacific

pacific: doc/cephadm: enhancing daemon operations documentation

Reviewed-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
3 years agoMerge pull request #46977 from adk3798/wip-56158-pacific
Adam King [Fri, 8 Jul 2022 12:22:17 +0000 (08:22 -0400)]
Merge pull request #46977 from adk3798/wip-56158-pacific

pacific: doc/cephadm: Add post-upgrade section

Reviewed-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
3 years agoMerge pull request #46974 from adk3798/wip-56473-pacific
Adam King [Fri, 8 Jul 2022 12:06:34 +0000 (08:06 -0400)]
Merge pull request #46974 from adk3798/wip-56473-pacific

pacific: pybind/mgr/cephadm/serve: don't remove ceph.conf which leads to qa failure

Reviewed-by: Dhairya Parmar <dparmar@redhat.com>
3 years agoMerge pull request #46069 from guits/wip-55479-pacific
Guillaume Abrioux [Fri, 8 Jul 2022 05:20:38 +0000 (07:20 +0200)]
Merge pull request #46069 from guits/wip-55479-pacific

pacific: ceph-volume: only warn when config file isn't found

3 years agoMerge pull request #46729 from guits/wip-56084-pacific
Guillaume Abrioux [Fri, 8 Jul 2022 05:15:38 +0000 (07:15 +0200)]
Merge pull request #46729 from guits/wip-56084-pacific

pacific: ceph-volume: do not log sensitive details

3 years agoMerge pull request #46731 from guits/wip-56040-pacific
Guillaume Abrioux [Fri, 8 Jul 2022 05:14:36 +0000 (07:14 +0200)]
Merge pull request #46731 from guits/wip-56040-pacific

pacific: ceph-volume: make is_valid() optional

3 years agoMerge pull request #46967 from guits/wip-56472-pacific
Guillaume Abrioux [Fri, 8 Jul 2022 05:13:03 +0000 (07:13 +0200)]
Merge pull request #46967 from guits/wip-56472-pacific

pacific: ceph-volume: decrease number of `pvs` calls in `lvm list`

3 years agoMerge pull request #47018 from guits/wip-56493-pacific
Guillaume Abrioux [Fri, 8 Jul 2022 04:10:05 +0000 (06:10 +0200)]
Merge pull request #47018 from guits/wip-56493-pacific

pacific: ceph-volume: allow listing devices by OSD ID

3 years agoMerge pull request #46975 from adk3798/wip-56455-pacific
Adam King [Fri, 8 Jul 2022 00:36:30 +0000 (20:36 -0400)]
Merge pull request #46975 from adk3798/wip-56455-pacific

pacific: mgr/cephadm: fixing scheduler consistent hashing

Reviewed-by: Redouane Kachach <rkachach@redhat.com>
3 years agoceph-volume: move get_lvs_from_path() to api/lvm.py 47018/head
Rishabh Dave [Mon, 30 Nov 2020 09:40:30 +0000 (15:10 +0530)]
ceph-volume: move get_lvs_from_path() to api/lvm.py

This makes this method reusable across ceph-volume codebase.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit 0ea399c01143673f0225b1b4a48136af75b0b6c5)

3 years agoceph-volume: allow listing devices by OSD ID
Rishabh Dave [Thu, 19 Nov 2020 13:06:38 +0000 (18:36 +0530)]
ceph-volume: allow listing devices by OSD ID

Adds the ability to list devices by OSD IDs, i.e. "ceph-volume lvm list
3" would list all devices under OSD ID 3 (which can be up to 2 and 3
devices under filestore and bluestore OSDs respectively).

Fixes: https://tracker.ceph.com/issues/41294
Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit 0792a396ce61b356ad004ac3dbd96fabed739eac)

3 years agoceph-volume: add methods to get LV from OSD ID
Rishabh Dave [Thu, 19 Nov 2020 13:05:39 +0000 (18:35 +0530)]
ceph-volume: add methods to get LV from OSD ID

Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit 7bf4937a597b37d98d3c4e69e2751c07992e33e4)

3 years agomgr/dashboard: Dashboard should display some helpful (error) message when the iframe... 47008/head
Ngwa Sedrick Meh [Mon, 14 Feb 2022 05:17:47 +0000 (06:17 +0100)]
mgr/dashboard: Dashboard should display some helpful (error) message when the iframe-embedded Grafana dashboard failed to load

This commit adds checks for successful grafana panel loads before displaying dashboards and informs the user if the request is blocked by the browser
Fixes: https://tracker.ceph.com/issues/54206
Signed-off-by: Ngwa Sedrick Meh <nsedrick101@gmail.com>
(cherry picked from commit a4b66efb2a2139bd88a6a088af9bd5e079e46105)

3 years agoMerge pull request #46934 from zdover23/wip-doc-2022-06-30-backport-46906-to-pacific
zdover23 [Thu, 7 Jul 2022 11:25:19 +0000 (21:25 +1000)]
Merge pull request #46934 from zdover23/wip-doc-2022-06-30-backport-46906-to-pacific

pacific: doc/dev: s/the the/the/ in basic-workflow.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
3 years agoRevert "pacific: mgr, mgr/prometheus: Fix regression with prometheus metrics" 46921/head
Prashant D [Fri, 1 Jul 2022 05:19:14 +0000 (15:19 +1000)]
Revert "pacific: mgr, mgr/prometheus: Fix regression with prometheus metrics"

This reverts pacific commit f07e47ae405218dbe4029ed9c82559d0cf1667a5

Reason: The pacific commit 98760e36c7bbbb48657e4d033bcbb080a56bbf7b has
added a new MMgrUpdate message which incorrectly handled service_daemon.
Reverting PR#46429 as a result of revert of PR#46427.

More details in https://tracker.ceph.com/issues/55322

Signed-off-by: Prashant D <pdhange@redhat.com>
3 years agoRevert "pacific: mgr, mon: Keep upto date metadata with mgr for MONs" 46920/head
Prashant D [Fri, 1 Jul 2022 05:18:24 +0000 (15:18 +1000)]
Revert "pacific: mgr, mon: Keep upto date metadata with mgr for MONs"

This reverts pacific commit 98760e36c7bbbb48657e4d033bcbb080a56bbf7b

Reason: The service_daemon is related to client daemons like rgw,
iscsi demons and it should not be handled in MMgrUpdate.

More details in https://tracker.ceph.com/issues/55322

Signed-off-by: Prashant D <pdhange@redhat.com>
3 years agodoc/cephadm: Add post-upgrade section 46977/head
Redouane Kachach [Thu, 16 Jun 2022 14:28:30 +0000 (16:28 +0200)]
doc/cephadm: Add post-upgrade section
Fixes: https://tracker.ceph.com/issues/54474
Signed-off-by: Redouane Kachach <rkachach@redhat.com>
(cherry picked from commit 49b234ae744dd3a2afc130d9c215f4f00c1ee4b8)

3 years agodoc/cephadm: enhancing daemon operations documentation 46976/head
Redouane Kachach [Wed, 15 Jun 2022 10:35:30 +0000 (12:35 +0200)]
doc/cephadm: enhancing daemon operations documentation
Fixes: https://tracker.ceph.com/issues/54399
Signed-off-by: Redouane Kachach <rkachach@redhat.com>
(cherry picked from commit df1aaacb7d09322528e42ae90c0d98793e9c254e)

3 years agomgr/cephadm: fixing E128 warning 46975/head
Redouane Kachach [Wed, 29 Jun 2022 15:03:10 +0000 (17:03 +0200)]
mgr/cephadm: fixing E128 warning

Signed-off-by: Redouane Kachach <rkachach@redhat.com>
(cherry picked from commit e37924c623d797b3118ddd9a804818432a5704dd)

3 years agomgr/cephadm: fixing scheduler consistent hashing
Redouane Kachach [Tue, 28 Jun 2022 13:32:19 +0000 (15:32 +0200)]
mgr/cephadm: fixing scheduler consistent hashing

Signed-off-by: Redouane Kachach <rkachach@redhat.com>
(cherry picked from commit 5d5829cd370e9278cb8b70d6426094e595badb5d)

3 years agopybind/mgr/cephadm/serve: don't remove ceph.conf which leads to qa failure 46974/head
dparmar18 [Tue, 14 Jun 2022 20:30:48 +0000 (02:00 +0530)]
pybind/mgr/cephadm/serve: don't remove ceph.conf which leads to qa failure

Fixes: https://tracker.ceph.com/issues/56024
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit 6436acc4b51b52635f8fa0e56cd79ba66c028d81)

3 years agoMerge pull request #46744 from mkogan1/rgw-backport-pacific-t54363
Yuri Weinstein [Tue, 5 Jul 2022 14:39:27 +0000 (07:39 -0700)]
Merge pull request #46744 from mkogan1/rgw-backport-pacific-t54363

pacific: rgwlc: fix segfault resharding during lc

Reviewed-by: Casey Bodley <cbodley@redhat.com>
3 years agoMerge pull request #46715 from mkogan1/rgw-pacific-fix-fips-segf
Yuri Weinstein [Tue, 5 Jul 2022 14:38:48 +0000 (07:38 -0700)]
Merge pull request #46715 from mkogan1/rgw-pacific-fix-fips-segf

pacific rgw: on FIPS enabled, fix segfault performing s3 multipart PUT

Reviewed-by: Casey Bodley <cbodley@redhat.com>
3 years agoMerge pull request #46235 from vumrao/wip-vumrao-54278
Yuri Weinstein [Tue, 5 Jul 2022 14:37:59 +0000 (07:37 -0700)]
Merge pull request #46235 from vumrao/wip-vumrao-54278

pacific: rgw: remove rgw_rados_pool_pg_num_min and its use on pool creation use the cluster defaults for pg_num_min

Reviewed-by: Casey Bodley <cbodley@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
3 years agoMerge pull request #43802 from rjhaverkamp/wip-53145-pacific
Yuri Weinstein [Tue, 5 Jul 2022 14:36:38 +0000 (07:36 -0700)]
Merge pull request #43802 from rjhaverkamp/wip-53145-pacific

pacific: rgw: fix self-comparison for RGWCopyObj optimization

Reviewed-by: Casey Bodley <cbodley@redhat.com>
3 years agoMerge pull request #46909 from idryomov/wip-rbd-deep-copy-progress-pacific
Yuri Weinstein [Tue, 5 Jul 2022 14:15:31 +0000 (07:15 -0700)]
Merge pull request #46909 from idryomov/wip-rbd-deep-copy-progress-pacific

pacific: librbd: update progress for non-existent objects on deep-copy

Reviewed-by: Mykola Golub <mgolub@suse.com>
Reviewed-by: Christopher Hoffman <choffman@redhat.com>
3 years agoMerge pull request #46815 from idryomov/wip-55389-pacific
Yuri Weinstein [Tue, 5 Jul 2022 14:14:32 +0000 (07:14 -0700)]
Merge pull request #46815 from idryomov/wip-55389-pacific

pacific: librbd/cache/pwl: fix endianness issue

Reviewed-by: Deepika Upadhyay <dupadhya@redhat.com>
3 years agoMerge pull request #46813 from idryomov/wip-rbd-mirror-remote-not-primary-pacific
Yuri Weinstein [Tue, 5 Jul 2022 14:13:46 +0000 (07:13 -0700)]
Merge pull request #46813 from idryomov/wip-rbd-mirror-remote-not-primary-pacific

pacific: rbd-mirror: generally skip replay/resync if remote image is not primary

Reviewed-by: Mykola Golub <mgolub@suse.com>
Reviewed-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch>
3 years agoMerge pull request #46778 from idryomov/wip-rbd-schedule-backports-pacific
Yuri Weinstein [Tue, 5 Jul 2022 14:12:31 +0000 (07:12 -0700)]
Merge pull request #46778 from idryomov/wip-rbd-schedule-backports-pacific

pacific: mirror snapshot schedule and trash purge schedule fixes

Reviewed-by: Deepika Upadhyay <dupadhya@redhat.com>
Reviewed-by: Sunny Kumar <sunkumar@redhat.com>
Reviewed-by: Mykola Golub <mgolub@suse.com>
3 years agoceph-volume: avoid unnecessary subprocess calls 46969/head
Guillaume Abrioux [Tue, 21 Jun 2022 07:55:49 +0000 (09:55 +0200)]
ceph-volume: avoid unnecessary subprocess calls

These calls are slowing down `ceph-volume inventory`.
Especially because of the class `ceph_volume.util.device.Devices`.
It calls the class `ceph_volume.util.device.Device` for each device found
on the host which calls many times the binaries `lsblk`, `pvs`, `vgs`, `lvs` itself.
We can make only one call in `Devices()` to gather all details and use them during the whole runtime

current implementation:

```
        1    0.000    0.000    0.892    0.892 device.py:35(__init__) (class Devices)
        8    0.001    0.000    0.853    0.107 device.py:151(_parse)
       56    0.002    0.000    0.882    0.016 process.py:154(call)
        8    0.001    0.000    0.245    0.031 lvm.py:1099(get_lvs)
        8    0.000    0.000    0.026    0.003 disk.py:231(lsblk)
        8    0.000    0.000    0.435    0.054 device.py:278(_set_lvm_membership)
        1    0.000    0.000    0.885    0.885 device.py:38(<listcomp>) (multiple calls to Device() class)
      8/5    0.000    0.000    0.885    0.177 device.py:92(__init__) (class Device)

>>> timeit.timeit('Inventory([]).main()', setup='from ceph_volume.inventory import Inventory', number=1)

Device Path               Size         rotates available Model name
/dev/sdb                  200.00 GB    True    True      QEMU HARDDISK
/dev/sda                  200.00 GB    True    False     QEMU HARDDISK
/dev/sdc                  200.00 GB    True    False     QEMU HARDDISK
/dev/sdd                  200.00 GB    True    False     QEMU HARDDISK
/dev/vda                  11.00 GB     True    False
0.9309048530412838
>>>
```

new approach:

```
        1    0.000    0.000    0.253    0.253 device.py:35(__init__) (class Devices)
        5    0.000    0.000    0.144    0.029 device.py:167(_parse)
       21    0.001    0.000    0.246    0.012 process.py:154(call)
        1    0.000    0.000    0.032    0.032 lvm.py:1110(get_lvs)
        1    0.000    0.000    0.005    0.005 disk.py:236(lsblk_all)
        5    0.000    0.000    0.062    0.012 device.py:309(_set_lvm_membership)
        1    0.000    0.000    0.179    0.179 device.py:41(<listcomp>) (multiple calls to Device() class)
        5    0.000    0.000    0.179    0.036 device.py:99(__init__) (class Device)

>>> timeit.timeit('Inventory([]).main()', setup='from ceph_volume.inventory import Inventory', number=1)

Device Path               Size         rotates available Model name
/dev/sdb                  200.00 GB    True    True      QEMU HARDDISK
/dev/sda                  200.00 GB    True    False     QEMU HARDDISK
/dev/sdc                  200.00 GB    True    False     QEMU HARDDISK
/dev/sdd                  200.00 GB    True    False     QEMU HARDDISK
/dev/vda                  11.00 GB     True    False
0.2486933789914474
>>>
```

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit bea9f4b643ce32268ad79c0fc257b25ff2f8333c)

3 years agoceph-volume: rename _get_pv_paths() in util/device.py
Guillaume Abrioux [Tue, 21 Jun 2022 07:53:43 +0000 (09:53 +0200)]
ceph-volume: rename _get_pv_paths() in util/device.py

This name is misleading, let's rename it for something clearer.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 325b6f50e88d8ee046fdba0b2bdb17a11577a2f6)

3 years agoceph-volume: decrease number of `pvs` calls in `lvm list` 46967/head
Guillaume Abrioux [Mon, 20 Jun 2022 11:43:43 +0000 (13:43 +0200)]
ceph-volume: decrease number of `pvs` calls in `lvm list`

current implementation of `List.create_report()` implies a lot of calls
to `pvs` process. This could be avoided.

current implementation:
```
>>> import timeit
>>> from ceph_volume.devices.lvm.listing import List
>>> timeit.timeit(List([]).main, number=1000)

...

93.03700458299136
```

new implementation:

```
>>> import timeit
>>> from ceph_volume.devices.lvm.listing import List
>>> timeit.timeit(List([]).main, number=1000)

...

62.16391600697534
```

In this example, it improves performance by ~30%

Fixes: https://tracker.ceph.com/issues/56127
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 1a38ea3b96dbe8fd2f2fca8ee9a501ef1423027f)

3 years agoMerge pull request #46531 from rhcs-dashboard/wip-55752-pacific
Ernesto Puerta [Mon, 4 Jul 2022 11:52:54 +0000 (13:52 +0200)]
Merge pull request #46531 from rhcs-dashboard/wip-55752-pacific

pacific: mgr/dashboard: batch rbd-mirror backports

Reviewed-by: Pegonzal <NOT@FOUND>
Reviewed-by: Sarthak Gupta <sarthak.dev.0702@gmail.com>
Reviewed-by: Aashish Sharma <aasharma@redhat.com>
Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: melissa-kun-li <NOT@FOUND>
Reviewed-by: Nizamudeen A <nia@redhat.com>
3 years agorbd-fuse: librados will filter out -r option from command-line 46953/head
wanwencong [Fri, 24 Jun 2022 15:54:52 +0000 (23:54 +0800)]
rbd-fuse: librados will filter out -r option from command-line

The -r option will be filtered out by librados
when exec cmd "rbd-fuse /mountpoint -p pool_name -r rbd_name"
other rbds can be seen under the mount point

Fixes: https://tracker.ceph.com/issues/56387
Signed-off-by: wanwencong <wanwc@chinatelecom.cn>
(cherry picked from commit e99d64bc8a5c3bbb8a3632f211d4f56751cf499e)

3 years agodoc/dev: s/the the/the/ in basic-workflow.rst 46934/head
Zac Dover [Thu, 30 Jun 2022 11:08:51 +0000 (21:08 +1000)]
doc/dev: s/the the/the/ in basic-workflow.rst

This PR does exactly what it says: it changes a "the the"
to a "the".

I'd like to thank Josh Soref for bringing this
to my attention.

(cherry picked from commit 326034807d58fe6569851c043f71a222f59cde5e)

3 years agoMerge pull request #46870 from rhcs-dashboard/wip-56118-pacific
Pere Diaz Bou [Fri, 1 Jul 2022 09:35:51 +0000 (11:35 +0200)]
Merge pull request #46870 from rhcs-dashboard/wip-56118-pacific

pacific: mgr/dashboard: iterate through copy of items

Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
3 years agomgr/dashboard: fix rbd remote calls 46531/head
Pere Diaz Bou [Thu, 30 Jun 2022 11:44:49 +0000 (13:44 +0200)]
mgr/dashboard: fix rbd remote calls

Signed-off-by: Pere Diaz Bou <pdiazbou@redhat.com>
3 years agomgr/dashboard: rbd image primary ui
Pedro Gonzalez Gomez [Fri, 3 Jun 2022 10:13:44 +0000 (12:13 +0200)]
mgr/dashboard: rbd image primary ui

Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
Signed-off-by: Pere Diaz Bou <pdiazbou@redhat.com>
3 years agomgr/dashboard: fix linting issues
Pere Diaz Bou [Tue, 21 Jun 2022 08:34:56 +0000 (10:34 +0200)]
mgr/dashboard: fix linting issues

Signed-off-by: Pere Diaz Bou <pdiazbou@redhat.com>
3 years agomgr/dashboard: Error page cleanup
Nizamudeen A [Wed, 15 Jun 2022 17:22:39 +0000 (22:52 +0530)]
mgr/dashboard: Error page cleanup

Some error page cleanups

Signed-off-by: Nizamudeen A <nia@redhat.com>
3 years agomgr/dashboard: add rbd status endpoint
Melissa Li [Thu, 26 May 2022 18:07:30 +0000 (14:07 -0400)]
mgr/dashboard: add rbd status endpoint

Show "No RBD pools available" error page when accessing block/rbd if there are no rbd pools.
Add a "button_name" and "button_route" property to `ModuleStatusGuardService` config to customize the button on the error page.
Modify `ModuleStatusGuardService` to execute API calls to `/ui-api/<uiApiPath>/status` which uses the `UIRouter`.

Fixes: https://tracker.ceph.com/issues/42109
Signed-off-by: Melissa Li <melissali@redhat.com>
(cherry picked from commit 6ac9b3cfe171a8902454ea907b3ba37d83eda3dc)

3 years agomgr/dashboard: configure rbd mirroring
Nizamudeen A [Mon, 6 Jun 2022 05:51:29 +0000 (11:21 +0530)]
mgr/dashboard: configure rbd mirroring

One-click button in the case of an orch cluster for configuring the
rbd-mirroring when its not properly setup. This button will create an
rbd-mirror service and also an rbd labelled pool(replicated: size-3) (if they are not
existing)

Fixes: https://tracker.ceph.com/issues/55646
Signed-off-by: Nizamudeen A <nia@redhat.com>
 Conflicts:
src/pybind/mgr/dashboard/frontend/src/app/core/error/error.component.html
src/pybind/mgr/dashboard/frontend/src/app/shared/services/module-status-guard.service.ts
src/pybind/mgr/dashboard/tests/test_rbd_mirroring.py

error component and module-status-guard.service.ts had one minor
conflict which was resolved by getting incoming changes (add missing
logic).
test_rbd_mirroring had an import conflict which was resolved by
accepting incoming changes which had the same imports with new ones.

3 years agomgr/dashboard: improve edit site name action in rbd-mirroring
Nizamudeen A [Mon, 13 Jun 2022 08:24:15 +0000 (13:54 +0530)]
mgr/dashboard: improve edit site name action in rbd-mirroring

Fixes: https://tracker.ceph.com/issues/55896
Signed-off-by: Nizamudeen A <nia@redhat.com>
3 years agomgr/dashboard: rbd force resync from fornt-end
Sarthak0702 [Thu, 2 Jun 2022 22:58:31 +0000 (04:28 +0530)]
mgr/dashboard: rbd force resync from fornt-end

Signed-off-by: Sarthak0702 <sarthak.dev.0702@gmail.com>
3 years agomgr/dashboard: fix mirroring e2e and lint errors
Avan Thakkar [Wed, 8 Jun 2022 11:03:18 +0000 (16:33 +0530)]
mgr/dashboard: fix mirroring e2e and lint errors

Signed-off-by: Avan Thakkar <athakkar@redhat.com>
3 years agomgr/dashboard: add byte info, move state, add idle state
Pere Diaz Bou [Tue, 7 Jun 2022 17:57:44 +0000 (19:57 +0200)]
mgr/dashboard: add byte info, move state, add idle state

Idle substate added from snapshot mode.
Instead of seconds info we display bytes and entries info.

Signed-off-by: Pere Diaz Bou <pdiazbou@redhat.com>
Conflicts:
src/pybind/mgr/dashboard/controllers/rbd_mirroring.py

Fixed a minor import conflict

3 years agomgr/dashboard: move replaying images to Syncing tab
Pere Diaz Bou [Mon, 30 May 2022 14:10:11 +0000 (16:10 +0200)]
mgr/dashboard: move replaying images to Syncing tab

Images with 'Replaying' state will be displayed in Syncing tab. syncTmpl
removed as it was unnecessary if sate is provided from the backend.

Replaying images in contrast of Syncing images don't have a progress
percentage, nevertheless, we have an approximation of how much time left
there is until the image is fully synced. Therefore, we can use seconds_until_synced to represent the progress.

Signed-off-by: Pere Diaz Bou <pdiazbou@redhat.com>
3 years agomgr/dashboard: snapshot mirroring from dashboard
Pere Diaz Bou [Fri, 13 May 2022 15:15:33 +0000 (17:15 +0200)]
mgr/dashboard: snapshot mirroring from dashboard

Enable snapshot mirroring from the Pools -> Image

Also show the mirror-snapshot in the image where snapshot is enabled

When parsing images if an image has the snapshot mode enabled, it will
try to  run commands that don't work with that mode. The solution was
not running those for now and appending the mode in the get call.

Fixes: https://tracker.ceph.com/issues/55648
Signed-off-by: Pere Diaz Bou <pdiazbou@redhat.com>
Signed-off-by: Nizamudeen A <nia@redhat.com>
Signed-off-by: Aashish Sharma <aasharma@redhat.com>
Signed-off-by: Avan Thakkar <athakkar@redhat.com>
(cherry picked from commit 489a385a95d6ffa5dbd4c5f9c53c1f80ea179142)

3 years agomgr/dashboard: expose image mirroring commands as endpoints
Pere Diaz Bou [Thu, 12 May 2022 18:29:01 +0000 (20:29 +0200)]
mgr/dashboard: expose image mirroring commands as endpoints

Expose:
  - enable/disable mirroring in image
  - promote/demote (primary and non-primary)
  - resync
  - snapshot mode:
    - mirror image snapshot (manual snapshot)
    - schedule

Fixes: https://tracker.ceph.com/issues/55645
Signed-off-by: Pere Diaz Bou <pdiazbou@redhat.com>
(cherry picked from commit 8bd89415fe340512f457acd58225934e9ed8e4e1)

3 years agolibrbd: make ImageCopyRequest::send_next_object_copy() return void 46909/head
Ilya Dryomov [Tue, 28 Jun 2022 18:47:25 +0000 (20:47 +0200)]
librbd: make ImageCopyRequest::send_next_object_copy() return void

Make send_object_copies() consistent with handle_object_copy() wrt
calling send_next_object_copy().

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit 155fabd994f8a2052482d216df1ec0dfb40cb5b7)

3 years agolibrbd: update progress for non-existent objects on deep-copy
Ilya Dryomov [Sun, 26 Jun 2022 11:05:09 +0000 (13:05 +0200)]
librbd: update progress for non-existent objects on deep-copy

As a side effect of commit e5a21e904142 ("librbd: deep-copy image copy
state machine skips clean objects"), handle_object_copy() stopped being
called for non-existent objects.  This broke progress_object_no logic,
which expects to "see" all object numbers so that update_progress()
callback invocations can be ordered.  Currently update_progress() based
progress reporting gets stuck after encountering a hole in the image.

To fix, arrange for handle_object_copy() to be called for all object
numbers, even if ObjectCopyRequest isn't created.  Defer the extra call
to the image work queue to avoid locking issues.

Fixes: https://tracker.ceph.com/issues/56181
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit 6813a7879146aec40f204a174b40a5a54e00b780)

3 years agoMerge pull request #46903 from zdover23/wip-doc-2022-06-30-backport-46984-to-pacific
zdover23 [Thu, 30 Jun 2022 10:58:24 +0000 (20:58 +1000)]
Merge pull request #46903 from zdover23/wip-doc-2022-06-30-backport-46984-to-pacific

pacific: doc/index.rst: add link to Dev Guide basic workfl.

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
3 years agodoc/index.rst: add link to Dev Guide basic workfl. 46903/head
Zac Dover [Wed, 29 Jun 2022 12:57:13 +0000 (22:57 +1000)]
doc/index.rst: add link to Dev Guide basic workfl.

This PR adds a link to the "Basic Workflow" section of the
Developer Guide on the landing page of docs.ceph.com.

This PR is meant to improve the documentation for developers
new to Ceph and to guide them to instructions that will allow
them to become full-fledged contributors to the Ceph project
as quickly as possible.

The "Basic Workflow" page of the Developer Guide contains
information that answers almost all of the questions that I had
about contributing to the Ceph project when I was new to it,
and I am finally acting on my long-held conviction that the
"Basic Workflow" page of the Developer Guide should have a more
prominent position in the documentation suite than it has had.

Signed-off-by: Zac Dover <zac.dover@gmail.com>
(cherry picked from commit c624a2a8ef71035eef34ee866ef7668f83a139e7)

3 years agoMerge pull request #46717 from rhcs-dashboard/wip-56075-pacific
Ernesto Puerta [Wed, 29 Jun 2022 18:20:30 +0000 (20:20 +0200)]
Merge pull request #46717 from rhcs-dashboard/wip-56075-pacific

pacific: mgr/dashboard: bump moment from 2.29.1 to 2.29.3 in /src/pybind/mgr/dashboard/frontend

Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
3 years agoMerge pull request #46626 from neesingh-rh/wip-55937-pacific
Laura Flores [Wed, 29 Jun 2022 16:01:01 +0000 (11:01 -0500)]
Merge pull request #46626 from neesingh-rh/wip-55937-pacific

pacific: client: Inode::hold_caps_until is time from monotonic clock now.

3 years agotest/librados: modify LibRadosMiscConnectFailure.ConnectFailure to comply with new... 46626/head
Laura Flores [Thu, 9 Jun 2022 18:55:48 +0000 (18:55 +0000)]
test/librados: modify LibRadosMiscConnectFailure.ConnectFailure to comply with new seconds unit

The unit type for `client_mount_timeout` was changed from "float" to "secs" in
983b10506dc8466a0e47ff0d320d480dd09999ec. To make this test comply with the new
seconds unit change, we need to change the value to an integer, as seconds
does not accept float values.

Fixes: https://tracker.ceph.com/issues/55971
Signed-off-by: Laura Flores <lflores@redhat.com>
(cherry picked from commit f357459e6b159229ad40491709f756b06a6e87f1)
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
3 years agoclient: Inode::hold_caps_until is time from monotonic clock now.
Neeraj Pratap Singh [Wed, 8 Dec 2021 08:42:59 +0000 (14:12 +0530)]
client: Inode::hold_caps_until is time from monotonic clock now.

Inode::hold_caps_until storing time from ceph::coarse_mono_clock now.

This upstream code of this PR i.e. the parent PR contains the file
`src/common/options/mds-client.yaml.in` which intends to fix a part
of this PR whereas this file didn't exist in pacific branch.So, those
changes of `src/common/options/mds-client.yaml.in` are incorporated in
the below mentioned files in Conlicts.
Fixes: https://tracker.ceph.com/issues/52982
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
(cherry picked from commit 983b10506dc8466a0e47ff0d320d480dd09999ec)

Conflicts:`/src/common/legacy_config_opts.h`
          `/src/common/options.cc`

3 years agoMerge pull request #46802 from lxbsz/wip-56105
Yuri Weinstein [Tue, 28 Jun 2022 15:17:28 +0000 (08:17 -0700)]
Merge pull request #46802 from lxbsz/wip-56105

pacific: qa: wait rank 0 to become up:active state before mounting fuse client

Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Dhairya Parmar <dparmar@redhat.com>
3 years agoMerge pull request #46799 from lxbsz/wip-56110
Yuri Weinstein [Tue, 28 Jun 2022 15:15:42 +0000 (08:15 -0700)]
Merge pull request #46799 from lxbsz/wip-56110

pacific: client: choose auth MDS for getxattr with the Xs caps

Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
3 years agoMerge pull request #46798 from lxbsz/wip-55449
Yuri Weinstein [Tue, 28 Jun 2022 15:15:08 +0000 (08:15 -0700)]
Merge pull request #46798 from lxbsz/wip-55449

pacific: client: add option to disable collecting and sending metrics

Reviewed-by: Venky Shankar <vshankar@redhat.com>
3 years agoMerge pull request #46682 from lxbsz/wip-56016
Yuri Weinstein [Tue, 28 Jun 2022 15:14:27 +0000 (08:14 -0700)]
Merge pull request #46682 from lxbsz/wip-56016

Pacific: mds: clear MDCache::rejoin_*_q queues before recovering file inodes

Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
3 years agoMerge pull request #46679 from lxbsz/wip-55993
Yuri Weinstein [Tue, 28 Jun 2022 15:13:53 +0000 (08:13 -0700)]
Merge pull request #46679 from lxbsz/wip-55993

Pacific: client: switch AT_NO_ATTR_SYNC to AT_STATX_DONT_SYNC

Reviewed-by: Nikhilkumar Shelke <nshelke@redhat.com>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>