]> git.apps.os.sepia.ceph.com Git - ceph.git/log
ceph.git
8 weeks agolibrbd/migration/NBDStream: be more thorough around nbd_block_status()
Ilya Dryomov [Sun, 25 Aug 2024 11:10:58 +0000 (13:10 +0200)]
librbd/migration/NBDStream: be more thorough around nbd_block_status()

nbd_block_status() is specified to be really loose: it can return
more status (go beyond the requested range), less status (cover only
some part of the requested range) or nothing at all.  The former would
likely violate some assumptions in ObjectCopyRequest, while the latter
would lead to straight data corruption -- parent blocks that weren't
covered would never get copied up into the destination image.

On top of that, the current implementation is very suboptimal.
Because only "nr_entries == 2" responses are effectively processed,
any extent which has a small amount of data and is otherwise zeroed
(e.g. 8K of data in a 4M object) gets listed as DATA.  A copyup for
such an object would write 4M worth of data instead of 8K, consuming
space in the destination image unnecessarily.

Also, it's unclear whether zero-sized entries can be returned.
Currently, if it happens we would hit an assert in interval_map.

This fixes all of these issues.

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

8 weeks agolibrbd/migration: add nbd stream
Effi Ofer [Tue, 18 Jan 2022 12:43:32 +0000 (14:43 +0200)]
librbd/migration: add nbd stream

Co-authored-by: Ilya Dryomov <idryomov@gmail.com>
Signed-off-by: Effi Ofer <effio@il.ibm.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit dc8eed68ce58077433e38317f20f3a96ec81f467)

8 weeks agoqa/workunits/rbd: exercise snap_{name,id} parsing in test_import_native_format()
Ilya Dryomov [Wed, 21 Aug 2024 19:16:30 +0000 (21:16 +0200)]
qa/workunits/rbd: exercise snap_{name,id} parsing in test_import_native_format()

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

8 weeks agodoc/rbd: clarify when image_id is expected for import-only migration
Ilya Dryomov [Sat, 17 Aug 2024 08:28:50 +0000 (10:28 +0200)]
doc/rbd: clarify when image_id is expected for import-only migration

"optional if image in trash" can be easily interpreted as "required if
image not in trash".

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

8 weeks agolibrbd/migration: add external clusters support
Ilya Dryomov [Fri, 16 Aug 2024 17:09:39 +0000 (19:09 +0200)]
librbd/migration: add external clusters support

This commit extends NativeFormat (aka migration where the migration
source is an RBD image) to support external Ceph clusters, limited to
import-only mode.

Co-authored-by: Or Ozeri <oro@il.ibm.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit 293d523ea69aaf1fa5c372f058f4426d49a3e196)

8 weeks agolibrbd/migration: move away from util::create_ioctx() in NativeFormat
Ilya Dryomov [Mon, 5 Aug 2024 15:52:10 +0000 (17:52 +0200)]
librbd/migration: move away from util::create_ioctx() in NativeFormat

This is another step towards supporting migration from external
clusters, where creating an IoCtx from a Rados instance that has
nothing to do with dst_io_ctx would be needed.  It also allows to
get rid of a pool lookup in the middle of parsing code.

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

Conflicts:
src/librbd/migration/NativeFormat.cc [ commit 8e58a52a408b
  ("librbd/migration: don't include ImageState.h in formats")
  was backported to squid out of order ]

8 weeks agocommon/config: export CEPH_CONF_FILE_DEFAULT
Ilya Dryomov [Fri, 16 Aug 2024 12:12:38 +0000 (14:12 +0200)]
common/config: export CEPH_CONF_FILE_DEFAULT

It used to be exported until commit 318c62f8ae16 ("common/config:
cleanup remove some unused macros").  Having CEPH_CONF_FILE_DEFAULT
avaialable is handy to prevent parse_config_files() from picking up
CEPH_CONF environment variable.

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

8 weeks agolibrbd: RefreshParentRequest::m_parent_snap_id is unused
Ilya Dryomov [Wed, 14 Aug 2024 16:36:57 +0000 (18:36 +0200)]
librbd: RefreshParentRequest::m_parent_snap_id is unused

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

8 weeks agolibrbd: assert on parent in ImageCtx destructor
Ilya Dryomov [Wed, 14 Aug 2024 17:42:09 +0000 (19:42 +0200)]
librbd: assert on parent in ImageCtx destructor

... and switch to in-class initializers while at it.

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

8 weeks agoqa/tasks/qemu: remove hard-coding of cluster name
Or Ozeri [Mon, 6 Nov 2023 11:56:27 +0000 (13:56 +0200)]
qa/tasks/qemu: remove hard-coding of cluster name

This commit allows running the qemu task on an arbitrary cluster name.

Signed-off-by: Or Ozeri <oro@il.ibm.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit 409d36c7087e1e12dd2ad321c39dc26b3271582f)

8 weeks agoqa/tasks/rbd: support non-default ceph clusters
Or Ozeri [Wed, 15 Nov 2023 09:47:54 +0000 (11:47 +0200)]
qa/tasks/rbd: support non-default ceph clusters

This commit allows running the rbd task on an arbitrary cluster name.

Signed-off-by: Or Ozeri <oro@il.ibm.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit 4f8dc834cc6cfb006a10fe4b3e715df5e720f6fe)

8 weeks agolibrbd/migration: don't clone when flattening
Or Ozeri [Tue, 31 Jan 2023 11:08:22 +0000 (13:08 +0200)]
librbd/migration: don't clone when flattening

When the flatten flag is set, instead of creating the
destination image by cloning, create it independently,
as the parent relation is unnecessary in this case.
This will be particularly useful when the migration source
is located in an external Ceph cluster, which will soon be
supported.

Signed-off-by: Or Ozeri <oro@il.ibm.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit 1ee644d97104c9caaa83918b98b388888fc5497b)

