]> git.apps.os.sepia.ceph.com Git - ceph.git/log
ceph.git
8 months agoMerge pull request #60532 from joscollin/wip-mirroring-create-snapshot
Anthony D'Atri [Tue, 29 Oct 2024 15:13:48 +0000 (11:13 -0400)]
Merge pull request #60532 from joscollin/wip-mirroring-create-snapshot

doc: mention how to initiate snapshot mirroring

8 months agoMerge PR #60010 into main
Patrick Donnelly [Tue, 29 Oct 2024 12:53:41 +0000 (08:53 -0400)]
Merge PR #60010 into main

* refs/pull/60010/head:
script/ceph-debug-docker: add debuginfod for ubuntu
script/ceph-debug-docker: drop /ceph mount

Reviewed-by: Brad Hubbard <bhubbard@redhat.com>
8 months agoMerge pull request #60508 from zdover23/wip-doc-2024-10-27-rados-operations-erasure...
Zac Dover [Tue, 29 Oct 2024 12:51:07 +0000 (22:51 +1000)]
Merge pull request #60508 from zdover23/wip-doc-2024-10-27-rados-operations-erasure-code-jerasure

doc/rados: add blaum_roth coding guidance

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
8 months agoMerge pull request #60510 from Matan-B/wip-matanb-backfill-crimson-docs
Matan Breizman [Tue, 29 Oct 2024 09:44:58 +0000 (11:44 +0200)]
Merge pull request #60510 from Matan-B/wip-matanb-backfill-crimson-docs

doc/dev/crimson: Add BackfillMachine state graph

Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
8 months agodoc: mention how to initiate snapshot mirroring 60532/head
Jos Collin [Tue, 29 Oct 2024 09:03:10 +0000 (14:33 +0530)]
doc: mention how to initiate snapshot mirroring

Signed-off-by: Jos Collin <jcollin@redhat.com>
8 months agoMerge pull request #60054 from rhcs-dashboard/realm-popup
afreen23 [Tue, 29 Oct 2024 08:14:53 +0000 (13:44 +0530)]
Merge pull request #60054 from rhcs-dashboard/realm-popup

mgr/dashboard: Carbon Design - Create realm popup

Reviewed-by: Afreen Misbah <afreen@ibm.com>
8 months agoMerge pull request #60393 from rhcs-dashboard/nvmeofServiceUnitTest
afreen23 [Tue, 29 Oct 2024 07:05:15 +0000 (12:35 +0530)]
Merge pull request #60393 from rhcs-dashboard/nvmeofServiceUnitTest

Add API tests for nvmeof service.ts file

Reviewed-by: Afreen Misbah <afreen@ibm.com>
8 months agoMerge pull request #60476 from rhcs-dashboard/usermanagment-roles-defect
afreen23 [Tue, 29 Oct 2024 06:43:25 +0000 (12:13 +0530)]
Merge pull request #60476 from rhcs-dashboard/usermanagment-roles-defect

mgr/dashboard:Roles section is not aligned with column name

Reviewed-by: Afreen Misbah <afreen@ibm.com>
8 months agoMerge pull request #60318 from ronen-fr/wip-rf-schedlogl
Ronen Friedman [Tue, 29 Oct 2024 05:47:46 +0000 (07:47 +0200)]
Merge pull request #60318 from ronen-fr/wip-rf-schedlogl

osd/scrub: improve scrub information conveyed in standard

Reviewed-by: Aishwarya Mathuria <amathuri@redhat.com>
Reviewed-by: Samuel Just <sjust@redhat.com>
8 months agoMerge pull request #60470 from joscollin/wip-doc-cephfs-mirroring-improve-note
Zac Dover [Tue, 29 Oct 2024 05:36:06 +0000 (15:36 +1000)]
Merge pull request #60470 from joscollin/wip-doc-cephfs-mirroring-improve-note

doc: explain the consequence of enabling mirroring through monitor commands

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Reviewed-by: Zac Dover <zac.dover@proton.me>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
8 months agoMerge PR #60400 into main
Venky Shankar [Tue, 29 Oct 2024 04:48:50 +0000 (10:18 +0530)]
Merge PR #60400 into main

* refs/pull/60400/head:
mon,cephfs: fix indentation level of a code block

Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
8 months agodoc/dev/crimson: update index 60510/head
Matan Breizman [Sun, 27 Oct 2024 16:32:58 +0000 (16:32 +0000)]
doc/dev/crimson: update index

Signed-off-by: Matan Breizman <mbreizma@redhat.com>
8 months agodoc/dev/crimson: Add initial BackfillMachine docs
Matan Breizman [Sun, 27 Oct 2024 14:24:29 +0000 (14:24 +0000)]
doc/dev/crimson: Add initial BackfillMachine docs

Signed-off-by: Matan Breizman <mbreizma@redhat.com>
8 months agoMerge pull request #60190 from avanthakkar/teuthology-smb-cluster-public-addrs
Adam King [Mon, 28 Oct 2024 17:53:27 +0000 (13:53 -0400)]
Merge pull request #60190 from avanthakkar/teuthology-smb-cluster-public-addrs

