]> git.apps.os.sepia.ceph.com Git - ceph.git/log
ceph.git
7 months agodoc/dev: instruct devs to backport 60955/head
Zac Dover [Wed, 4 Dec 2024 20:43:12 +0000 (21:43 +0100)]
doc/dev: instruct devs to backport

Add a note to doc/dec/development-workflow.rst that instructs developers
to do their own backports. This change was requested by Laura Flores on
04 Dec 2024.

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
7 months agoMerge pull request #61049 from zdover23/wip-doc-2024-12-12-mount-using-kernel-driver...
Zac Dover [Thu, 12 Dec 2024 02:23:04 +0000 (03:23 +0100)]
Merge pull request #61049 from zdover23/wip-doc-2024-12-12-mount-using-kernel-driver-main-2

doc/cephfs: edit 2nd 3rd of mount-using-kernel-driver

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
7 months agoMerge pull request #61048 from zdover23/wip-doc-2024-12-12-mount-using-kernel-driver...
Zac Dover [Thu, 12 Dec 2024 02:14:51 +0000 (03:14 +0100)]
Merge pull request #61048 from zdover23/wip-doc-2024-12-12-mount-using-kernel-driver-main

doc/cephfs: edit first 3rd of mount-using-kernel-driver

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
7 months agodoc/cephfs: edit 2nd 3rd of mount-using-kernel-driver 61049/head
Zac Dover [Wed, 11 Dec 2024 21:17:40 +0000 (07:17 +1000)]
doc/cephfs: edit 2nd 3rd of mount-using-kernel-driver

Edit the second third of doc/cephfs/mount-using-kernel-driver.rst in
preparation for correcting mount commands that may not work in Reef as
described in this documentation.

This commit edits only English-language strings in
doc/cephfs/mount-using-kernel-driver.rst. No technical content (that is,
no commands and no settings) have been altered in this commit.

Technical alterations to this file will be made only after the English
is unambiguous.

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
7 months agoMerge pull request #60907 from shashalu/wip-fix-stale-bucket-indexes
J. Eric Ivancich [Wed, 11 Dec 2024 22:58:28 +0000 (17:58 -0500)]
Merge pull request #60907 from shashalu/wip-fix-stale-bucket-indexes

rgw: Delete stale entries in bucket indexes while deleting obj

Reviewed-by: Casey Bodley <cbodley@redhat.com>
7 months agoMerge pull request #60916 from shashalu/fix-data-corruption-ETIMEDOUT
J. Eric Ivancich [Wed, 11 Dec 2024 22:57:03 +0000 (17:57 -0500)]
Merge pull request #60916 from shashalu/fix-data-corruption-ETIMEDOUT

rgw: fix data corruption when rados op return ETIMEDOUT

Reviewed-by: Casey Bodley <cbodley@redhat.com>
7 months agoMerge pull request #60845 from yehudasa/wip-69060
Casey Bodley [Wed, 11 Dec 2024 21:49:03 +0000 (16:49 -0500)]
Merge pull request #60845 from yehudasa/wip-69060

rgw: fix bucket link operation

Reviewed-by: Casey Bodley <cbodley@redhat.com>
7 months agodoc/cephfs: edit first 3rd of mount-using-kernel-driver 61048/head
Zac Dover [Wed, 11 Dec 2024 14:15:14 +0000 (15:15 +0100)]
doc/cephfs: edit first 3rd of mount-using-kernel-driver

Edit the first third of doc/cephfs/mount-using-kernel-driver.rst in
preparation for correcting mount commands that may not work in Reef as
described in this documentation.

This commit is a cherry-pick from a branch that targeted the Reef
release branch. After some thought I realized that there was no reason
that the Engliish grammar shouldn't be clean in this branch too.

Signed-off-by: Zac Dover <zac.dover@proton.me>
7 months agoMerge pull request #60851 from ivancich/wip-optimize-listing-over-namespaces
J. Eric Ivancich [Wed, 11 Dec 2024 20:32:10 +0000 (15:32 -0500)]
Merge pull request #60851 from ivancich/wip-optimize-listing-over-namespaces

rgw: optimize bucket listing to skip past regions of namespaced entries

Reviewed-by: Casey Bodley <cbodley@redhat.com>
7 months agoMerge pull request #61010 from ivancich/wip-fix-s3java
J. Eric Ivancich [Wed, 11 Dec 2024 20:30:15 +0000 (15:30 -0500)]
Merge pull request #61010 from ivancich/wip-fix-s3java

qa/rgw: fix s3 java tests by forcing gradle to run on Java 8

Reviewed-by: Casey Bodley <cbodley@redhat.com>
7 months agoMerge pull request #61002 from ebocchi/ebocchi-fix-text-for-17.2.8-release 61044/head
Laura Flores [Wed, 11 Dec 2024 16:31:34 +0000 (10:31 -0600)]
Merge pull request #61002 from ebocchi/ebocchi-fix-text-for-17.2.8-release

doc/releases: fix formatting for the 17.2.8 release (el8 CERN)

7 months agoqa/rgw: force s3 java tests to run gradle on Java 8 61010/head
J. Eric Ivancich [Mon, 9 Dec 2024 22:23:59 +0000 (17:23 -0500)]
qa/rgw: force s3 java tests to run gradle on Java 8