8 weeks agoMerge pull request #63393 from zdover23/wip-doc-2025-05-21-backport-63365-to-squid
Anthony D'Atri [Wed, 21 May 2025 14:41:27 +0000 (10:41 -0400)]
Merge pull request #63393 from zdover23/wip-doc-2025-05-21-backport-63365-to-squid

squid: doc/mgr: edit debug.inc.rst

8 weeks agoMerge pull request #63396 from zdover23/wip-doc-2025-05-21-backport-63367-to-squid
Anthony D'Atri [Wed, 21 May 2025 14:40:12 +0000 (10:40 -0400)]
Merge pull request #63396 from zdover23/wip-doc-2025-05-21-backport-63367-to-squid

squid: doc/mgr: edit feature_toggles.inc.rst

8 weeks agodoc/mgr: edit feature_toggles.inc.rst 63396/head
Zac Dover [Tue, 20 May 2025 05:37:55 +0000 (15:37 +1000)]
doc/mgr: edit feature_toggles.inc.rst

Edit doc/mgr/dashboard_plugins/features_toggles.inc.rst.

This commit is part of a project to separate out the twenty-five files
that were committed to https://github.com/ceph/ceph/pull/62782.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 18e4ca258194238699649857f5f0ba195b67dea2)

8 weeks agodoc/mgr: edit debug.inc.rst 63393/head
Zac Dover [Tue, 20 May 2025 05:29:23 +0000 (15:29 +1000)]
doc/mgr: edit debug.inc.rst

Edit doc/mgr/dashboard_plugins/debug.inc.rst.

This commit is part of a project to separate out the twenty-five files
that were committed to https://github.com/ceph/ceph/pull/63255.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 1e87aa66b9de0a4e74bb4b1fbd806d974e0e8dcf)

8 weeks agoMerge pull request #62994 from clwluvw/wip-71041
Yuri Weinstein [Mon, 19 May 2025 14:37:03 +0000 (07:37 -0700)]
Merge pull request #62994 from clwluvw/wip-71041

squid: rgw: skip empty check on non-owned buckets by zonegroup

Reviewed-by: Adam Emerson <aemerson@redhat.com>
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
8 weeks agoMerge pull request #62959 from galsalomon66/squid-s3select
Yuri Weinstein [Mon, 19 May 2025 14:36:31 +0000 (07:36 -0700)]
Merge pull request #62959 from galsalomon66/squid-s3select

squid: rgw/s3select

Reviewed-by: Adam Emerson <aemerson@redhat.com>
8 weeks agoMerge pull request #61764 from thuvh/wip-69373-squid
Yuri Weinstein [Mon, 19 May 2025 14:34:39 +0000 (07:34 -0700)]
Merge pull request #61764 from thuvh/wip-69373-squid

squid: rgw: allow send bucket notification to multiple brokers of kafka cluster

Reviewed-by: Yuval Lifshitz <ylifshit@redhat.com>
8 weeks agoMerge pull request #63337 from zdover23/wip-doc-2025-05-19-backport-63329-to-squid
Anthony D'Atri [Mon, 19 May 2025 14:06:07 +0000 (07:06 -0700)]
Merge pull request #63337 from zdover23/wip-doc-2025-05-19-backport-63329-to-squid

squid: doc: Fix missing blank line Sphinx warnings

8 weeks agodoc: Fix missing blank line Sphinx warnings 63337/head
Ville Ojamo [Sun, 18 May 2025 05:25:07 +0000 (12:25 +0700)]
doc: Fix missing blank line Sphinx warnings

Fix four warnings from Sphinx about missing blank line after explicit
markup.

Indent content in list items correctly, fixing formatting errors.

Signed-off-by: Ville Ojamo <14869000+bluikko@users.noreply.github.com>
(cherry picked from commit 5e1b3cd566fd700b36001551d1c7e092fc850def)

2 months agoMerge pull request #63311 from zdover23/wip-doc-2025-05-16-backport-63293-to-squid
Anthony D'Atri [Sat, 17 May 2025 03:51:27 +0000 (23:51 -0400)]
Merge pull request #63311 from zdover23/wip-doc-2025-05-16-backport-63293-to-squid

squid: doc/radosgw: Use ref for hyperlinking to multisite

2 months agoMerge pull request #63315 from zdover23/wip-doc-2025-05-16-backport-63301-to-squid
Anthony D'Atri [Fri, 16 May 2025 06:47:27 +0000 (02:47 -0400)]
Merge pull request #63315 from zdover23/wip-doc-2025-05-16-backport-63301-to-squid

squid: doc/mgr: edit dashboard.rst

2 months agodoc/mgr: edit dashboard.rst 63315/head
Zac Dover [Thu, 15 May 2025 13:24:58 +0000 (23:24 +1000)]
doc/mgr: edit dashboard.rst

Edit doc/mgr/crash.rst. Add prompts.

This changes eighty-nine prompts. Because this makes so many changes,
all other edits included in https://github.com/ceph/ceph/pull/63255 will
be made in a separate commit. This done for the sake of the patience of
the reviewers (probably Anthony, if history is any guide).

This commit is part of a project to separate out the twenty-five files
that were committed to https://github.com/ceph/ceph/pull/63255.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 1f587883a671775b44b656cf6b7c641d02319b5f)

2 months agodoc/radosgw: Use ref for hyperlinking to multisite 63311/head
Ville Ojamo [Thu, 15 May 2025 09:46:21 +0000 (16:46 +0700)]
doc/radosgw: Use ref for hyperlinking to multisite

Use validated ":ref:" hyperlinks instead of "external links" in "target
definitions" when linking within the Ceph docs:
- Update to use existing label in multisite.rst.
- Remove unused "target definitions".

Also use existing label for linking from multisite.rst.
Fix a broken link within multisite.rst.

The rendered PR should look the same as the old docs, only differing in
the source RST.