qa/suites/orch: add a tests for testing public_addr for smb cluster create cmd

Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: John Mulligan <jmulligan@redhat.com>
8 months agoMerge pull request #59465 from galsalomon66/limit_mem_usage_on_parquet_flow
Gal Salomon [Mon, 28 Oct 2024 09:21:58 +0000 (11:21 +0200)]
Merge pull request #59465 from galsalomon66/limit_mem_usage_on_parquet_flow

rgw/s3select: limit memory usage on Parquet flow

8 months agoMerge pull request #60454 from batrick/mds-warn-cleanup
Venky Shankar [Mon, 28 Oct 2024 05:31:32 +0000 (11:01 +0530)]
Merge pull request #60454 from batrick/mds-warn-cleanup

mds: remove obsolete comments

Reviewed-by: Venky Shankar <vshankar@redhat.com>
8 months agodoc/rados: add blaum_roth coding guidance 60508/head
Zac Dover [Sun, 27 Oct 2024 12:04:16 +0000 (22:04 +1000)]
doc/rados: add blaum_roth coding guidance

Direct Ceph administrators using blaum_roth coding for erasure-coded
pools to change the default value of w=7 to a different value in order
to ensure that w+1 is prime.

This information was provided to the Ceph upstream by Benjamin Mare in
September of 2024.

Signed-off-by: Zac Dover <zac.dover@proton.me>
8 months agoMerge pull request #60357 from ronen-fr/wip-rf-transitive
Ronen Friedman [Sun, 27 Oct 2024 11:08:14 +0000 (13:08 +0200)]
Merge pull request #60357 from ronen-fr/wip-rf-transitive

osd/scrub: make sched-targets comparator transitive

Reviewed-by: Samuel Just <sjust@redhat.com>
8 months agoosd/scrub: improve scrub information conveyed in standard 60318/head
Ronen Friedman [Tue, 15 Oct 2024 11:38:06 +0000 (06:38 -0500)]
osd/scrub: improve scrub information conveyed in standard
PG log line references

When "mentioning" a PG in a log message, we include a set of data
items, including some scrub related information. This
commit improves the scrubber information conveyed, following
changes to the scrub scheduler.

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
8 months agoMerge pull request #60506 from zdover23/2024-10-26-fix-grammar-check_license_name...
Zac Dover [Sat, 26 Oct 2024 23:19:00 +0000 (09:19 +1000)]
Merge pull request #60506 from zdover23/2024-10-26-fix-grammar-check_license_name-main

.github/check-license: correct grammar in "license check"

Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
8 months ago.github/check-license: correct grammar in "license check" 60506/head
Zac Dover [Sat, 26 Oct 2024 13:06:05 +0000 (23:06 +1000)]
.github/check-license: correct grammar in "license check"

Change "Check Incompatible Licenses" to "Check for Incompatible
Licenses".

This commit is made as part of a project to separate the changes in
https://github.com/ceph/ceph/pull/60448 into individual pull requests.

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
8 months agoMerge pull request #60434 from zdover23/wip-doc-2024-10-23-glossary-plp
Zac Dover [Sat, 26 Oct 2024 12:59:01 +0000 (22:59 +1000)]
Merge pull request #60434 from zdover23/wip-doc-2024-10-23-glossary-plp

doc/glossary: add "PLP" to glossary

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
8 months agoMerge pull request #60444 from zdover23/wip-doc-2024-10-23-rados-ops-health-checks
Zac Dover [Sat, 26 Oct 2024 04:03:23 +0000 (14:03 +1000)]
Merge pull request #60444 from zdover23/wip-doc-2024-10-23-rados-ops-health-checks

doc/rados: standardize markup of "clean"

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
8 months agoMerge pull request #60471 from Jayaprakash-ibm/patch-2
Neha Ojha [Fri, 25 Oct 2024 18:40:37 +0000 (11:40 -0700)]
Merge pull request #60471 from Jayaprakash-ibm/patch-2

Update .githubmap

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
8 months agoMerge PR #60386 into main
Patrick Donnelly [Fri, 25 Oct 2024 16:21:34 +0000 (12:21 -0400)]
Merge PR #60386 into main

* refs/pull/60386/head:
qa/cephfs: override testing kernel with -k option

Reviewed-by: Venky Shankar <vshankar@redhat.com>
8 months agoqa/suites/orch: add a tests for testing public_addr for smb cluster create cmd 60190/head
Avan Thakkar [Tue, 8 Oct 2024 10:41:12 +0000 (16:11 +0530)]
qa/suites/orch: add a tests for testing public_addr for smb cluster create cmd

Signed-off-by: Avan Thakkar <athakkar@redhat.com>
8 months agoMerge pull request #60117 from rishabh-d-dave/mgr-vol-base-init
Rishabh Dave [Fri, 25 Oct 2024 13:10:59 +0000 (18:40 +0530)]
Merge pull request #60117 from rishabh-d-dave/mgr-vol-base-init

mgr/vol: better to call base class __init__() at beginning

Reviewed-by: Milind Changire <mchangir@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
8 months agoMerge pull request #60153 from rishabh-d-dave/mgr-vol-cv-waiting
Rishabh Dave [Fri, 25 Oct 2024 13:10:19 +0000 (18:40 +0530)]
Merge pull request #60153 from rishabh-d-dave/mgr-vol-cv-waiting

mgr/vol: remove unused variable

