summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #61387 from yuvalif/wip-yuval-69473J. Eric Ivancich26 hours2-2/+11
|\ | | | | | | | | | | test/rgw/lua: install luarocks Reviewed-by: Casey Bodley <cbodley@redhat.com> Reviewed-by: Kefu Chai <tchaikov@gmail.com>
| * test/rgw/lua: install luarocksYuval Lifshitz2 days2-2/+11
| | | | | | | | | | | | | | | | | | since it is only a suggestion for debian. this is a regression from: https://github.com/ceph/ceph/pull/60666 Fixes: https://tracker.ceph.com/issues/69473 Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
* | Merge pull request #59143 from Sean10/fix_rados_pybind_zero_paramYuri Weinstein26 hours3-1/+10
|\ \ | | | | | | | | | | | | | | | | | | pybind/rados: fix the incorrect order of offset,length in WriteOp.zero Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com> Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
| * | pybind/rados: add note for reversed arguments to WriteOp.zero()Wang Chao2024-11-051-0/+4
| | | | | | | | | | | | Signed-off-by: Wang Chao <sean10reborn@gmail.com>
| * | test/pybind/test_rados.py: add test for reversed arguments offset,length in ↵Wang Chao2024-11-051-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | WriteOp.zero Before the fix, zero(0, 2) would have no effect, and read would get '12345' instead of the expected '\x00\x00345'. Signed-off-by: Wang Chao <sean10reborn@gmail.com>
| * | pybind/rados: fix the incorrect order of offset,length in WriteOp.zeroWang Chao2024-11-051-1/+1
| | | | | | | | | | | | | | | | | | | | | The offset and length parameters in the rados pybind `WriteOp.zero()` method are being passed to the rados_write_op_zero() function in the incorrect order. Incorrect order cause OP_ZERO not work correctly when use pybind's rados. Signed-off-by: Wang Chao <sean10reborn@gmail.com>
* | | Merge pull request #61384 from sj14/sj/delete-policy-204J. Eric Ivancich28 hours1-1/+4
|\ \ \ | | | | | | | | | | | | | | | | rgw: S3 Delete Bucket Policy should return 204 on success Reviewed-by: Casey Bodley <cbodley@redhat.com>
| * | | rgw: S3 Delete Bucket Policy should return 204 on successSimon Jürgensmeyer3 days1-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, RGW returns a 200 on a successful DELETE on a bucket policy but the S3 API expects a 204: https://docs.aws.amazon.com/AmazonS3/latest/API/API_DeleteBucketPolicy.html The sample response in the example above is a 204. This patch checks the op_ret in `RGWDeleteBucketPolicy::send_response()` and on a success we turn it to a 204 (or STATUS_NO_CONTENT). Fixes: https://tracker.ceph.com/issues/69539 Signed-off-by: Simon Jürgensmeyer <simon.juergensmeyer@hetzner-cloud.de>
* | | | Merge pull request #61370 from yehudasa/wip-69536J. Eric Ivancich28 hours1-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | rgw: RGWRados::get_olh() needs to use the correct attr Reviewed-by: Casey Bodley <cbodley@redhat.com>
| * | | | rgw: RGWRados::get_olh() needs to use the correct attrYehuda Sadeh4 days1-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/69536 Signed-off-by: Yehuda Sadeh <ysadehwe@ibm.com>
* | | | | Merge pull request #61285 from cbodley/wip-69462J. Eric Ivancich28 hours1-0/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | rgw: /admin/user api dumps account-related user info Reviewed-by: J. Eric Ivancich <ivancich@redhat.com>
| * | | | | rgw: /admin/user api dumps account-related user infoCasey Bodley9 days1-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the admin api uses a separate dump_user_info() function that wasn't updated with the account changes to RGWUserInfo::dump() Fixes: https://tracker.ceph.com/issues/69462 Signed-off-by: Casey Bodley <cbodley@redhat.com>
* | | | | | Merge pull request #61244 from awojno-bloomberg/fix-bucket-sync-statusJ. Eric Ivancich28 hours1-8/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rgw: fix radosgw-admin use after free in bucket sync status Reviewed-By: Adam Emerson <aemerson@ibm.com> Reviewed-By: Oguzhan Ozmen <oozmen@bloomberg.net>
| * | | | | | rgw:fix radosgw-admin use after freeAlex Wojno9 days1-8/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Converted string_view members to string to avoid use after free. Bug introduced in commit: e84c7fc Signed-off-by: Alex Wojno <awojno@bloomberg.net>
* | | | | | | Merge pull request #61411 from ↵Zac Dover30 hours1-19/+22
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | zdover23/wip-doc-2025-01-16-cephfs-disaster-recovery-experts doc/cephfs: edit disaster-recovery-experts Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
| * | | | | | | doc/cephfs: edit disaster-recovery-expertsZac Dover2 days1-19/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Edit the first two sections of doc/cephfs/disaster-recovery-experts.rst in preparation for adding deeper explanations of the contexts in which one should use the various commands listed on that page. https://tracker.ceph.com/issues/69557 Signed-off-by: Zac Dover <zac.dover@proton.me>
* | | | | | | | Merge pull request #59155 from badone/wip-tracker-67489-fixed-4-for-verifySrinivasaBharathKanta32 hours1-0/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | qa/suites: Make fixed-4 an option for verify tests
| * | | | | | | | qa/suites: Make fixed-4 an option for verify testsBrad Hubbard2024-08-121-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/67489 Signed-off-by: Brad Hubbard <bhubbard@redhat.com>
* | | | | | | | | Merge pull request #60556 from ↵Adam Kupczyk34 hours2-31/+110
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | aclamk/wip-aclamk-bluefs-truncate-allocations-main os/bluestore: Make truncate() drop unused allocations - addendum
| * | | | | | | | | os/bluestore: Add unittest for BlueFS::truncate()Adam Kupczyk2024-10-301-0/+81
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add unittest for some truncate scenarios. Fixes: https://tracker.ceph.com/issues/68385 (addendum) Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>
| * | | | | | | | | os/bluestore: Make truncate() drop unused allocationsAdam Kupczyk2024-10-291-31/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Review fixes. Removed overcatious assert. Improved if .. else style. Skipped processing extent truncation when seek() goes to end. Fixes: https://tracker.ceph.com/issues/68385 (addendum) Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>
* | | | | | | | | | Merge pull request #60363 from aclamk/wip-aclamk-fix-bluefs-bdev-expandAdam Kupczyk34 hours2-3/+28
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | os/bluestore: Create additional bdev labels when expanding block device.
| * | | | | | | | | | os/bluestore: Fix CBT bluefs-bdev-expandAdam Kupczyk2024-10-161-2/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was a problem when expansion of 'block' device crossed location of bdev label copy. The extra label that did not exist before and now exist was not initialized. Also changed logic of 'bluestore_bdev_label_require_all'. Previously label locations 0..bdev->size() were taken in to account. Now labels in range bdev_label.size..bdev->size() are excluded. Fixes: https://tracker.ceph.com/issues/68577 Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>
| * | | | | | | | | | qa/standalone/bluefs: Fix CBT bluefs-bdev-expandAdam Kupczyk2024-10-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was a problem when expansion of 'block' device crossed location of bdev label copy. The extra label that did not exist before and now exists was not initialized. This makes test to fail. Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>
* | | | | | | | | | | Merge pull request #61369 from laimis9133/patch-2Anthony D'Atri41 hours1-4/+5
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | doc/radosgw/config-ref: fix lifecycle workload tuning description
| * | | | | | | | | | | doc/radosgw/config-ref: fix lifecycle workload tuning descriptionLaimis Juzeliūnas4 days1-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit updates RGW Config Reference - Lifecycle Settings section. In particular it changes the section where two different tunings for two different workloads are suggested. It aims for a more understandable description as asked by the contributors. Signed-off-by: Laimis Juzeliunas <laimis.juzeliunas@oxylabs.io>
* | | | | | | | | | | | Merge pull request #61412 from gbregman/mainGil Bregman42 hours3-0/+8
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/cephadm/nvmeof: Add SPDK iobuf options field to NVMeOF configuration
| * | | | | | | | | | | | mgr/cephadm/nvmeof: Add SPDK iobuf options field to NVMeOF configurationGil Bregman2 days3-0/+8
| | |_|_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes https://tracker.ceph.com/issues/69554 Signed-off-by: Gil Bregman <gbregman@il.ibm.com>
* | | | | | | | | | | | Merge pull request #60265 from MaxKellermann/timeout_round_upYuri Weinstein43 hours3-6/+52
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | msg/async: round timeouts up (fix busy loops) Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
| * | | | | | | | | | | | msg/async/Timeout: always round upMax Kellermann2024-10-221-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, we always round down, which has a bad side effect: when a timer comes closer, we have lots of early wakeups, and eventually we'll run into a busy loop (timeout=0) when the timeout is less than one millisecond; the process will remain this busy loop for one millisecond, wasting lots of CPU time. Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | | | | | | | msg/async/Event{Poll,Epoll}: move timeout calculation to Timeout.hMax Kellermann2024-10-223-6/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is duplicate code, and it's buggy, but I want to fix only one copy. Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
* | | | | | | | | | | | | Merge pull request #48539 from chrisphoffman/wip-57864Ilya Dryomov2 days3-6/+24
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qa/tasks: Include stderr on tasks badness check Reviewed-by: Adam King <adking@redhat.com> Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
| * | | | | | | | | | | | | qa/tasks: Include stderr on tasks badness check.Christopher Hoffman4 days3-6/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure that first_in_ceph_log() doesn't return None (which is treated as success/"no badness" by the caller) if the cluster log file is missing. Fixes: https://tracker.ceph.com/issues/57864 Co-authored-by: Ilya Dryomov <idryomov@gmail.com> Signed-off-by: Christopher Hoffman <choffman@redhat.com> Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
* | | | | | | | | | | | | | Merge pull request #61393 from abhishek-kane/mapping_emailAdam King2 days4-0/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add Abhishek Kane into peoplemap, githubmap, organizationmap, mailmap Reviewed-by: Adam King <adking@redhat.com>
| * | | | | | | | | | | | | Add Abhishek Kane into peoplemap, githubmap, organizationmap, mailmapabhishek-kane3 days4-0/+4
| | |_|_|_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: abhishek-kane <abhishek.kane@ibm.com>
* | | | | | | | | | | | | Merge pull request #61349 from VallariAg/wip-vallari-nvmeof-ceph-versionVallari Agrawal2 days1-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | mon: do not show nvmeof in 'ceph versions' output
| * | | | | | | | | | | | | mon: do not show nvmeof in 'ceph versions' outputVallari Agrawal3 days1-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NVMeoF gateway version is independent of ceph version so 'ceph version' shows wrong nvmeof version in output (i.e. instead of gateway version, it shows Ceph version). Hence, remove nvmeof in 'ceph versions' output. To check for gateway version, use 'gw info' command. Signed-off-by: Vallari Agrawal <vallari.agrawal@ibm.com>
* | | | | | | | | | | | | | Merge pull request #60889 from anoopcs9/fix-invalid-access-mdsMilind Changire2 days1-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mds: Fix invalid access of mdr->dn[0].back()
| * | | | | | | | | | | | | | mds: Fix invalid access of mdr->dn[0].back()Anoop C S2024-11-291-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See https://github.com/ceph/ceph/pull/31534 for a similar fix. Fixes: https://tracker.ceph.com/issues/69059 Signed-off-by: Anoop C S <anoopcs@cryptolab.net>
* | | | | | | | | | | | | | | Merge pull request #61218 from dmick/wip-containerfile-ceph-versionDan Mick3 days1-0/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | container/Containerfile: replace CEPH_VERSION label for backward compat
| * | | | | | | | | | | | | | | container/Containerfile: replace CEPH_VERSION var for backward compatDan Mick12 days1-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CEPH_VERSION used to be present in the env to hold the ref. Other projects relied on this. Make life easier for them. Signed-off-by: Dan Mick <dan.mick@redhat.com>
* | | | | | | | | | | | | | | | Merge pull request #60847 from cbodley/wip-rgw-boto-headbucket-statsCasey Bodley3 days2-1/+57
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | examples/rgw: register boto extension for HeadBucket stats Reviewed-by: Seena Fallah <seenafallah@gmail.com>
| * | | | | | | | | | | | | | | | examples/rgw: register boto extension for HeadBucket statsCasey Bodley2024-12-172-1/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Casey Bodley <cbodley@redhat.com>
* | | | | | | | | | | | | | | | | Merge pull request #61211 from yuvalif/wip-yuval-logging-tenantYuval Lifshitz3 days13-165/+658
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rgw/logging: add more validations to conf, cross tenant support and cleanup Reviewed-By: Adam Emerson <aemerson@ibm.com>, Ali Masarwe <ali.masarwa@ibm.com>
| * | | | | | | | | | | | | | | | | rgw/logging: clean pending objectsYuval Lifshitz5 days13-129/+562
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * when source bucket is deleted * when logging is disabled * when logging conf changes * when target bucket is deleted also add "bucket logging info" admin command returning logging conf for source bucket, and list of source buckets for log bucket Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
| * | | | | | | | | | | | | | | | | rgw/logging: retry attribuite set in case of raceYuval Lifshitz5 days1-45/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
| * | | | | | | | | | | | | | | | | rgw/logging: log bucket must not have encryptionYuval Lifshitz5 days1-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
| * | | | | | | | | | | | | | | | | rgw/logging: source and log bucket must be differentYuval Lifshitz5 days2-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
| * | | | | | | | | | | | | | | | | rgw/logging: support source and destination buckets on different tenantsYuval Lifshitz5 days3-44/+77
| | |_|_|_|_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
* | | | | | | | | | | | | | | | | Merge pull request #61373 from anthonyeleven/fix-file-descriptionsZac Dover3 days1-4/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AsyncMessenger.cc : improve error messages Reviewed-by: Zac Dover <zac.dover@proton.me>
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-18 19:26:31 +0100