Signed-off-by: Ville Ojamo <14869000+bluikko@users.noreply.github.com>
(cherry picked from commit e296c756ca191753080cb65e3df4bf3bb160398b)

2 months agoMerge pull request #63298 from zdover23/wip-doc-2025-05-15-backport-63256-to-squid
Anthony D'Atri [Thu, 15 May 2025 15:57:33 +0000 (11:57 -0400)]
Merge pull request #63298 from zdover23/wip-doc-2025-05-15-backport-63256-to-squid

squid: doc/dev/cephfs-mirroring: edit file 1 of x

2 months agoMerge pull request #63211 from ceph/squid-rtd
David Galloway [Thu, 15 May 2025 13:37:54 +0000 (09:37 -0400)]
Merge pull request #63211 from ceph/squid-rtd

squid: .github: Fix RTD build retrigger

2 months agodoc/dev/cephfs-mirroring: edit file 1 of x 63298/head
Zac Dover [Tue, 13 May 2025 06:31:42 +0000 (16:31 +1000)]
doc/dev/cephfs-mirroring: edit file 1 of x

Add prompts (and perform necessary corrections to glaring grammatical
errors) to doc/dev/cephfs-mirroring.rst, as requested by Jos Collin in https://github.com/ceph/ceph/pull/63237/files#r2085886075.

This commit edits the first quarter of the doc/dev/cephfs-mirroring.rst
file. This commit encompasses about one-hundred lines of RST.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit e9a27051a7c0f4884e685a4105e45273aef0abe0)

2 months agoMerge pull request #63054 from shraddhaag/wip-70999-squid
SrinivasaBharathKanta [Thu, 15 May 2025 10:33:03 +0000 (16:03 +0530)]
Merge pull request #63054 from shraddhaag/wip-70999-squid

squid: qa/suites/orch/cephadm: add PG_DEGRADED to ignorelist

2 months agoMerge pull request #62198 from NitzanMordhai/wip-69887-squid
SrinivasaBharathKanta [Thu, 15 May 2025 10:32:29 +0000 (16:02 +0530)]
Merge pull request #62198 from NitzanMordhai/wip-69887-squid

squid: dencoder tests fix type backwards incompatible checks

2 months agoMerge pull request #63273 from zdover23/wip-doc-2025-05-14-backport-63259-to-squid
Anthony D'Atri [Wed, 14 May 2025 10:54:35 +0000 (06:54 -0400)]
Merge pull request #63273 from zdover23/wip-doc-2025-05-14-backport-63259-to-squid

squid: doc/dev/cephfs-mirroring: edit file 2 of x

2 months agodoc/dev/cephfs-mirroring: edit file 2 of x 63273/head
Zac Dover [Tue, 13 May 2025 06:58:39 +0000 (16:58 +1000)]
doc/dev/cephfs-mirroring: edit file 2 of x

Add prompts (and perform necessary corrections to glaring grammatical
errors) to doc/dev/cephfs-mirroring.rst, as requested by Jos Collin in
https://github.com/ceph/ceph/pull/63237/files#r2085886075.

This commit edits the second quarter of the doc/dev/cephfs-mirroring.rst
file. This commit encompasses about one-hundred lines of RST.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 0768ef3fcab69392635d32ed6c4c7051b27afd37)

2 months agoMerge pull request #63250 from zdover23/wip-doc-2025-05-13-backport-63237-to-squid
Anthony D'Atri [Tue, 13 May 2025 14:09:15 +0000 (10:09 -0400)]
Merge pull request #63250 from zdover23/wip-doc-2025-05-13-backport-63237-to-squid

squid: doc: fix formatting in cephfs_mirror dev doc

2 months agodoc: fix formatting in cephfs_mirror dev doc 63250/head
Jos Collin [Mon, 12 May 2025 12:42:36 +0000 (18:12 +0530)]
doc: fix formatting in cephfs_mirror dev doc

Signed-off-by: Jos Collin <jcollin@redhat.com>
(cherry picked from commit 8280d334b9b3ee98cb1372d91991fa476a89424b)

2 months agoMerge pull request #63187 from rhcs-dashboard/wip-71258-squid
Nizamudeen A [Mon, 12 May 2025 14:22:26 +0000 (19:52 +0530)]
Merge pull request #63187 from rhcs-dashboard/wip-71258-squid

squid: mgr/dashboard: fix make check tests

2 months agoMerge pull request #63229 from zdover23/wip-doc-2025-05-12-backport-63209-to-squid
Anthony D'Atri [Mon, 12 May 2025 13:55:21 +0000 (09:55 -0400)]
Merge pull request #63229 from zdover23/wip-doc-2025-05-12-backport-63209-to-squid

squid: doc/radosgw: Cosmetic and formatting improvements in vault.rst

2 months agopybind/mgr: attempt to fix mypy importing from python-common 63187/head
John Mulligan [Wed, 18 Sep 2024 03:21:31 +0000 (20:21 -0700)]
pybind/mgr: attempt to fix mypy importing from python-common

For some reason mypy on python 3.12 can no longer automatically find
imports from python-common. Help it out by expanding the MYPYPATH
value for the tox.ini.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
(cherry picked from commit d1c942a98499560bb15c50db4498f12f4916866f)

2 months agodoc/radosgw: Cosmetic and formatting improvements in vault.rst 63229/head
Ville Ojamo [Fri, 9 May 2025 10:15:02 +0000 (17:15 +0700)]
doc/radosgw: Cosmetic and formatting improvements in vault.rst

Use generated "contents" instead of repeating section titles in an
ordered list.

Use Title Case for section titles.

Capitalize consistently "Ceph" and "Object Gateway", "Vault".

Use consistently all caps for acronyms.

Change one entry of "kms" to "SSE-KMS", seems obvious from the context.
Use "certificates" instead of "certs".

Use double backticks for file paths, commands, config options etc.