Reviewed-by: Jos Collin <jcollin@redhat.com>
8 months agomgr/dashboard: Carbon Design - Create realm popup 60054/head
Dnyaneshwari [Tue, 8 Oct 2024 07:18:57 +0000 (12:48 +0530)]
mgr/dashboard: Carbon Design - Create realm popup

Fixes: https://tracker.ceph.com/issues/68341
Signed-off-by: Dnyaneshwari Talwekar <dtalweka@redhat.com>
8 months agoMerge pull request #60395 from guits/replace-osd-enhancement
Guillaume Abrioux [Fri, 25 Oct 2024 11:45:20 +0000 (13:45 +0200)]
Merge pull request #60395 from guits/replace-osd-enhancement

ceph-volume: support zapping by osd-id for RAW OSDs

8 months agoceph-volume: support zapping by osd-id for RAW OSDs 60395/head
Guillaume Abrioux [Fri, 18 Oct 2024 08:56:32 +0000 (08:56 +0000)]
ceph-volume: support zapping by osd-id for RAW OSDs

Currently it is only possible to zap by osd-id for LVM-based OSDs.

This commit introduces the required changes in order to support zapping
by osd-id for RAW-based OSDs.

Fixes: https://tracker.ceph.com/issues/68576
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
8 months agoMerge pull request #59847 from rishabh-d-dave/mgr-vol-spawn-threads
Rishabh Dave [Fri, 25 Oct 2024 06:28:38 +0000 (11:58 +0530)]
Merge pull request #59847 from rishabh-d-dave/mgr-vol-spawn-threads

mgr/vol: reuse code to spawn threads

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
8 months agoAdd API tests for nvmeof service.ts file 60393/head
Achint Kaur [Fri, 18 Oct 2024 12:11:38 +0000 (17:41 +0530)]
Add API tests for nvmeof service.ts file

Fixes: https://tracker.ceph.com/issues/68551
Signed-off-by: Achint Kaur <ackaur@redhat.com>
8 months agoMerge pull request #60480 from truongsinh/patch-2
Anthony D'Atri [Fri, 25 Oct 2024 01:47:29 +0000 (21:47 -0400)]
Merge pull request #60480 from truongsinh/patch-2

doc: Note that to set layouts or quotas, clients require the `p` flag

8 months agodoc/cephfs: use 'p' flag to set layouts or quotas 60480/head
TruongSinh Tran-Nguyen [Thu, 24 Oct 2024 20:57:49 +0000 (13:57 -0700)]
doc/cephfs: use 'p' flag to set layouts or quotas

Remind users that clients must use the 'p' flag when writing file
layouts.

This PR is made in response to
https://www.reddit.com/r/ceph/comments/1gbad51/comment/ltkgd5v/

Co-authored-by: Zac Dover <zac.dover@proton.me>
Signed-off-by: TruongSinh Tran-Nguyen <i@truongsinh.pro>
8 months agomgr/dashboard:Roles section is not aligned with column name 60476/head
pujaoshahu [Thu, 24 Oct 2024 14:10:45 +0000 (19:40 +0530)]
mgr/dashboard:Roles section is not aligned with column name

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

Signed-off-by:Puja Shahu pshahu@redhat.com
Signed-off-by: pujaoshahu <pshahu@redhat.com>
8 months agodoc: explain the consequence of enabling mirroring through monitor commands 60470/head
Jos Collin [Thu, 24 Oct 2024 06:03:19 +0000 (11:33 +0530)]
doc: explain the consequence of enabling mirroring through monitor commands

Signed-off-by: Jos Collin <jcollin@redhat.com>
8 months agoMerge pull request #59982 from rkachach/fix_issue_mgmt_gw_high_availability
Adam King [Thu, 24 Oct 2024 14:45:42 +0000 (10:45 -0400)]
Merge pull request #59982 from rkachach/fix_issue_mgmt_gw_high_availability

Adding HA support for mgmt-gateway and oauth2-proxy services

Reviewed-by: Adam king <adking@redhat.com>
Reviewed-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
Reviewed-by: Juan Miguel Olmo Martínez <jolmomar@redhat.com>
8 months agoMerge pull request #58898 from neesingh-rh/wip-fix-strict-iec-cast
Rishabh Dave [Thu, 24 Oct 2024 14:16:48 +0000 (19:46 +0530)]
Merge pull request #58898 from neesingh-rh/wip-fix-strict-iec-cast

src/common : proper handling of units in `strict_iec_cast`

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Dhairya Parmar <dparmar@redhat.com>
8 months agomgr/vol: add helpers to spawn all threads and more threads 59847/head
Rishabh Dave [Wed, 18 Sep 2024 05:20:01 +0000 (10:50 +0530)]
mgr/vol: add helpers to spawn all threads and more threads

Also add log messages for in these helper methods to allow tracking when
and why more threads were spawned.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
8 months agomgr/vol: reuse code for spawning threads
Rishabh Dave [Mon, 16 Sep 2024 09:16:45 +0000 (14:46 +0530)]
mgr/vol: reuse code for spawning threads

Signed-off-by: Rishabh Dave <ridave@redhat.com>
8 months agoMerge PR #60473 into main
Patrick Donnelly [Thu, 24 Oct 2024 12:34:54 +0000 (08:34 -0400)]
Merge PR #60473 into main

* refs/pull/60473/head:
mailmap, githubmap, organisationmap: Add Anuradha Gadge