Previously gradle would run using the default Java version. This looks
for Java 8 using `alternatives` and sets JAVA_HOME to the
corresponding directory prior to launching gradle.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>
7 months agoMerge pull request #60749 from rhcs-dashboard/warn_feature_toggle
afreen23 [Wed, 11 Dec 2024 09:23:43 +0000 (14:53 +0530)]
Merge pull request #60749 from rhcs-dashboard/warn_feature_toggle

mgr/dashboard: add a custom warning message when enabling feature

Reviewed-by: Afreen Misbah <afreen@ibm.com>
7 months agomgr/dashboard: add a custom warning message when enabling feature 60749/head
Nizamudeen A [Fri, 15 Nov 2024 07:20:07 +0000 (12:50 +0530)]
mgr/dashboard: add a custom warning message when enabling feature

would be helpful if we want to notify user about certain issues when
enabling something.

```
[root@ceph ceph]# ceph dashboard feature enable iscsi

WARNING: iscsi warning message
Feature 'iscsi': enabled
```

Fixes: https://tracker.ceph.com/issues/68969
Signed-off-by: Nizamudeen A <nia@redhat.com>
7 months agoMerge pull request #60928 from adk3798/fix-nvme-loop-parsing
Adam King [Tue, 10 Dec 2024 20:09:26 +0000 (15:09 -0500)]
Merge pull request #60928 from adk3798/fix-nvme-loop-parsing

qa/tasks/nvme_loop: update task to work with new nvme list format

Reviewed-by: Laura Flores <lflores@ibm.com>
7 months agoMerge pull request #60992 from MaxKellermann/test__includes
Casey Bodley [Tue, 10 Dec 2024 18:00:20 +0000 (13:00 -0500)]
Merge pull request #60992 from MaxKellermann/test__includes

test: add missing includes

Reviewed-by: Casey Bodley <cbodley@redhat.com>
7 months agoMerge pull request #60773 from pritha-srivastava/wip-rgw-revert-auth-code
Casey Bodley [Tue, 10 Dec 2024 17:07:40 +0000 (12:07 -0500)]
Merge pull request #60773 from pritha-srivastava/wip-rgw-revert-auth-code

Revert "rgw/auth: Fix the return code returned by AuthStrategy,"

Reviewed-by: Casey Bodley <cbodley@redhat.com>
7 months agoMerge pull request #60968 from cbodley/wip-69134
Casey Bodley [Tue, 10 Dec 2024 16:31:04 +0000 (11:31 -0500)]
Merge pull request #60968 from cbodley/wip-69134

qa/rgw: bump keystone/barbican from 2023.1 to 2024.1

Reviewed-by: Shilpa Jagannath <smanjara@redhat.com>
7 months agoMerge pull request #59808 from yuvalif/wip-yuval-bucket-logging3
Yuval Lifshitz [Tue, 10 Dec 2024 16:08:31 +0000 (18:08 +0200)]
Merge pull request #59808 from yuvalif/wip-yuval-bucket-logging3

rgw/logging: add support for GetBucketLogging and PutBucketLogging

Reviewed-By: Anthony D'Atri <anthony.datri@gmail.com>, Adam Emerson <aemerson@ibm.com>, Casey Bodley <cbodley@ibm.com>
7 months agoMerge pull request #61016 from idryomov/wip-p-snapname-type
Ilya Dryomov [Tue, 10 Dec 2024 13:56:21 +0000 (14:56 +0100)]
Merge pull request #61016 from idryomov/wip-p-snapname-type

pybind/rbd: fix a type comment for p_snapshot of clone()

Reviewed-by: Vinay Bhaskar Varada <vvarada@redhat.com>
7 months agotest: add missing includes 60992/head
Max Kellermann [Mon, 14 Oct 2024 14:29:57 +0000 (16:29 +0200)]
test: add missing includes

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
7 months agopybind/rbd: fix a type comment for p_snapshot of clone() 61016/head
Ilya Dryomov [Tue, 10 Dec 2024 10:48:42 +0000 (11:48 +0100)]
pybind/rbd: fix a type comment for p_snapshot of clone()