Improve language in one sentence, several words missing.

Call it consistently "the Object Gateway" and not sometimes "the
Gateway".

Add pre-formatted block for CLI command examples where missing (should
be promptified in the future).

Signed-off-by: Ville Ojamo <14869000+bluikko@users.noreply.github.com>
(cherry picked from commit aa2e91ff70ea37a56f4013e6087f7e790f527329)

2 months ago.github: Fix RTD build retrigger 63211/head
David Galloway [Tue, 6 May 2025 21:31:03 +0000 (17:31 -0400)]
.github: Fix RTD build retrigger

Fixes: https://tracker.ceph.com/issues/70430
Signed-off-by: David Galloway <david.galloway@ibm.com>
(cherry picked from commit 27521bf5580d4f878fbf7cafbd854bc6ef94aed4)

2 months agoMerge pull request #63197 from zdover23/wip-doc-2025-05-09-backport-63172-to-squid
Anthony D'Atri [Fri, 9 May 2025 12:45:20 +0000 (08:45 -0400)]
Merge pull request #63197 from zdover23/wip-doc-2025-05-09-backport-63172-to-squid

squid: doc/mgr/ceph_api: edit index.rst

2 months agoMerge pull request #63200 from zdover23/wip-doc-2025-05-08-backport-63173-to-squid
Anthony D'Atri [Fri, 9 May 2025 12:44:06 +0000 (08:44 -0400)]
Merge pull request #63200 from zdover23/wip-doc-2025-05-08-backport-63173-to-squid

squid: doc/mgr: edit alerts.rst

2 months agoMerge pull request #63207 from zdover23/wip-doc-2025-05-09-backport-63084-to-squid
Anthony D'Atri [Fri, 9 May 2025 12:27:59 +0000 (08:27 -0400)]
Merge pull request #63207 from zdover23/wip-doc-2025-05-09-backport-63084-to-squid

squid: doc/mgr: edit administrator.rst

2 months agodoc/mgr: edit administrator.rst 63207/head
Zac Dover [Thu, 1 May 2025 04:12:45 +0000 (14:12 +1000)]
doc/mgr: edit administrator.rst

Edit doc/mgr/administrator.rst. Add prompts and correct some of the
inelegant English present in the file.

This commit is part of a project to separate out the twenty-five files
that were committed to https://github.com/ceph/ceph/pull/62782.

Credit for this initiative should go to Ville Ojamo.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 859950180f0b7314b097ee9b2f9f192f4cfb3ed4)

2 months agodoc/mgr: edit alerts.rst 63200/head
Zac Dover [Thu, 8 May 2025 02:29:25 +0000 (12:29 +1000)]
doc/mgr: edit alerts.rst

Edit doc/mgr/alerts.rst as part of the project to determine where the
error is in https://github.com/ceph/ceph/pull/62782 that prevents the
Jenkins tests from passing.

This commit adds to the work done in
https://github.com/ceph/ceph/pull/62782 by correcting some of the
English that was present in that PR.

This is a change to one of twenty-five files in
https://github.com/ceph/ceph/pull/62782, and this commit represents one
of what will be at least twenty-five other commits made to track this
error down.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit db28c1f32ff67bd1087866c7976233e8ab23fa2c)

2 months agodoc/mgr/ceph_api: edit index.rst 63197/head
Zac Dover [Thu, 8 May 2025 00:08:06 +0000 (10:08 +1000)]
doc/mgr/ceph_api: edit index.rst

Edit doc/mgr/ceph_api/index.rst as part of the project to determine
where the error is in https://github.com/ceph/ceph/pull/62782 that
prevents the Jenkins tests from passing.

This is a change to one of twenty-five files in
https://github.com/ceph/ceph/pull/62782, and this commit represents one
of what will be at least twenty-five other commits made to track this
error down.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 5c17f2242318d57e9faaafc83774a8f8a36f6e0e)

2 months agomgr/dashboard: fix make check tests
Afreen Misbah [Thu, 8 May 2025 08:53:07 +0000 (14:23 +0530)]
mgr/dashboard: fix make check tests

Fixes https://tracker.ceph.com/issues/71246

Pr are failing with errors import errors running pylint

Signed-off-by: Afreen Misbah <afreen@ibm.com>
(cherry picked from commit 29f0f2914efacd2e6b9bc7115f5c2d3790bb12cc)

2 months agoMerge pull request #63147 from zdover23/wip-doc-2025-05-07-backport-63139-to-squid
Anthony D'Atri [Wed, 7 May 2025 12:52:32 +0000 (08:52 -0400)]
Merge pull request #63147 from zdover23/wip-doc-2025-05-07-backport-63139-to-squid

squid: doc/cephadm: Correct formatting in upgrade.rst

2 months agodoc/cephadm: Correct formatting in upgrade.rst 63147/head
Anthony D'Atri [Tue, 6 May 2025 16:42:18 +0000 (12:42 -0400)]
doc/cephadm: Correct formatting in upgrade.rst

Signed-off-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
(cherry picked from commit 0e1adc4ae69b9fdc8f4a8480f3e498efa06520f4)

2 months agoMerge pull request #62039 from kamoltat/wip-70220-squid
SrinivasaBharathKanta [Tue, 6 May 2025 09:26:26 +0000 (14:56 +0530)]
Merge pull request #62039 from kamoltat/wip-70220-squid

squid: src/mon/OSDMonitor.cc: [Stretch Mode] WRN non-existent CRUSH location assigned to MON

2 months agoMerge pull request #63108 from zdover23/wip-doc-2025-05-05-backport-62860-to-squid
Zac Dover [Tue, 6 May 2025 00:07:17 +0000 (10:07 +1000)]
Merge pull request #63108 from zdover23/wip-doc-2025-05-05-backport-62860-to-squid