Reviewed-by: Adam King <adking@redhat.com>
8 months agoMerge PR #60457 into main
Patrick Donnelly [Thu, 24 Oct 2024 12:32:46 +0000 (08:32 -0400)]
Merge PR #60457 into main

* refs/pull/60457/head:
.github/check-license: add friendly name

Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
8 months agoMerge pull request #59002 from neesingh-rh/wip-65820
Rishabh Dave [Thu, 24 Oct 2024 11:27:17 +0000 (16:57 +0530)]
Merge pull request #59002 from neesingh-rh/wip-65820

qa/tasks/fwd_scrub: remove unnecessary traceback

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
8 months agoMerge pull request #60119 from rishabh-d-dave/mgr-vol-comments
Rishabh Dave [Thu, 24 Oct 2024 11:24:50 +0000 (16:54 +0530)]
Merge pull request #60119 from rishabh-d-dave/mgr-vol-comments

mgr/vol: add comments to explain queuing data structures

Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Jos Collin <jcollin@redhat.com>
8 months agoMerge pull request #60042 from sunilangadi2/migration_with_namespace_test
Ilya Dryomov [Thu, 24 Oct 2024 10:02:09 +0000 (12:02 +0200)]
Merge pull request #60042 from sunilangadi2/migration_with_namespace_test

qa/workunits/rbd: cover native migration between namespaces

Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
8 months agomailmap, githubmap, organisationmap: Add Anuradha Gadge 60473/head
Anuradha Gadge [Thu, 24 Oct 2024 07:06:58 +0000 (12:36 +0530)]
mailmap, githubmap, organisationmap: Add Anuradha Gadge

Signed-off-by: Anuradha Gadge <anuradha.gadge@ibm.com>
8 months agoMerge pull request #60420 from nbalacha/wip-nbalacha-namespace-doc
Ilya Dryomov [Thu, 24 Oct 2024 07:12:30 +0000 (09:12 +0200)]
Merge pull request #60420 from nbalacha/wip-nbalacha-namespace-doc

doc/rbd: namespace mirroring updates

Reviewed-by: Sunil Angadi <Sunil.Angadi@ibm.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
8 months agoUpdate .githubmap 60471/head
Jayaprakash-ibm [Thu, 24 Oct 2024 06:45:24 +0000 (12:15 +0530)]
Update .githubmap

Signed-off-by: Jayaprakash-ibm <jayaprakash@ibm.com>
8 months agoMerge pull request #60355 from rhcs-dashboard/carbonize-hosts-form
afreen23 [Thu, 24 Oct 2024 06:41:07 +0000 (12:11 +0530)]
Merge pull request #60355 from rhcs-dashboard/carbonize-hosts-form

mgr/dashboard: carbonize Cluster > Hosts form

Reviewed-by: Afreen Misbah <afreen@ibm.com>
8 months agoMerge pull request #60465 from zdover23/wip-doc-2024-10-24-cephfs-troubleshooting
Zac Dover [Thu, 24 Oct 2024 05:27:17 +0000 (15:27 +1000)]
Merge pull request #60465 from zdover23/wip-doc-2024-10-24-cephfs-troubleshooting

doc/cephfs: edit "Disabling Volumes Plugin"

Reviewed-by: Jos Collin <jcollin@redhat.com>
8 months agodoc/cephfs: edit "Disabling Volumes Plugin" 60465/head
Zac Dover [Thu, 24 Oct 2024 04:52:58 +0000 (14:52 +1000)]
doc/cephfs: edit "Disabling Volumes Plugin"

Improve the English language in the section 'Disabling Volumes Plugin"
(which here becomes "Disabling the Volumes Plugin") in the
doc/cephfs/troubleshooting.rst file.

The change here is similar to a change introduced in
https://github.com/ceph/ceph/pull/60448/. That PR seems to be unable to
pass the "make check", and this PR is part of a test to see if breaking
the changes in that PR into several smaller changes results in mergeable
PRs.

Signed-off-by: Zac Dover <zac.dover@proton.me>
8 months agoMerge pull request #59027 from hit1943/65971_fix
Venky Shankar [Thu, 24 Oct 2024 04:24:11 +0000 (09:54 +0530)]
Merge pull request #59027 from hit1943/65971_fix

client: fixed a bug that read operation hung

Reviewed-by: Venky Shankar <vshankar@redhat.com>
8 months agodoc/rbd : add namespace mirroring updates 60420/head
N Balachandran [Tue, 22 Oct 2024 04:47:00 +0000 (10:17 +0530)]
doc/rbd : add namespace mirroring updates

Updates the rbd docs with the new remote-namespace
mirroring configuration option.

Signed-off-by: N Balachandran <nibalach@redhat.com>
8 months agoMerge pull request #60276 from ceph/bypass_sr
Yingxin [Thu, 24 Oct 2024 01:29:11 +0000 (09:29 +0800)]
Merge pull request #60276 from ceph/bypass_sr

test/crimson: disable stall-detector on aarch64 to fix asan failures

Reviewed-by: Kefu Chai <tchaikov@gmail.com>
Reviewed-by: Matan Breizman <mbreizma@redhat.com>
Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
8 months agoMerge pull request #60460 from zdover23/wip-doc-2024-10-24-start-beginners-guide
Zac Dover [Thu, 24 Oct 2024 00:19:40 +0000 (10:19 +1000)]
Merge pull request #60460 from zdover23/wip-doc-2024-10-24-start-beginners-guide