This was missed in commit d7fd66ec9944 ("librbd: add rbd_clone4() API
to take parent snapshot by ID").

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
7 months agoMerge pull request #60997 from MaxKellermann/sstring__char8_t
Casey Bodley [Mon, 9 Dec 2024 21:44:16 +0000 (16:44 -0500)]
Merge pull request #60997 from MaxKellermann/sstring__char8_t

tools/ceph-dencoder/sstring.h: use `char8_t` instead of `unsigned char`

Reviewed-by: Casey Bodley <cbodley@redhat.com>
7 months agoMerge pull request #60831 from kamoltat/wip-ksirivad-fix-68338
Kamoltat (Junior) Sirivadhna [Mon, 9 Dec 2024 20:33:28 +0000 (15:33 -0500)]
Merge pull request #60831 from kamoltat/wip-ksirivad-fix-68338

doc/rados/operations/stretch-mode.rst: Added Limitations to stretch pool configurations
Reviewed-by: Zac Dover <zac.dover@proton.me>
7 months agodoc/releases: fix formatting for the 17.2.8 release with hyperlinks (el8 CERN) 61002/head
Enrico Bocchi [Mon, 9 Dec 2024 20:30:29 +0000 (21:30 +0100)]
doc/releases: fix formatting for the 17.2.8 release with hyperlinks (el8 CERN)

Co-authored-by: Laura Flores <lflores@redhat.com>
Signed-off-by: Enrico Bocchi <enrico.bocchi@cern.ch>
7 months agoMerge pull request #60927 from ljflores/wip-tracker-69107
Kamoltat (Junior) Sirivadhna [Mon, 9 Dec 2024 19:32:07 +0000 (14:32 -0500)]
Merge pull request #60927 from ljflores/wip-tracker-69107

qa/workunits/mon: ensure election strategy is "connectivity" for stretch mode
Reviewed-by: Kamoltat Sirivadhna <ksirivad@redhat.com>
7 months agoMerge pull request #60998 from MaxKellermann/test_time__using_mono_clock
Casey Bodley [Mon, 9 Dec 2024 18:13:37 +0000 (13:13 -0500)]
Merge pull request #60998 from MaxKellermann/test_time__using_mono_clock

test/common/test_time: add missing `using ceph::mono_clock`

Reviewed-by: Casey Bodley <cbodley@redhat.com>
7 months agodoc/releases: fix formatting for the 17.2.8 release (el8 CERN)
Enrico Bocchi [Mon, 9 Dec 2024 12:46:29 +0000 (13:46 +0100)]
doc/releases: fix formatting for the 17.2.8 release (el8 CERN)

Signed-off-by: Enrico Bocchi <enrico.bocchi@cern.ch>
7 months agotest/common/test_time: add missing `using ceph::mono_clock` 60998/head
Max Kellermann [Tue, 5 Nov 2024 21:14:57 +0000 (22:14 +0100)]
test/common/test_time: add missing `using ceph::mono_clock`

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
7 months agotools/ceph-dencoder/sstring.h: use `char8_t` instead of `unsigned char` 60997/head
Max Kellermann [Sat, 2 Nov 2024 21:32:23 +0000 (22:32 +0100)]
tools/ceph-dencoder/sstring.h: use `char8_t` instead of `unsigned char`

This fixes a build failure with libc++ (clang/LLVM).  This build
failure is correct: there exists no specialization for
`std::char_traits<unsigned char>`.  The standards-compliant way to use
unsigned chars in strings is to use `char8_t`.

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
7 months agotest/rgw/logging: add bucket logging test suite to teuthology 59808/head
Yuval Lifshitz [Mon, 11 Nov 2024 17:48:39 +0000 (17:48 +0000)]
test/rgw/logging: add bucket logging test suite to teuthology

this is depended with this s3test PR:
https://github.com/ceph/s3-tests/pull/576

Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
7 months agorgw/logging: add support for GetBucketLogging and PutBucketLogging
Yuval Lifshitz [Wed, 6 Dec 2023 18:51:59 +0000 (18:51 +0000)]
rgw/logging: add support for GetBucketLogging and PutBucketLogging

this is based on AWS server access logs:
- https://docs.aws.amazon.com/AmazonS3/latest/userguide/enable-server-access-logging.html
- https://docs.aws.amazon.com/AmazonS3/latest/userguide/ServerLogs.html
- https://docs.aws.amazon.com/AmazonS3/latest/API/API_PutBucketLogging.html
- https://docs.aws.amazon.com/AmazonS3/latest/API/API_GetBucketLogging.html

however, a new mode was added called "journal" where:
- logs of PUT, COPY and MPU are guaranteed
- we have logs of DELETE and multi-DELETE operations (not guaranteed)
- log records hold only minimal amount of information

Fixes: https://tracker.ceph.com/issues/984
Signed-off-by: Yuval Lifshitz <ylifshit@redhat.com>
7 months agoMerge pull request #60812 from pecastro/improve-a-few-tests
Casey Bodley [Fri, 6 Dec 2024 18:23:47 +0000 (13:23 -0500)]
Merge pull request #60812 from pecastro/improve-a-few-tests

tests: Improve unittest_dbstore_tests and unittest_deferred to be suitable for reruns.

Reviewed-by: Soumya Koduri <skoduri@redhat.com>
7 months agoMerge pull request #60829 from cbodley/wip-69042
Casey Bodley [Fri, 6 Dec 2024 16:26:12 +0000 (11:26 -0500)]
Merge pull request #60829 from cbodley/wip-69042

radosgw-admin: bucket link/unlink support accounts

Reviewed-by: Adam Emerson <aemerson@redhat.com>
7 months agoradosgw-admin: bucket link/unlink support accounts 60829/head
Casey Bodley [Mon, 25 Nov 2024 17:24:07 +0000 (12:24 -0500)]
radosgw-admin: bucket link/unlink support accounts

buckets are owned by accounts instead of their individual users

add support for --account-id to 'bucket link' and 'bucket unlink'

reject 'bucket link' when the requested --uid belongs to an account

Fixes: https://tracker.ceph.com/issues/69042
Signed-off-by: Casey Bodley <cbodley@redhat.com>
7 months agoMerge pull request #60129 from cbodley/wip-68396
Casey Bodley [Fri, 6 Dec 2024 14:08:56 +0000 (09:08 -0500)]
Merge pull request #60129 from cbodley/wip-68396

test/rgw/multisite: add meta checkpoint after bucket creation

Reviewed-by: Shilpa Jagannath <smanjara@redhat.com>
7 months agoMerge pull request #60304 from cbodley/wip-async-completion-recycling
Casey Bodley [Fri, 6 Dec 2024 14:08:30 +0000 (09:08 -0500)]
Merge pull request #60304 from cbodley/wip-async-completion-recycling

common/async: Completion uses asio::recycling_allocator by default

Reviewed-by: Adam Emerson <aemerson@redhat.com>
7 months agoMerge pull request #60415 from cbodley/wip-68645
Casey Bodley [Fri, 6 Dec 2024 14:08:09 +0000 (09:08 -0500)]
Merge pull request #60415 from cbodley/wip-68645

radosgw-admin: allow 'sync group pipe modify' with existing user

Reviewed-by: Soumya Koduri <skoduri@redhat.com>
7 months agoMerge pull request #60830 from cbodley/wip-69043
Casey Bodley [Fri, 6 Dec 2024 14:07:44 +0000 (09:07 -0500)]
Merge pull request #60830 from cbodley/wip-69043

radosgw-admin: 'user create' rejects uids matching the account id format

Reviewed-by: Adam Emerson <aemerson@redhat.com>
7 months agoMerge pull request #59383 from joscollin/wip-B67360-counters-set-on-mds-failed
Milind Changire [Fri, 6 Dec 2024 07:13:11 +0000 (12:43 +0530)]
Merge pull request #59383 from joscollin/wip-B67360-counters-set-on-mds-failed

qa: restrict 'perf dump' on active mds only

7 months agorgw: shouldn't call index_op.cancel() when rados op return ETIMEDOUT 60916/head
lu.shasha [Fri, 6 Dec 2024 04:40:27 +0000 (12:40 +0800)]
rgw: shouldn't call index_op.cancel() when rados op return ETIMEDOUT

when rados op return ETIMEOUT, rgw can't determine whether or not the rados op succeeded,
we shouldn't be calling index_op->cancel() in this case
Instead, we should leave that pending entry in the index so than bucket listing can recover with check_disk_state() and cls_rgw_suggest_changes()

Signed-off-by: Shasha Lu <lu.shasha@aishu.cn>
7 months agoMerge pull request #60929 from dmick/wip-release-docs
Zac Dover [Fri, 6 Dec 2024 04:04:52 +0000 (05:04 +0100)]
Merge pull request #60929 from dmick/wip-release-docs

doc/dev/release-process.rst: New container build/release process

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
7 months agoMerge pull request #60947 from idryomov/wip-67759
Ilya Dryomov [Thu, 5 Dec 2024 20:20:52 +0000 (21:20 +0100)]
Merge pull request #60947 from idryomov/wip-67759

qa/suites/upgrade/*-x: skip TestClsRbd.group_snap_{set,remove}

Reviewed-by: N Balachandran <nibalach@redhat.com>
Reviewed-by: Laura Flores <lflores@ibm.com>
7 months agoqa/rgw: bump keystone/barbican from 2023.1 to 2024.1 60968/head
Casey Bodley [Thu, 5 Dec 2024 19:57:14 +0000 (14:57 -0500)]
qa/rgw: bump keystone/barbican from 2023.1 to 2024.1

Signed-off-by: Casey Bodley <cbodley@redhat.com>
7 months agoMerge pull request #60966 from gbregman/main
Gil Bregman [Thu, 5 Dec 2024 18:44:06 +0000 (20:44 +0200)]
Merge pull request #60966 from gbregman/main

mgr/cephadm/nvmeof: Add encryption key file for nvmeof encryption

7 months agoMerge pull request #60948 from idryomov/wip-68588
Ilya Dryomov [Thu, 5 Dec 2024 13:53:40 +0000 (14:53 +0100)]
Merge pull request #60948 from idryomov/wip-68588

qa/suites/rados/thrash-old-clients: skip TestClsRbd.group_snap_{set,remove}

Reviewed-by: N Balachandran <nibalach@redhat.com>
7 months agomgr/cephadm/nvmeof: Add key encryption support to nvmeof configuration 60966/head
Gil Bregman [Thu, 5 Dec 2024 12:07:45 +0000 (14:07 +0200)]
mgr/cephadm/nvmeof: Add key encryption support to nvmeof configuration

Signed-off-by: Gil Bregman <gbregman@il.ibm.com>
7 months agoMerge pull request #60926 from ktdreyer/container-labeler
Dan Mick [Thu, 5 Dec 2024 11:49:07 +0000 (03:49 -0800)]
Merge pull request #60926 from ktdreyer/container-labeler

labeler: add container labelers

7 months agoMerge pull request #60946 from nbalacha/wip-nbalacha-69112
Ilya Dryomov [Thu, 5 Dec 2024 08:22:22 +0000 (09:22 +0100)]
Merge pull request #60946 from nbalacha/wip-nbalacha-69112

cls/rbd: fix mirror_image_map_list object

Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
7 months agoMerge pull request #60913 from myoungwon/wip-fix-null-pointer-crimson
Yingxin [Thu, 5 Dec 2024 01:45:03 +0000 (09:45 +0800)]
Merge pull request #60913 from myoungwon/wip-fix-null-pointer-crimson

crimson/osd/ops_executer: prevent access to null pointers caused by gcc bug

Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Xuehan Xu <xuxuehan@qianxin.com>
7 months agotest/objectstore: Make test idempotent 60812/head
Paulo E. Castro [Sat, 16 Nov 2024 12:35:04 +0000 (12:35 +0000)]
test/objectstore: Make test idempotent

Give it a new DB name for each test so we don't get failures on rerun.

Signed-off-by: Paulo E. Castro <pecastro@wormholenet.com>
7 months agorgw: Make test idempotent.
Paulo E. Castro [Sat, 16 Nov 2024 11:32:20 +0000 (11:32 +0000)]
rgw: Make test idempotent.

Give it a new DB name for each test so we don't get failures on rerun.

Signed-off-by: Paulo E. Castro <pecastro@wormholenet.com>
7 months agorgw: Remove unnecessary whitespace.
Paulo E. Castro [Sat, 16 Nov 2024 11:30:42 +0000 (11:30 +0000)]
rgw: Remove unnecessary whitespace.

Signed-off-by: Paulo E. Castro <pecastro@wormholenet.com>
7 months agoMerge pull request #60944 from dalgaaf/dalgaaf-fix-doc-ec
Anthony D'Atri [Wed, 4 Dec 2024 21:49:18 +0000 (22:49 +0100)]
Merge pull request #60944 from dalgaaf/dalgaaf-fix-doc-ec

doc: fix EC overhead table m-numbering

7 months agoMerge pull request #60954 from zdover23/wip-doc-2024-12-04-dev-development-workflow...
Anthony D'Atri [Wed, 4 Dec 2024 21:04:39 +0000 (22:04 +0100)]
Merge pull request #60954 from zdover23/wip-doc-2024-12-04-dev-development-workflow-backporter-manual

doc/dev: update link to backporter manual

7 months agodoc/dev: update link to backporter manual 60954/head
Zac Dover [Wed, 4 Dec 2024 20:33:59 +0000 (21:33 +0100)]
doc/dev: update link to backporter manual

Update the link to the backporter manual, in
doc/dev/development-workflow.rst. Laura Flores requested this change on
04 Dec 2024.

Signed-off-by: Zac Dover <zac.dover@proton.me>
7 months agoMerge pull request #60924 from ktdreyer/containerfile-date-utc
Ken Dreyer [Wed, 4 Dec 2024 19:40:04 +0000 (14:40 -0500)]
Merge pull request #60924 from ktdreyer/containerfile-date-utc

container/build.sh: use UTC dates for tags

7 months agorgw: optimize bucket listing to skip past regions of namespaced entries 60851/head
J. Eric Ivancich [Fri, 22 Nov 2024 17:40:24 +0000 (12:40 -0500)]
rgw: optimize bucket listing to skip past regions of namespaced entries

When listing a bucket and the parameters are such that we're not
listing namespaced entries, this commit adds an optimization to
advance the marker such that we skip past a whole region of namespaced
entries rather than evaluating each entry one-by-one.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>
7 months agoMerge pull request #60930 from zdover23/wip-doc-2024-12-03-rados-ops-health-checks-3
Zac Dover [Wed, 4 Dec 2024 14:02:49 +0000 (15:02 +0100)]
Merge pull request #60930 from zdover23/wip-doc-2024-12-03-rados-ops-health-checks-3

doc/rados: fix sentences in health-checks (3 of x)

Reviewed-by: Cole Mitchell <cole.mitchell.ceph@gmail.com>
7 months agoqa/suites/upgrade/*-x: skip TestClsRbd.group_snap_{set,remove} 60947/head
Ilya Dryomov [Wed, 4 Dec 2024 11:59:07 +0000 (12:59 +0100)]
qa/suites/upgrade/*-x: skip TestClsRbd.group_snap_{set,remove}

These tests assert on the number of keys on the group header object,
which changed with commits ec6224db3e55 ("cls/rbd: save group snapshot
creation order in a new key") and 3662bc7b3dc3 ("cls/rbd: save max
group snap order").

Fixes: https://tracker.ceph.com/issues/67759
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
7 months agocls/rbd: fix mirror_image_map_list object 60946/head
N Balachandran [Wed, 4 Dec 2024 11:40:13 +0000 (17:10 +0530)]
cls/rbd: fix mirror_image_map_list object

The rbd-mirror image map is stored in the rbd_mirror_leader object.
Fixed mirror_image_map_list() to operate on the correct object.

Fixes: https://tracker.ceph.com/issues/69112
Signed-off-by: N Balachandran <nibalach@redhat.com>
7 months agoqa/suites/rados/thrash-old-clients: skip TestClsRbd.group_snap_{set,remove} 60948/head
Ilya Dryomov [Wed, 4 Dec 2024 11:21:35 +0000 (12:21 +0100)]
qa/suites/rados/thrash-old-clients: skip TestClsRbd.group_snap_{set,remove}

These tests assert on the number of keys on the group header object,
which changed with commits ec6224db3e55 ("cls/rbd: save group snapshot
creation order in a new key") and 3662bc7b3dc3 ("cls/rbd: save max
group snap order").

Fixes: https://tracker.ceph.com/issues/68588
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
7 months agoqa/suites/rados/thrash-old-clients: switch to cls/test_cls_rbd.sh workunit
Ilya Dryomov [Wed, 4 Dec 2024 11:19:08 +0000 (12:19 +0100)]
qa/suites/rados/thrash-old-clients: switch to cls/test_cls_rbd.sh workunit

For consistency with other upgrade jobs, don't invoke ceph_test_cls_rbd
directly.  While at it, drop a long outdated filter.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
7 months agoMerge pull request #60514 from rhcs-dashboard/fix-68733-main
afreen23 [Wed, 4 Dec 2024 09:46:03 +0000 (15:16 +0530)]
Merge pull request #60514 from rhcs-dashboard/fix-68733-main

mgr/dashboard: fix total objects/Avg object size in RGW Overview Page

Reviewed-by: Afreen Misbah <afreen@ibm.com>
7 months agodoc/dev/release-process.rst: New container build/release process 60929/head
Dan Mick [Wed, 4 Dec 2024 01:15:41 +0000 (17:15 -0800)]
doc/dev/release-process.rst: New container build/release process

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agorgw: fix stale entries in bucket indexes 60907/head
lu.shasha [Mon, 2 Dec 2024 09:10:23 +0000 (17:10 +0800)]
rgw: fix stale entries in bucket indexes

If rados_osd_op_timeout is set, the primary osd is slow, the rgw_rados_operate for deleting the rgw head obj may return -ETIMEDOUT
rgw can't determine whether or not the delete succeeded, we shouldn't be calling index_op.complete_del or cancel()
Instead, we should leave that pending entry in the index so than bucket listing can recover with check_disk_state() and cls_rgw_suggest_changens()
When raced with another delete op, deleting the rgw head obj may return ENOENT, calling index_op.complete_del() instead of index_op.cancel()

Fixes: https://tracker.ceph.com/issues/58965
Signed-off-by: Shasha Lu <lu.shasha@aishu.cn>
7 months agocrimson/osd/ops_executer: prevent access to null pointers caused by gcc bug pull/60913/head 60913/head
myoungwon oh [Mon, 2 Dec 2024 10:02:08 +0000 (10:02 +0000)]
crimson/osd/ops_executer: prevent access to null pointers caused by gcc bug

This commit assigns the parameter eariler to work around a coroutine related gcc bug
that causes a null pointer access.

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
7 months agoMerge pull request #60904 from cyx1231st/wip-seastore-hex-log
Yingxin [Wed, 4 Dec 2024 07:07:56 +0000 (15:07 +0800)]
Merge pull request #60904 from cyx1231st/wip-seastore-hex-log

crimson/os/seastore: convert offset/length in logs to hex format

Reviewed-by: Xuehan Xu <xuxuehan@qianxin.com>
7 months agoMerge pull request #60755 from dmick/wip-release-container
Dan Mick [Wed, 4 Dec 2024 06:32:15 +0000 (22:32 -0800)]
Merge pull request #60755 from dmick/wip-release-container

Modify container/ software to support release containers and the promotion of prerelease containers

7 months agoMerge pull request #60018 from smanjara/rgw-script-multiple-zg
Shilpa Jagannath [Wed, 4 Dec 2024 05:35:05 +0000 (21:35 -0800)]
Merge pull request #60018 from smanjara/rgw-script-multiple-zg

rgw/multisite: test-rgw-multisite.sh can create multiple zonegroups

7 months agoMerge pull request #60919 from zdover23/wip-doc-2024-12-03-rados-ops-health-checks-2
Zac Dover [Wed, 4 Dec 2024 02:16:50 +0000 (03:16 +0100)]
Merge pull request #60919 from zdover23/wip-doc-2024-12-03-rados-ops-health-checks-2

doc/rados: fix sentences in health-checks (2 of x)

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
7 months agodoc/rados: fix sentences in health-checks (3 of x) 60930/head
Zac Dover [Wed, 4 Dec 2024 02:13:05 +0000 (03:13 +0100)]
doc/rados: fix sentences in health-checks (3 of x)

Make sentences agree at the head of each section in
doc/rados/operations/health-checks.rst. The sentences were sometimes in
the imperative mood and sometimes in the declarative mood.

This commit edits the second third of
doc/rados/operations/health-checks.rst.

Note to (I hope soon) future Zac: There are a a couple of places near
the end of this file where the sentences are ungrammatical. Update these
in a separate PR (in isolation, so that the grammar and technical
accuracy of these sentences can be the primary focus of the reviewers).

Zac: cf. 000228

Signed-off-by: Zac Dover <zac.dover@proton.me>
7 months agocontainer/{Containerfile,build.sh}: use a secret to pass secrets 60755/head
Dan Mick [Thu, 21 Nov 2024 03:11:24 +0000 (19:11 -0800)]
container/{Containerfile,build.sh}: use a secret to pass secrets

Use podman's build --secret mechanism to pass credentials for
download.ceph.com

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agocontainer/Containerfile: allow failure from curl to stop build
Dan Mick [Thu, 21 Nov 2024 02:23:50 +0000 (18:23 -0800)]
container/Containerfile: allow failure from curl to stop build

curl doesn't actually fail unless -f is used; also, add
some sh -e flags.

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agocontainer/Containerfile: purge .repo files with secrets before commit
Dan Mick [Thu, 21 Nov 2024 02:18:59 +0000 (18:18 -0800)]
container/Containerfile: purge .repo files with secrets before commit

ceph.repo had creds in it for download.ceph.com/prerelease.
Remove the .repo files we construct, since they're not necessary
once the container is built (no one should be dnf'ing anything
in the container).

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agocontainer/Containerfile fix boolean IS_RELEASE usage
Dan Mick [Wed, 20 Nov 2024 00:44:17 +0000 (16:44 -0800)]
container/Containerfile fix boolean IS_RELEASE usage

Existing syntax was just wrong.

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agoqa/tasks/nvme_loop: update task to work with new nvme list format 60928/head
Adam King [Tue, 3 Dec 2024 20:22:22 +0000 (15:22 -0500)]
qa/tasks/nvme_loop: update task to work with new nvme list format

Specifically on some centos 9 tests, we've seen that a newer
version of some nvme related package is causing this task to fail
with "KeyError: 'DevicePath'" due to the format of the output
of the nvme list command changing. This patch adds handling for
the new format we've seen while also still supporting the old
format (necessary for the tests running on ubuntu).

Fixes: https://tracker.ceph.com/issues/69067
Signed-off-by: Adam King <adking@redhat.com>
7 months agoqa/workunits/mon: ensure election strategy is "connectivity" for stretch mode 60927/head
Laura Flores [Tue, 3 Dec 2024 22:15:19 +0000 (16:15 -0600)]
qa/workunits/mon: ensure election strategy is "connectivity" for stretch mode

The election strategy is randomly chosen for this type of test. Sometimes,
the test passes if the "connectivity" election strategy happens to be picked.
But if a different strategy, i.e. "classic", is picked, then the test will fail.

We can ensure that the election strategy is "connectivity" by setting it in the
workunit with the ceph CLI command. Although connectivity was specified in
stretch-mode-5-mons-8-osds.yaml, that config ultimately gets overridden by
the "qa/mon_config" yaml.

Fixes: https://tracker.ceph.com/issues/69107
Signed-off-by: Laura Flores <lflores@ibm.com>
7 months agoMerge pull request #59862 from jamiepryde/isa-xor-handle-misaligned-buffers
Yuri Weinstein [Tue, 3 Dec 2024 22:07:53 +0000 (14:07 -0800)]
Merge pull request #59862 from jamiepryde/isa-xor-handle-misaligned-buffers

erasure-code/isa: Handle case where data and coding buffers are not 3…

Reviewed-by: Jose J Palacios-Perez <perezjos@uk.ibm.com>
7 months agoMerge pull request #58473 from YiteGu/add-mon-scrub-lat
Yuri Weinstein [Tue, 3 Dec 2024 22:05:05 +0000 (14:05 -0800)]
Merge pull request #58473 from YiteGu/add-mon-scrub-lat

mon/scrub: add mon scrub latency

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
7 months agoMerge pull request #60131 from Shreyasapale/wip-fix-shreya-68009
Laura Flores [Tue, 3 Dec 2024 21:44:08 +0000 (15:44 -0600)]
Merge pull request #60131 from Shreyasapale/wip-fix-shreya-68009

osd/scrubber: Clean up unused function add_double

7 months agocontainer/make-manifest-list.py: use LOCALMANIFEST throughout
Dan Mick [Wed, 20 Nov 2024 00:40:05 +0000 (16:40 -0800)]
container/make-manifest-list.py: use LOCALMANIFEST throughout

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agocontainers/make-manifest.py: Add --promote
Dan Mick [Sat, 16 Nov 2024 00:21:26 +0000 (16:21 -0800)]
containers/make-manifest.py: Add --promote

The promote function finds the latest image on the prerelease repo,
finds all its tags, and copies them all to the release repo.

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agocontainer/Containerfile, build.sh: Add PRERELEASE_USERNAME/PASSWORD
Dan Mick [Fri, 15 Nov 2024 01:49:15 +0000 (17:49 -0800)]
container/Containerfile, build.sh: Add PRERELEASE_USERNAME/PASSWORD

Used for accessing download.ceph.com

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agocontainer/build.sh:
Dan Mick [Thu, 14 Nov 2024 22:28:32 +0000 (14:28 -0800)]
container/build.sh:

- change repo path to prerelease
- remove image id arg for release push
- include 'v' in version name for release push

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agocontainer/make-manifest-list.py
Dan Mick [Fri, 1 Nov 2024 02:55:36 +0000 (19:55 -0700)]
container/make-manifest-list.py

- don't print command failure in worker; let the caller print them
  if desired (allow silent failure)
- allow for empty tags list
- look for CEPH_SHA1.  GIT_COMMIT was the sha1 of the ceph-container.git
  commit
- change default paths to prerelease
- add --dry-run to avoid final push
- rename 'HOST' to 'CONTAINER_HOST'
- Use ARCH_SPECIFIC_HOST instead of CONTAINER_HOST (which is used by podman)

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agoMerge pull request #60172 from smanjara/rgw-multisite-two-zonegroups
Shilpa Jagannath [Tue, 3 Dec 2024 19:22:25 +0000 (11:22 -0800)]
Merge pull request #60172 from smanjara/rgw-multisite-two-zonegroups

qa/multisite: enable two zonegroup yaml

7 months agoMerge pull request #60591 from smanjara/wip-datalog-autotrim
Shilpa Jagannath [Tue, 3 Dec 2024 18:20:43 +0000 (10:20 -0800)]
Merge pull request #60591 from smanjara/wip-datalog-autotrim

qa/multisite: add extra checkpoints in datalog_autotrim testcase

7 months agolabeler: add container labelers 60926/head
Ken Dreyer [Tue, 3 Dec 2024 17:44:52 +0000 (12:44 -0500)]
labeler: add container labelers

Signed-off-by: Ken Dreyer <kdreyer@ibm.com>
7 months agocontainer/build.sh: use UTC dates for tags 60924/head
Ken Dreyer [Tue, 3 Dec 2024 17:37:11 +0000 (12:37 -0500)]
container/build.sh: use UTC dates for tags

Do not rely on the builder host's local timezone setting.

Signed-off-by: Ken Dreyer <kdreyer@ibm.com>
7 months agodoc: fix EC overhead table m-numbering 60944/head
Danny Al-Gaaf [Tue, 3 Dec 2024 14:53:18 +0000 (15:53 +0100)]
doc: fix EC overhead table m-numbering

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
7 months agoMerge pull request #60918 from gbregman/main
Gil Bregman [Tue, 3 Dec 2024 14:02:24 +0000 (16:02 +0200)]
Merge pull request #60918 from gbregman/main

python-common/ceph/cephadm: Change NVME image version to 1.4.

7 months agoMerge pull request #60917 from zdover23/wip-doc-2024-12-03-rados-ops-health-checks
Zac Dover [Tue, 3 Dec 2024 11:07:28 +0000 (12:07 +0100)]
Merge pull request #60917 from zdover23/wip-doc-2024-12-03-rados-ops-health-checks

doc/rados: make sentences agree in health-checks.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
7 months agodoc/rados: fix sentences in health-checks (2 of x) 60919/head
Zac Dover [Tue, 3 Dec 2024 11:02:43 +0000 (12:02 +0100)]
doc/rados: fix sentences in health-checks (2 of x)

Make sentences agree at the head of each section in
doc/rados/operations/health-checks.rst. The sentences were sometimes in
the imperative mood and sometimes in the declarative mood.

This commit edits the second third of
doc/rados/operations/health-checks.rst.

Zac: cf. 000228

Signed-off-by: Zac Dover <zac.dover@proton.me>
7 months agopython-common/ceph/cephadm: Change NVME image version to 1.4. 60918/head
Gil Bregman [Tue, 3 Dec 2024 09:31:44 +0000 (11:31 +0200)]
python-common/ceph/cephadm: Change NVME image version to 1.4.
Fixes https://tracker.ceph.com/issues/69099

Signed-off-by: Gil Bregman <gbregman@il.ibm.com>
7 months agoMerge pull request #60767 from rhcs-dashboard/smb-cluster
afreen23 [Tue, 3 Dec 2024 09:17:32 +0000 (14:47 +0530)]
Merge pull request #60767 from rhcs-dashboard/smb-cluster

mgr/dashboard: list smb

Reviewed-by: Afreen Misbah <afreen@ibm.com>
7 months agoMerge pull request #60099 from MaxKellermann/json_spirit_thread
Ilya Dryomov [Tue, 3 Dec 2024 09:09:19 +0000 (10:09 +0100)]
Merge pull request #60099 from MaxKellermann/json_spirit_thread

json_spirit: add missing dependency on Boost::thread

Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
7 months agodoc/rados: make sentences agree in health-checks.rst 60917/head
Zac Dover [Tue, 3 Dec 2024 08:28:09 +0000 (09:28 +0100)]
doc/rados: make sentences agree in health-checks.rst

Make sentences agree at the head of each section in
doc/rados/operations/health-checks.rst. The sentences were sometimes in
the imperative mood and sometimes in the declarative mood.

This commit edits the first third of
doc/rados/operations/health-checks.rst.

Zac: cf. 000228

Signed-off-by: Zac Dover <zac.dover@proton.me>
7 months agorgw: fix data corruption when rados op return ETIMEDOUT
lu.shasha [Tue, 3 Dec 2024 07:46:35 +0000 (15:46 +0800)]
rgw: fix data corruption when rados op return ETIMEDOUT

when rados op return ETIMEOUT, rgw can't determine whether or not the rados op succeeded,
To avoid data corruption, when return -ETIMEDOUT, we should clear all objects in RadosWriter

Signed-off-by: Shasha Lu <lu.shasha@aishu.cn>
7 months agocrimson/os/seastore: convert offset/length in seastore logs to hex format 60904/head
Yingxin Cheng [Mon, 2 Dec 2024 08:17:21 +0000 (16:17 +0800)]
crimson/os/seastore: convert offset/length in seastore logs to hex format

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>