squid: doc: update cephfs-journal-tool docs

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
2 months agodoc: update cephfs-journal-tool docs 63108/head
Jos Collin [Thu, 17 Apr 2025 10:13:46 +0000 (15:43 +0530)]
doc: update cephfs-journal-tool docs

Fixes: https://tracker.ceph.com/issues/70953
Signed-off-by: Jos Collin <jcollin@redhat.com>
(cherry picked from commit 7ae0c22dfd63d14773966ab3796c68f1be913534)

2 months agoMerge pull request #63012 from guits/wip-71105-squid
Guillaume Abrioux [Thu, 1 May 2025 06:42:22 +0000 (08:42 +0200)]
Merge pull request #63012 from guits/wip-71105-squid

squid: ceph-volume: Fix splitting with too many parts

2 months agoMerge pull request #63073 from zdover23/wip-doc-2025-04-30-backport-63060-to-squid
Anthony D'Atri [Wed, 30 Apr 2025 14:51:25 +0000 (10:51 -0400)]
Merge pull request #63073 from zdover23/wip-doc-2025-04-30-backport-63060-to-squid

squid: doc/cephadm: correct markup in rgw.rst

2 months agodoc/cephadm: correct markup in rgw.rst 63073/head
Zac Dover [Wed, 30 Apr 2025 08:13:40 +0000 (18:13 +1000)]
doc/cephadm: correct markup in rgw.rst

Correct the presentation of an example string in doc/cephadm/rgw.rst in
order to obviate an error reading "rgw.rst:202: WARNING: Inline emphasis start-string without end-string."

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 64bb1cffe3dd2fc495f9f4ddab1dc6c512eda624)

2 months agoMerge pull request #63071 from sseshasa/wip-71142-squid
Anthony D'Atri [Wed, 30 Apr 2025 13:28:55 +0000 (09:28 -0400)]
Merge pull request #63071 from sseshasa/wip-71142-squid

squid: doc/rados: Update mClock doc on steps to override OSD IOPS capacity config

2 months agoMerge PR #61287 into squid
Patrick Donnelly [Wed, 30 Apr 2025 12:36:28 +0000 (08:36 -0400)]
Merge PR #61287 into squid

* refs/pull/61287/head:
mds: add or update MDS thread names
log: cache recent threads up to a day
common: cache pthread names
log: concatenate thread names and print once per thread

Reviewed-by: Laura Flores <lflores@redhat.com>
2 months agoceph-volume: Fix splitting with too many parts 63012/head
Janne Heß [Mon, 28 Apr 2025 09:04:25 +0000 (11:04 +0200)]
ceph-volume: Fix splitting with too many parts

The data can be anything and also contain a `=`, causing the line to
fail with `Too many values to unpack`. In my case, it failed with
`ID_FS_LABEL=pvc_name=rook-ceph-lvm-data-44f2gc`.

Regression was introduced here: https://github.com/ceph/ceph/pull/60006

Fixes: https://tracker.ceph.com/issues/71101
Signed-off-by: Janne Heß <janne@hess.ooo>
(cherry picked from commit 315fb96243549afba316c60a8d74c24c9417c18e)

2 months agodoc/rados: Update mClock doc on steps to override OSD IOPS capacity config 63071/head
Sridhar Seshasayee [Tue, 15 Apr 2025 13:01:50 +0000 (18:31 +0530)]
doc/rados: Update mClock doc on steps to override OSD IOPS capacity config

Describe the steps involved to
 - Specify a global value for osd_mclock_max_capacity_iops_{ssd,hdd}, and
 - Override existing individually scoped values for OSDs determined during
   start-up for osd_mclock_max_capacity_iops_{ssd,hdd}.

The above is to help with the following:
 - Steps to override existing setting with a global value.
 - reduce the number of entries in the mon store and instead use a single
   global specification for all OSDs in the cluster in case the underlying
   hardware is the same for all OSDs.

Signed-off-by: Sridhar Seshasayee <sseshasa@redhat.com>
Fixes: https://tracker.ceph.com/issues/70774
(cherry picked from commit 899da82ae2e857f2a769c08d6623e6da7678d16b)

2 months agoqa/suites/orch/cephadm: add PG_DEGRADED to ignorelist 63054/head
Shraddha Agrawal [Thu, 3 Apr 2025 11:09:47 +0000 (16:39 +0530)]
qa/suites/orch/cephadm: add PG_DEGRADED to ignorelist

Issue: tests are failing in rados/cephadm due to
PG_DEGRADED warning in cluster log.

Cause: This is expected as we are intentionally killing OSDs.
Adding PG_DEGRADED warning to ignorelist will prevent the
test from failing when this warning is raised.

Fixes: https://tracker.ceph.com/issues/70466
Signed-off-by: Shraddha Agrawal <shraddha.agrawal000@gmail.com>
(cherry picked from commit 5ffbc5bd4fbb47ce8949ae20f955bc4598b427c8)

2 months agoMerge pull request #59191 from NitzanMordhai/wip-67504-squid
Yuri Weinstein [Tue, 29 Apr 2025 14:50:49 +0000 (07:50 -0700)]
Merge pull request #59191 from NitzanMordhai/wip-67504-squid

squid: qa/tasks: watchdog should terminate thrasher

https://github.com/ceph/ceph/pull/59191

2 months agoMerge pull request #62802 from sseshasa/wip-70904-squid
SrinivasaBharathKanta [Tue, 29 Apr 2025 14:50:36 +0000 (20:20 +0530)]
Merge pull request #62802 from sseshasa/wip-70904-squid

squid: osd/scheduler/OpSchedulerItem: Fix calculation of recovery latency counters

2 months agoMerge pull request #62437 from linuxkidd/backport-max-avail-pr-57003-to-squid
SrinivasaBharathKanta [Tue, 29 Apr 2025 14:49:53 +0000 (20:19 +0530)]
Merge pull request #62437 from linuxkidd/backport-max-avail-pr-57003-to-squid