doc/start: add vstart install guide

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
8 months agodoc/start: add vstart install guide 60460/head
Zac Dover [Wed, 23 Oct 2024 23:04:08 +0000 (09:04 +1000)]
doc/start: add vstart install guide

Add "vstart Cluster Installation and Configuration Procedure" to the
Beginner's Guide.

Signed-off-by: Zac Dover <zac.dover@proton.me>
8 months agoMerge pull request #60413 from gbregman/main
Gil Bregman [Wed, 23 Oct 2024 19:54:01 +0000 (22:54 +0300)]
Merge pull request #60413 from gbregman/main

 Add namespace netmask parameters to nvmeof configuration

8 months ago.github/check-license: add friendly name 60457/head
Ernesto Puerta [Wed, 23 Oct 2024 18:34:40 +0000 (20:34 +0200)]
.github/check-license: add friendly name

In the absence of `name`, GH uses the `job` ID of the action to select which checks are required, and `pull_request` is rather confusing.

Signed-off-by: Ernesto Puerta <37327689+epuertat@users.noreply.github.com>
8 months agomgr/dashboard: carbonize Cluster > Hosts form 60355/head
Dnyaneshwari [Wed, 16 Oct 2024 11:03:19 +0000 (16:33 +0530)]
mgr/dashboard: carbonize Cluster > Hosts form

Fixes: https://tracker.ceph.com/issues/68264
Signed-off-by: Dnyaneshwari Talwekar <dtalweka@redbat.com>
8 months agomds: remove obsolete comments 60454/head
Patrick Donnelly [Mon, 23 Sep 2024 14:54:42 +0000 (10:54 -0400)]
mds: remove obsolete comments

An mdr is a smart pointer now. The reference is now codified.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
8 months agoMerge pull request #60399 from rhcs-dashboard/rgw-bucket-get-fix-for-account
afreen23 [Wed, 23 Oct 2024 15:58:03 +0000 (21:28 +0530)]
Merge pull request #60399 from rhcs-dashboard/rgw-bucket-get-fix-for-account

mgr/dashboard: fix bucket get for s3 account owned bucket

Reviewed-by: Afreen Misbah <afreen@ibm.com>
8 months agoMerge pull request #60388 from ceph/fix-check_license_typo-main
Ernesto Puerta [Wed, 23 Oct 2024 15:12:35 +0000 (16:12 +0100)]
Merge pull request #60388 from ceph/fix-check_license_typo-main

ci: fix typo in check-license

8 months agoadded tests migration with namespace 60042/head
Sunil Angadi [Sun, 29 Sep 2024 09:56:12 +0000 (15:26 +0530)]
added tests migration with namespace

Signed-off-by: Sunil Angadi <Sunil.Angadi@ibm.com>
8 months agoMerge pull request #58647 from rishabh-d-dave/mgr-vol-mod-disable
Hemanth [Wed, 23 Oct 2024 12:27:50 +0000 (17:57 +0530)]
Merge pull request #58647 from rishabh-d-dave/mgr-vol-mod-disable

mgr: allow disabling always-on modules

8 months agoMerge pull request #60316 from VallariAg/wip-nvmeof-teuthology-mtls-test
Vallari Agrawal [Wed, 23 Oct 2024 11:58:53 +0000 (17:28 +0530)]
Merge pull request #60316 from VallariAg/wip-nvmeof-teuthology-mtls-test

qa/suites/nvmeof: add mtls test

8 months agoMerge pull request #60157 from soumyakoduri/wip-skoduri-lc-nullinstance
Soumya Koduri [Wed, 23 Oct 2024 11:52:08 +0000 (17:22 +0530)]
Merge pull request #60157 from soumyakoduri/wip-skoduri-lc-nullinstance

rgw/lc: Fix issues with non-current objects with instance empty

Reviewed-by: Casey Bodley <cbodley@redhat.com>
8 months agoMerge pull request #60258 from aclamk/wip-aclamk-cbt-improve-show-label
Adam Kupczyk [Wed, 23 Oct 2024 11:38:15 +0000 (13:38 +0200)]
Merge pull request #60258 from aclamk/wip-aclamk-cbt-improve-show-label

os/bluestore/ceph-bluestore-tool: Modify show-label for many devs

8 months agodoc/glossary: add "PLP" to glossary 60434/head
Zac Dover [Wed, 23 Oct 2024 00:50:25 +0000 (10:50 +1000)]
doc/glossary: add "PLP" to glossary

Add "PLP" to the glossary.

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
8 months agodoc/rados: standardize markup of "clean" 60444/head
Zac Dover [Wed, 23 Oct 2024 08:34:25 +0000 (18:34 +1000)]
doc/rados: standardize markup of "clean"

Standardize the markup around the status "clean" in the documentation so
that readers don't mistakenly get the idea that inconsistent
presentation of the word "clean" implies a never-stated difference
between one instance and the other.

Signed-off-by: Zac Dover <zac.dover@proton.me>
8 months agotest/crimson: disable stall-detector on aarch64 to fix asan failures 60276/head
cailianchun [Fri, 18 Oct 2024 02:39:46 +0000 (02:39 +0000)]
test/crimson: disable stall-detector on aarch64 to fix asan failures