squid: PGMap: remove pool max_avail scale factor

2 months agoMerge pull request #62514 from ifed01/wip-ifed-fix-bluefs-reserved2-squi
Igor Fedotov [Tue, 29 Apr 2025 13:57:36 +0000 (16:57 +0300)]
Merge pull request #62514 from ifed01/wip-ifed-fix-bluefs-reserved2-squi

squid: os/bluestore: use block size (4K) as minimal allocation unit for dedicated DB/WAL volumes

Reviewed-by: Adam Kupczyk <akupczyk@ibm.com>
2 months agoos/bluestore: introduce locking for the very first DB/WAL alloc unit. 62514/head
Igor Fedotov [Fri, 21 Mar 2025 10:25:41 +0000 (13:25 +0300)]
os/bluestore: introduce locking for the very first DB/WAL alloc unit.

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit eb461b4fca243d9ac5885cfa9e5d5bf1b6af1fcb)

2 months agoos/bluestore: rename bluefs_super_t::version -> seq
Igor Fedotov [Thu, 20 Mar 2025 17:36:29 +0000 (20:36 +0300)]
os/bluestore: rename bluefs_super_t::version -> seq

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit 4130c43d50e6cbd7697b57753f1a889528912334)

2 months agoos/bluestore: rename BlueFS::get_total -> get_block_device_size()
Igor Fedotov [Thu, 20 Mar 2025 17:17:42 +0000 (20:17 +0300)]
os/bluestore: rename BlueFS::get_total -> get_block_device_size()

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit d6a2d378707cd807444243029252e9d9bc0d08b6)

2 months agotest/store_test: add BlueFS reserved space tests
Igor Fedotov [Fri, 7 Mar 2025 17:35:42 +0000 (20:35 +0300)]
test/store_test: add BlueFS reserved space tests

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit 2e3578b01490c3350a4a955155dabbc2ea2710e8)

2 months agoos/bluestore: get rid off BlueFS::get_block_device_size
Igor Fedotov [Fri, 7 Mar 2025 11:29:34 +0000 (14:29 +0300)]
os/bluestore: get rid off BlueFS::get_block_device_size

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit acdd0e363c76073e2fc60de5e9b55d3441c74a66)

2 months agotools/ceph-bluestore-tool: introduce bluefs-super-dump command.
Igor Fedotov [Thu, 6 Mar 2025 20:44:11 +0000 (23:44 +0300)]
tools/ceph-bluestore-tool: introduce bluefs-super-dump command.

This is a rework of Adam's commit:
https://github.com/ceph/ceph/pull/62069/commits/c8e57c4e06691bef7718277267e7590ae695bd02

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit e9d3ea827568afb6063db2ed4fe222b455e5ddec)

2 months agoos/bluestore: use dev's block size as a minimal BlueFS allocation unit.
Igor Fedotov [Fri, 28 Feb 2025 09:40:33 +0000 (12:40 +0300)]
os/bluestore: use dev's block size as a minimal BlueFS allocation unit.

Additionall this locks tail of DB/WAL volumes which is unaligned to
configured (not minimal!!) BlueFS allocation unit.

Effectively replaces changes from
https://github.com/ceph/ceph/pull/57015

Fixes: https://tracker.ceph.com/issues/68772
Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit effaa686f38b9eff2f7b9c8df2ffaf76c9a49aff)

2 months agoos/bluestore: do not substract 'reserved' from bluefs bdev sizes
Igor Fedotov [Fri, 28 Feb 2025 08:35:01 +0000 (11:35 +0300)]
os/bluestore: do not substract 'reserved' from bluefs bdev sizes

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit 33b43af750fa2c62bf153b87124ee947460dac64)

2 months agoos/bluestore: show allocator types in metadata dump
Igor Fedotov [Fri, 28 Feb 2025 07:54:57 +0000 (10:54 +0300)]
os/bluestore: show allocator types in metadata dump

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit 1aae3b66fa55f893f7ec002854bb699541635d53)

2 months agoMerge pull request #62839 from aclamk/aclamk-bluefs-remove-truncate-squid
Igor Fedotov [Tue, 29 Apr 2025 08:51:53 +0000 (11:51 +0300)]
Merge pull request #62839 from aclamk/aclamk-bluefs-remove-truncate-squid

squid: os/bluestore: Fix race in BlueFS truncate / remove

Reviewed-by: Igor Fedotov <igor.fedotov@croit.io>
2 months agoMerge pull request #62254 from ifed01/wip-ifed-bool-async-discard-back-squi
Igor Fedotov [Tue, 29 Apr 2025 08:49:06 +0000 (11:49 +0300)]
Merge pull request #62254 from ifed01/wip-ifed-bool-async-discard-back-squi

squid: blk/kernel: bring "bdev_async_discard" config parameter back.

Reviewed-by: Yite Gu <guyite@bytedance.com>
2 months agoMerge pull request #62968 from idryomov/wip-71026-squid
Yuri Weinstein [Mon, 28 Apr 2025 19:17:20 +0000 (12:17 -0700)]
Merge pull request #62968 from idryomov/wip-71026-squid

squid: librbd: disallow "rbd trash mv" if image is in a group

Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
2 months agoMerge pull request #62965 from idryomov/wip-70640-squid
Yuri Weinstein [Mon, 28 Apr 2025 19:16:18 +0000 (12:16 -0700)]
Merge pull request #62965 from idryomov/wip-70640-squid

squid: mgr/rbd_support: always parse interval and start_time in Schedules::remove()

Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
2 months agoMerge pull request #62963 from idryomov/wip-70632-squid
Yuri Weinstein [Mon, 28 Apr 2025 19:15:13 +0000 (12:15 -0700)]
Merge pull request #62963 from idryomov/wip-70632-squid

squid: librbd: respect rbd_default_snapshot_quiesce_mode in group_snap_create()

Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
2 months agoMerge pull request #62940 from nbalacha/wip-71047-squid
Yuri Weinstein [Mon, 28 Apr 2025 19:14:23 +0000 (12:14 -0700)]
Merge pull request #62940 from nbalacha/wip-71047-squid

squid: rbd: display mirror state creating

Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
2 months agoMerge pull request #62417 from ivancich/wip-70594-squid
Yuri Weinstein [Mon, 28 Apr 2025 18:41:54 +0000 (11:41 -0700)]
Merge pull request #62417 from ivancich/wip-70594-squid

squid: rgw: radoslist improvements primarily to better support gap list tool

Reviewed-by: Adam Emerson <aemerson@redhat.com>
2 months agoMerge pull request #62607 from cbodley/wip-70750-squid
Yuri Weinstein [Mon, 28 Apr 2025 18:35:35 +0000 (11:35 -0700)]
Merge pull request #62607 from cbodley/wip-70750-squid

squid: <rgw> Ensure the ETag format is consistent with AWS S3 API

Reviewed-by: Adam Emerson <aemerson@redhat.com>
2 months agoMerge pull request #62999 from zdover23/wip-doc-2025-04-28-backport-62979-to-squid
Anthony D'Atri [Mon, 28 Apr 2025 03:56:46 +0000 (23:56 -0400)]
Merge pull request #62999 from zdover23/wip-doc-2025-04-28-backport-62979-to-squid

squid: doc/radosgw: Improve formatting in layout.rst

2 months agodoc/radosgw: Improve formatting in layout.rst 62999/head
Anthony D'Atri [Fri, 25 Apr 2025 17:10:06 +0000 (13:10 -0400)]
doc/radosgw: Improve formatting in layout.rst

Signed-off-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
(cherry picked from commit a7dcc3cf24239a9728d9622e4a980894e7741e48)

2 months agoMerge pull request #62995 from ronen-fr/wip-rf-qa71099-squid
Ronen Friedman [Sun, 27 Apr 2025 15:29:29 +0000 (18:29 +0300)]
Merge pull request #62995 from ronen-fr/wip-rf-qa71099-squid

squid: osd/scrub: always round up reported scrub duration

Reviewed-by: Matan Breizman <mbreizma@redhat.com>
2 months agoMerge pull request #62956 from ronen-fr/wip-rf-62692-squid
Ronen Friedman [Sun, 27 Apr 2025 14:25:53 +0000 (17:25 +0300)]
Merge pull request #62956 from ronen-fr/wip-rf-62692-squid

squid: osd/scrub: additional configuration parameters to trigger scrub reschedule

Reviewed-by: Matan Breizman <mbreizma@redhat.com>
2 months agoMerge pull request #62974 from ronen-fr/wip-rf-24647e8-qatarget-squid
Ronen Friedman [Sun, 27 Apr 2025 14:09:10 +0000 (17:09 +0300)]
Merge pull request #62974 from ronen-fr/wip-rf-24647e8-qatarget-squid

squid: qa/standalone/scrub: fix osd-scrub-test.sh

Reviewed-by: Matan Breizman <mbreizma@redhat.com>
2 months agoosd/scrub: always round up reported scrub duration 62995/head
Ronen Friedman [Sat, 26 Apr 2025 08:33:59 +0000 (03:33 -0500)]
osd/scrub: always round up reported scrub duration

as expected by some tests, and clearer for the user.

Fixes: https://tracker.ceph.com/issues/71099
Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
2 months agoceph-object-corpus: update the submodule 62198/head
Nitzan Mordechai [Wed, 4 Dec 2024 11:33:33 +0000 (11:33 +0000)]
ceph-object-corpus: update the submodule

ceph-object-corpus submodule

Signed-off-by: Nitzan Mordechai <nmordech@redhat.com>
(cherry picked from commit 882eaa7ceece9043ebc82e7a700b7dca9af8b066)

2 months agoworkunit/tes_readable: enforce cascading backward compatibility checks
Nitzan Mordechai [Wed, 31 Jul 2024 10:38:11 +0000 (10:38 +0000)]
workunit/tes_readable: enforce cascading backward compatibility checks

should_skip_object function needed to handle cascading backward
compatibility checks.
I also added improved handling of backward compatibility filtering
in should_skip_object so we can better find backward comp. and skip
them.

Fixes: https://tracker.ceph.com/issues/69009
Signed-off-by: Nitzan Mordechai <nmordech@redhat.com>
(cherry picked from commit fa98a781b44f36791a00a0de48b21a3ba836341d)

2 months agoqa/standalone/scrub: fix osd-scrub-test.sh 62974/head
Ronen Friedman [Wed, 26 Jun 2024 15:02:19 +0000 (10:02 -0500)]
qa/standalone/scrub: fix osd-scrub-test.sh

following changes in scrub code

(cherry picked from commit 24647e87e8fba9b16d81730662b22798ed1885cb)
Conflict resolved by:
- electing to keep the up-to-date order between 'set noscrub' and 'set ..chunk_max'
in 'step 2'

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
2 months agorgw: allow bucket deletion from secondary zonegroup 62994/head
Seena Fallah [Wed, 9 Oct 2024 21:32:49 +0000 (23:32 +0200)]
rgw: allow bucket deletion from secondary zonegroup

Allow running `radosgw-admin bucket rm` from secondary zonegroup.
This allows bucket deletion with `--purge-objects` and with
`--bypass-gc` when deleting the bucket owned by non-master zonegroup.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit c7694ddedc3e12d2e7dcf5357e96e6a6f813bfa5)

2 months agorgw: call RGWBucketAdminOp::remove_bucket by RGWOp_Bucket_Remove
Seena Fallah [Wed, 8 Jan 2025 22:27:31 +0000 (23:27 +0100)]
rgw: call RGWBucketAdminOp::remove_bucket by RGWOp_Bucket_Remove