Ceph crimson uses seastar on debug mode which would enable ASAN, while stall-detector uses glibc backtrace function which would cause ASAN failures on aarch64.
Reason see scylladb/scylladb#15090 (comment)

Because arm ci servers in lab are "elderly", causing stall happened often, this PR is to disable stall-detector until seastar upstream migrated to libunwind, see scylladb/seastar#1878
Seastar does not provide a function to disable stall-detector, at present, increase stall-detector time to avoid stall-detector timeout on arm platform.

fixes: https://tracker.ceph.com/issues/65635

Signed-off-by: cailianchun <arm7star@qq.com>
8 months agoMerge pull request #60433 from indirasawant/doc_fix_1
Anthony D'Atri [Tue, 22 Oct 2024 22:48:06 +0000 (18:48 -0400)]
Merge pull request #60433 from indirasawant/doc_fix_1

doc/cephfs: rearrange subvolume group information

8 months agodoc/cephfs: rearrange subvolume group information 60433/head
Indira Sawant [Tue, 22 Oct 2024 21:45:10 +0000 (16:45 -0500)]
doc/cephfs: rearrange subvolume group information

Rearranging the documentation to align it with CephFS encapsulation design.

Signed-off-by: Indira Sawant <indira.sawant@ibm.com>
8 months agoMerge pull request #60423 from zdover23/wip-doc-2024-10-22-cephadm-services-smb
Zac Dover [Tue, 22 Oct 2024 20:17:21 +0000 (06:17 +1000)]
Merge pull request #60423 from zdover23/wip-doc-2024-10-22-cephadm-services-smb

doc/cephadm: correct "ceph orch apply" command

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
8 months agomgr/dashboard: fix bucket get for s3 account owned bucket 60399/head
Nizamudeen A [Fri, 18 Oct 2024 15:20:33 +0000 (20:50 +0530)]
mgr/dashboard: fix bucket get for s3 account owned bucket

**Issue 1:**

When a bucket is created with a user that is owner by the account user,
it fails to fetch the info for the bucket because the owner of that
bucket is considered as owned by the account rather than the user.

dashboard api try to fetch some info for the bucket on the basis of the
bucket owner which in this case fails since its not a real user.

To keep the existing behavior as it is and fix the current issue, I am
doing a check to make sure the user id exists or not. if it doesn't
exist, we goes on to fetch using the dashboard user.

**Issue 2**
Editing the bucket owner by the account is broken. So disabling the
ownership change for the bucket owned by account until we have user
account management for rgw in the dashboard

Fixes: https://tracker.ceph.com/issues/68622
Signed-off-by: Nizamudeen A <nia@redhat.com>
8 months agorgw/lc: Fix issues with non-current objects with instance empty 60157/head
Soumya Koduri [Mon, 7 Oct 2024 11:24:02 +0000 (16:54 +0530)]
rgw/lc: Fix issues with non-current objects with instance empty

When the bucket versioning is enabled, old plain object entry is converted
to versioned by updating its instance as "null" in its raw head/old object.
However its instance remains empty in the bi list entry. Same is the case for
the entries created after versioning is suspended and re-enabled.

So to access such objects which are non-current, we need to set rgw_obj_key.instance as
1) "null" to read the actual raw obj and
2) empty while accessing/updating their bi entry.

Fixes: https://tracker.ceph.com/issues/68402
Signed-off-by: Soumya Koduri <skoduri@redhat.com>
8 months agodoc/cephadm: correct "ceph orch apply" command 60423/head
Zac Dover [Tue, 22 Oct 2024 11:30:02 +0000 (21:30 +1000)]
doc/cephadm: correct "ceph orch apply" command

s/orch apply/ceph orch apply/ in a command where "ceph" was missing.

This was reported by Eugen Block to Zac Dover on 22 Oct 2024 through
Slack.

Co-authored-by: Eugen Block <eblock@nde.ag>
Signed-off-by: Zac Dover <zac.dover@proton.me>
8 months agoosd/scrub: remove the 'only_deadlined' flag 60357/head
Ronen Friedman [Wed, 16 Oct 2024 12:26:12 +0000 (07:26 -0500)]
osd/scrub: remove the 'only_deadlined' flag

as we no longer allow overdue targets to ignore time/load
constraints.

Also: fix the documentation for the relevant configuration
options.

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
8 months agoosd/scrub: make sched-targets comparator transitive
Ronen Friedman [Wed, 16 Oct 2024 12:04:38 +0000 (07:04 -0500)]
osd/scrub: make sched-targets comparator transitive

The comparator used to sort the scrub targets wasn't transitive, which
can lead to undefined behavior (and likely - to a crash of the sort
algorithm in some implementations). That - due to the clause comparing
the two targets of the same PG.

This commit fixes the comparator. The sorted queue is no longer
guaranteed to have a ripe deep target before the corresponding
shallow one, which is less than optimal - but it won't crash.

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
8 months agoMerge PR #60106 into main
Patrick Donnelly [Tue, 22 Oct 2024 01:00:21 +0000 (21:00 -0400)]
Merge PR #60106 into main

* refs/pull/60106/head:
msg/async/ProtocolV2: pass `desc` as `std::string_view` to write()

Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
8 months agoMerge PR #60174 into main
Patrick Donnelly [Tue, 22 Oct 2024 00:57:15 +0000 (20:57 -0400)]
Merge PR #60174 into main