To align same functionality for bucket deletion from both API and
rgw-admin, use the same function from RGWBucketAdminOp.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 9ae2d8c4e95807179fc17f84be6754d2b19fe639)

2 months agorgw: skip empty check on non-owned buckets by zonegroup
Seena Fallah [Wed, 9 Oct 2024 21:30:40 +0000 (23:30 +0200)]
rgw: skip empty check on non-owned buckets by zonegroup

Compare RGW's zonegroup with bucket's zonegroup and only do the empty
check when the bucket is owned by the RGW running the delete.

Fixes: https://tracker.ceph.com/issues/68190
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit e975658b105812fc66718ea1e1c77025869539e4)

2 months agoMerge pull request #62989 from zdover23/wip-doc-2025-04-26-backport-62988-to-squid
Anthony D'Atri [Sat, 26 Apr 2025 14:25:15 +0000 (10:25 -0400)]
Merge pull request #62989 from zdover23/wip-doc-2025-04-26-backport-62988-to-squid

squid: doc/radosgw: Fix RST syntax rendeded as text in oidc.rst

2 months agodoc/radosgw: Fix RST syntax rendeded as text in oidc.rst 62989/head
Ville Ojamo [Sat, 26 Apr 2025 04:17:16 +0000 (11:17 +0700)]
doc/radosgw: Fix RST syntax rendeded as text in oidc.rst

Empty line after starting a pre-formatted block with the double-colon
syntax is required, otherwise the double-colon does nothing and is just
rendered as-is as "::" and there would be no following pre-formatted
block.

Add empty lines after the double-colon syntax so that the following
block is rendered pre-formatted.

Also add bash privileged prompts to a block with 2 example CLI commands.

Signed-off-by: Ville Ojamo <14869000+bluikko@users.noreply.github.com>
(cherry picked from commit b4fb5dc4030b4de477942630ebe3b98b882ae978)

2 months agorgw: add release note for newly-quoted ETag values 62607/head
Casey Bodley [Wed, 26 Mar 2025 16:22:07 +0000 (12:22 -0400)]
rgw: add release note for newly-quoted ETag values

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit b1c795847bd68331d657cf9fe2259ae79bbdec40)

2 months ago<rgw> Ensure the ETag format is consistent with AWS S3 API
liubingrun [Thu, 24 Oct 2024 14:23:09 +0000 (10:23 -0400)]
<rgw> Ensure the ETag format is consistent with AWS S3 API

AWS S3 API quotes ETAG everywhere. We still missed a few places.

checked with following cmd.
```
$ rg -i "dump.*\"ETag\"" ceph
./src/rgw/rgw_rest_s3.cc
1863:        s->formatter->dump_format("ETag", "\"%s\"", iter->meta.etag.c_str());
1955:      s->formatter->dump_format("ETag", "\"%s\"", iter->meta.etag.c_str());
2030:        s->formatter->dump_format("ETag", "\"%s\"", iter->meta.etag.c_str());
2099:      s->formatter->dump_format("ETag", "\"%s\"", iter->meta.etag.c_str());
2785:      s->formatter->dump_format("ETag", "\"%s\"", etag.c_str());
3422:    s->formatter->dump_string("ETag", etag);
3725:      s->formatter->dump_format("ETag", "\"%s\"",etag.c_str());
4228:    s->formatter->dump_string("ETag", etag);
4298:      s->formatter->dump_format("ETag", "\"%s\"", part->get_etag().c_str());

./src/rgw/rgw_rest.cc
421:    return dump_header(s, "etag", etag);
423:    return dump_header_quoted(s, "ETag", etag);

./src/rgw/driver/rados/rgw_sync_module_es_rest.cc
347:      s->formatter->dump_format("ETag", "\"%s\"", e.meta.etag.c_str());

./src/rgw/rgw_admin.cc
8612:        handled = dump_string("etag", bl, formatter.get());
```

Signed-off-by: liubingrun <liubr1@chinatelecom.cn>
(cherry picked from commit b8795299aacfa2aee5ec4a6fe117424e2ce97b88)

Conflicts:
src/rgw/rgw_rest_s3.cc no checksums in response

2 months agoMerge pull request #62748 from ivancich/wip-70825-squid
Yuri Weinstein [Fri, 25 Apr 2025 16:04:50 +0000 (09:04 -0700)]
Merge pull request #62748 from ivancich/wip-70825-squid

squid: rgw: add force option to radosgw-admin object rm ...

Reviewed-by: Adam Emerson <aemerson@redhat.com>
2 months agoMerge pull request #62741 from jzhu116-bloomberg/wip-70741-squid
Yuri Weinstein [Fri, 25 Apr 2025 16:04:13 +0000 (09:04 -0700)]
Merge pull request #62741 from jzhu116-bloomberg/wip-70741-squid

squid: rgw: cleanup orphaned bucket entry in <user>.buckets OMAP

Reviewed-by: Adam Emerson <aemerson@redhat.com>
2 months agoMerge pull request #62559 from cbodley/wip-70517-squid
Yuri Weinstein [Fri, 25 Apr 2025 16:00:13 +0000 (09:00 -0700)]
Merge pull request #62559 from cbodley/wip-70517-squid

squid: rgw: rgw_init_ioctx() adds set_pool_full_try()

Reviewed-by: Adam Emerson <aemerson@redhat.com>
2 months agoMerge pull request #62356 from cbodley/wip-70528-squid
Yuri Weinstein [Fri, 25 Apr 2025 15:58:36 +0000 (08:58 -0700)]
Merge pull request #62356 from cbodley/wip-70528-squid

squid: rgw: sync fairness watcher reconnects on any error

Reviewed-by: Daniel Gryniewicz <dang@redhat.com>
Reviewed-by: Oguzhan Ozmen <oozmen@bloomberg.net>