* refs/pull/60174/head:
common/Finisher: pass name as std::string_view to ctor
common/Finisher: add method get_thread_name()
mgr/ActivePyModule: build thread name with fmt
mgr/ActivePyModule: return std::string_view instead of std::string copy
common/Finisher: use fmt to build strings
common/Finisher: un-inline ctor and dtor
common/Finisher: add `const` to several fields
common/Finisher: merge duplicate field initializers
common/Finisher: call notify_one() instead of notify_all()
common/Finisher: wake up after pushing to the queue
common/Finisher: do not wake up the thread if already running
common/Finisher: call logger without holding the lock
common/Finisher: use `std::lock_guard` instead of `std::unique_lock`
common/Finisher: merge all queue() container methods into one template

Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
8 months agoMerge PR #60214 into main
Patrick Donnelly [Tue, 22 Oct 2024 00:56:01 +0000 (20:56 -0400)]
Merge PR #60214 into main

* refs/pull/60214/head:
mds/MDCache: use `auto`
mds/CDir: use the erase() return value
mds/MDCache: remove unnecessary empty() check
mds/MDCache: use the erase() return value
mds/MDCache: pass iterator by value

Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
8 months agoMerge PR #60216 into main
Patrick Donnelly [Tue, 22 Oct 2024 00:54:15 +0000 (20:54 -0400)]
Merge PR #60216 into main

* refs/pull/60216/head:
common/options: pass name as rvalue reference
common/config: use libfmt to build strings
common/config: use emplace_back() instead of push_back()
common/HeartbeatMap: pass name as rvalue reference
common/config_obs_mgr: use the erase() return value
common/SloppyCRCMap: use the erase() return value
common: disable `boost::intrusive::constant_time_size`

Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
8 months agoMerge PR #60220 into main
Patrick Donnelly [Tue, 22 Oct 2024 00:53:44 +0000 (20:53 -0400)]
Merge PR #60220 into main

* refs/pull/60220/head:
msg/async/AsyncConnection: move the writeCallback instead of copying it
msg/async/AsyncConnection: do not wrap writeCallback in `std::optional`
msg/async/frames_v2: use zero-initialization instead of memset()
msg/async/Event: use zero-initialization instead of memset()
msg/Message: use zero-initialization instead of memset()
msg/async/ProtocolV2: eliminate redundant std::map lookups
msg/async/ProtocolV[12]: reverse the std::map sort order
msg/async/ProtocolV[12]: use `auto`
msg/async/ProtocolV[12]: use range-based `for`
msg/async/ProtocolV1: use zero-initialization instead of memset()

Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
8 months agoMerge PR #60324 into main
Patrick Donnelly [Tue, 22 Oct 2024 00:51:33 +0000 (20:51 -0400)]
Merge PR #60324 into main

* refs/pull/60324/head:
mds/Beacon: set a thread name

Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
8 months agomgr/cephadm: Add namespace netmask parameters to nvmeof configuration 60413/head
Gil Bregman [Mon, 21 Oct 2024 14:23:22 +0000 (17:23 +0300)]
mgr/cephadm: Add namespace netmask parameters to nvmeof configuration
Fixes https://tracker.ceph.com/issues/68542

Signed-off-by: Gil Bregman <gbregman@il.ibm.com>
8 months agopython-common/ceph/deployment: Add namespace netmask parameters to nvmeof configuration
Gil Bregman [Mon, 21 Oct 2024 14:22:17 +0000 (17:22 +0300)]
python-common/ceph/deployment: Add namespace netmask parameters to nvmeof configuration
Fixes https://tracker.ceph.com/issues/68542

Signed-off-by: Gil Bregman <gbregman@il.ibm.com>
8 months agoremove the code-lines that checking the {ofs,len} bounderies; and modifies them if... 59465/head
Gal Salomon [Mon, 21 Oct 2024 19:12:26 +0000 (22:12 +0300)]
remove the code-lines that checking the {ofs,len} bounderies; and modifies them if necessary(invalid read)

Signed-off-by: Gal Salomon <gal.salomon@gmail.com>
8 months agoMerge pull request #59470 from adk3798/cephadm-staggered-upgrade-test-wait-check...
Adam King [Mon, 21 Oct 2024 18:54:56 +0000 (14:54 -0400)]
Merge pull request #59470 from adk3798/cephadm-staggered-upgrade-test-wait-check-rgw-versions

qa/cephadm: wait a bit before checking rgw daemons upgraded w/ `ceph versions`

Reviewed-by: Guillaume Abrioux <gabrioux@ibm.com>
8 months agoMerge pull request #60225 from MaxKellermann/ceph_context_atomic
Casey Bodley [Mon, 21 Oct 2024 17:07:20 +0000 (13:07 -0400)]
Merge pull request #60225 from MaxKellermann/ceph_context_atomic

common/ceph_context: use std::atomic<std::shared_ptr<T>>

Reviewed-by: Casey Bodley <cbodley@redhat.com>
8 months agoMerge PR #60341 into main
Patrick Donnelly [Mon, 21 Oct 2024 16:20:21 +0000 (12:20 -0400)]
Merge PR #60341 into main

* refs/pull/60341/head:
doc/dev: add walkthrough for CephFS kernel development

Reviewed-by: Anthony D Atri <anthony.datri@gmail.com>
8 months agoqa/cephfs: override testing kernel with -k option 60386/head
Patrick Donnelly [Thu, 17 Oct 2024 19:39:33 +0000 (15:39 -0400)]
qa/cephfs: override testing kernel with -k option

Normally, the fs suite overrides the kernel branch whenever the kernel client
is used, according to the matrix of configs. This prevents easily testing a new
kernel with the -k option to teuthology-suite. So, using the base config passed
to the script, detect if an alternate testing kernel is desired and use that
instead.

The scheduler still needs to do some work:

    $ teuthology-suite ... --verbose -k wip-pdonnell-i66704 --filter k-testing
    ...
    2024-10-18 00:29:43,875.875 DEBUG:teuthology.suite.merge:base kernel {'branch': 'wip-pdonnell-i66704', 'kdb': 1, 'sha1': '745cacd8f31e50d7f3b6039bbd8c9a8dfc07bf03', 'flavor': 'default'}
    2024-10-18 00:29:43,875.875 DEBUG:teuthology.suite.merge:overriding testing kernel with {'branch': 'wip-pdonnell-i66704', 'kdb': 1, 'sha1': '745cacd8f31e50d7f3b6039bbd8c9a8dfc07bf03', 'flavor': 'default'}

vs.

    $ teuthology-suite ... --verbose --filter k-testing
    ...
    2024-10-17 20:04:52,265.265 DEBUG:teuthology.suite.merge:base kernel {'branch': 'distro', 'kdb': 1, 'sha1': 'distro'}

Which will only select jobs using the "testing" kernel.

See-also: https://github.com/ceph/teuthology/pull/2008
Fixes: https://tracker.ceph.com/issues/68603
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
8 months agoMerge pull request #60045 from AliMasarweh/wip-alimasa-notif-list-bucket
Ali Masarwa [Mon, 21 Oct 2024 13:34:57 +0000 (16:34 +0300)]
Merge pull request #60045 from AliMasarweh/wip-alimasa-notif-list-bucket

RGW|Bucket notification: fix for v2 topics rgw-admin list operation

Reviewed-by: yuvalif<ylifshit@redhat.com>
8 months agoMerge pull request #56849 from frittentheke/issue_64321_alerts
afreen23 [Mon, 21 Oct 2024 12:09:56 +0000 (17:39 +0530)]
Merge pull request #56849 from frittentheke/issue_64321_alerts

Add multi-cluster support (showMultiCluster=True) to alerts

Reviewed-by: Afreen Misbah <afreen@ibm.com>
8 months agoMerge pull request #60401 from rhcs-dashboard/cephfs
afreen23 [Mon, 21 Oct 2024 11:57:58 +0000 (17:27 +0530)]
Merge pull request #60401 from rhcs-dashboard/cephfs

mgr/dashboard: Removing duplicate cephfs constant file & importing correct default_subvolumegroup.

Reviewed-by: Afreen Misbah <afreen@ibm.com>
8 months agoMerge pull request #59926 from JonBailey1993/JonBailey1993/ceph_test_rados_io_sequenc...
Jon Bailey [Mon, 21 Oct 2024 08:50:44 +0000 (09:50 +0100)]
Merge pull request #59926 from JonBailey1993/JonBailey1993/ceph_test_rados_io_sequence_interactive

test/osd: Add interactive mode to ceph_test_rados_io_sequence

Reviewed-by: Nitzan Mordechai <nmordech@redhat.com>
8 months agoRGW|Bucket notification: fix for v2 topics rgw-admin list operation 60045/head
Ali Masarwa [Sun, 29 Sep 2024 12:00:13 +0000 (15:00 +0300)]
RGW|Bucket notification: fix for v2 topics rgw-admin list operation

Signed-off-by: Ali Masarwa <amasarwa@redhat.com>
9 months agoAdd multi-cluster support (showMultiCluster=True) to alerts 56849/head
Christian Rohmann [Fri, 26 Jan 2024 14:05:52 +0000 (15:05 +0100)]
Add multi-cluster support (showMultiCluster=True) to alerts

Following PR https://github.com/ceph/ceph/pull/55495 fixing the
dashboard in regards to multiple clusters storing their metrics
in a single Prometheus instance, this PR addresses the issues
for alerts.

Fixes: https://tracker.ceph.com/issues/64321
Signed-off-by: Christian Rohmann <christian.rohmann@inovex.de>
9 months agoqa/suites/nvmeof: add mtls test 60316/head
Vallari Agrawal [Tue, 15 Oct 2024 10:57:37 +0000 (16:27 +0530)]
qa/suites/nvmeof: add mtls test

Add qa/workunits/nvmeof/mtls_test.sh which enables
mtls config and redeploy, then verify and disables
mtls config.

Signed-off-by: Vallari Agrawal <vallari.agrawal@ibm.com>
9 months agomgr/vol: add comments to explain queuing data structures 60119/head
Rishabh Dave [Tue, 27 Aug 2024 08:14:31 +0000 (13:44 +0530)]
mgr/vol: add comments to explain queuing data structures

Add some comment to explains the purpose and details of data structures
that are used for queuing of async jobs since right now it's not obvious
from initialization.

Signed-off-by: Rishabh Dave <ridave@redhat.com>