summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* 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 #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 Lifshitz6 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 Lifshitz6 days1-45/+56
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
| * | | | | rgw/logging: log bucket must not have encryptionYuval Lifshitz6 days1-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
| * | | | | rgw/logging: source and log bucket must be differentYuval Lifshitz6 days2-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
| * | | | | rgw/logging: support source and destination buckets on different tenantsYuval Lifshitz6 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>
| * | | | | | AsyncMessenger.cc : improve error messagesAnthony D'Atri4 days1-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
* | | | | | | Merge pull request #61289 from ronen-fr/wip-rf-catcher-fixRonen Friedman3 days4-50/+58
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | common: fix md_config_cacher_t Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
| * | | | | | | qa/scrub: more delay when waiting for noscrub to take effectRonen Friedman4 days1-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
| * | | | | | | common: config_cacher: use set::contains() instead of count()Ronen Friedman4 days1-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | moved out of the main commit to facilitate backporting it to pre-C++20. Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
| * | | | | | | qa/scrub: change 'bin/ceph' to 'ceph'Ronen Friedman4 days3-39/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
| * | | | | | | common: fix md_config_cacher_tRonen Friedman4 days1-7/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In its get_tracked_conf_keys() member function, the cacher (in the existing code) initializes a static function-block variable ('keys'), and uses it for registering the observer. But the cacher is instantiated on the type of the configuration value. Thus, multiple cacher objects for which the configuration values are of the same type - share the static 'keys'. Only one of the observers is registered. Note that the code could have been simplified somewhat, if the signature of the get_tracked_conf_keys() function was changed to return 'const char* const *'. Fixes: https://tracker.ceph.com/issues/69236 Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
* | | | | | | | Merge pull request #61072 from pecastro/doc_and_test_fixesAnthony D'Atri4 days3-7/+7
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Doc and test fixes
| * | | | | | | | tools/cephfs: fix flake8 f-string formatting for py3.12Paulo E. Castro2024-12-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Paulo E. Castro <pecastro@wormholenet.com>
| * | | | | | | | doc: Fix typo.Paulo E. Castro2024-12-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Paulo E. Castro <pecastro@wormholenet.com>
| * | | | | | | | test/pybind: Test method has been renamed in unittest 3.2Paulo E. Castro2024-12-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Paulo E. Castro <pecastro@wormholenet.com>
| * | | | | | | | test/pybind: Clean whitespace.Paulo E. Castro2024-12-121-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Paulo E. Castro <pecastro@wormholenet.com>
* | | | | | | | | Merge pull request #59739 from sabaini/fix/ceph-volume-epAdam King4 days1-1/+9
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ceph-volume] fix importlib.metadata compat Reviewed-by: Guillaume Abrioux <gabrioux@ibm.com> Reviewed-by: John Mulligan <jmulligan@redhat.com>
| * | | | | | | | | ceph-volume: fix importlib.metadata compatPeter Sabaini2024-11-191-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The importlib.metadata library removed older shims in releases >5.0.0 where EntryPoints objects use .select() instead of dict-like access. Fixes: https://tracker.ceph.com/issues/68032 Signed-off-by: Peter Sabaini <peter.sabaini@canonical.com>
* | | | | | | | | | Merge pull request #60073 from mkogan1/wip-fix-rate-limCasey Bodley4 days7-83/+113
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rgw: fix user rate limit is not enforced w/ global rate limit set Reviewed-by: Casey Bodley <cbodley@redhat.com>
| * | | | | | | | | | rgw: fix user rate limit is not enforced w/ global rate limit setMark Kogan12 days7-83/+113
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/68346 Signed-off-by: Mark Kogan <mkogan@redhat.com>
* | | | | | | | | | | Merge pull request #61322 from ↵Jon Bailey4 days1-36/+60
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | JonBailey1993/JonBailey1993/ceph_test_rados_io_sequence_ostringstream_fix common/io_exerciser: Ensure empty stringstream in ceph_test_rados_io_sequence RadosIO between ReST calls Reviewed-by: Ronen Friedman <rfriedma@redhat.com>
| * | | | | | | | | | | common/io_exerciser: Reorder lines in RadosIo send_command functionsJon Bailey5 days1-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
| * | | | | | | | | | | common/io_exerciser: Clear the stringstream in RadosIO classJon Bailey5 days1-36/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ensures the stringstream is empty before sending a command in the RadosIO class Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
* | | | | | | | | | | | Merge pull request #61352 from zdover23/wip-doc-2025-01-14-releases-quincy-eolZac Dover4 days1-0/+1
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | doc/releases: add actual_eol for quincy Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
| * | | | | | | | | | | | doc/releases: add actual_eol for quincyZac Dover5 days1-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the actual EOL date for the Quincy release (it's 2025-01-13). Signed-off-by: Zac Dover <zac.dover@proton.me>
* | | | | | | | | | | | | Merge pull request #61314 from aclamk/wip-aclamk-bluefs-truncate-fixAdam Kupczyk4 days4-2/+92
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | os/bluestore: Fix BlueFS::truncate()
| * | | | | | | | | | | | | os/bluestore: Fix BlueFS::truncate()Adam Kupczyk5 days3-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In `struct bluefs_fnode_t` there is a vector `extents` and the vector `extents_index` that is a log2 seek cache. Until modifications to truncate() we never removed extents from files. Modified truncate() did not update extents_index. For example 10 extents long files when truncated to 0 will have: 0 extents, 10 extents_index. After writing some data to file: 1 extents, 11 extents_index. Now, `bluefs_fnode_t::seek` will binary search extents_index, lets say it located seek at item #3. It will then jump up from #0 extent (that exists) to #3 extent which does not exist at. The worst part is that code is now broken, as #3 != extent.end(). There are 3 parts of the fix: 1) assert in `bluefs_fnode_t::seek` to protect against jumping outside extents 2) code in BlueFS::truncate to sync up `extents_index` with `extents` 3) dampening down assert in _replay to give a way out of cases where incorrect "offset 12345" (12345 is file size) instead of "offset 20000" (allocations occupied) was written to log. Fixes: https://tracker.ceph.com/issues/69481 Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>
| * | | | | | | | | | | | | os/bluestore: bluefs unittest for truncate bugAdam Kupczyk8 days1-0/+85
| | |/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unittest showing 2 different flavours of problems: 1) bluefs log corruption 2) bluefs sigsegv Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>
* | | | | | | | | | | | | Merge pull request #61261 from xxhdx1985126/wip-seastore-lba-backref-mappingYingxin Cheng5 days14-190/+221
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | crimson/os/seastore: refactor LBAMapping Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
| * | | | | | | | | | | | | crimson/os/seastore: refactor LBAMappingXuehan Xu9 days14-190/+221
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So that PhyscalNodeMapping doesn't contain LBAMapping specific interfaces Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
* | | | | | | | | | | | | | Merge pull request #60215 from MaxKellermann/osd_optimizationsSrinivasaBharathKanta5 days4-6/+5
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OSD: Small optimizations
| * | | | | | | | | | | | | | osd/osd_types: perfect forwarding in pool_opts_t::set()Max Kellermann2024-11-131-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This eliminates temporary objects. Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | | | | | | | | | osd/osd_types: use std::map::insert_or_assign()Max Kellermann2024-11-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This eliminates the overhead for constructing an empty value if the key does not yet exist. Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | | | | | | | | | osdc/Objecter: use the erase() return valueMax Kellermann2024-11-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | | | | | | | | | osd: disable `boost::intrusive::constant_time_size`Max Kellermann2024-11-132-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By default, the Boost intrusive containers enable the `constant_time_size` option which adds overhead to each modification for tracking the size in a field. We don't need that. Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
* | | | | | | | | | | | | | | Merge pull request #61312 from adamemerson/wip-boost-1.86-fixesIlya Dryomov5 days4-25/+37
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | {mgr, rbd, tests/os}: Fixes for Boost 1.86 Reviewed-by: Casey Bodley <cbodley@redhat.com> Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
| * | | | | | | | | | | | | | | librbd/migration/HttpClient: Use asio::ssl::streamAdam Emerson5 days3-25/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `beast::ssl_stream` is deprecated as of 1.86, and its loss of the move constructor keeps it from compiling on that version. Hopefully this passes tests on 1.85, too, or it will have to wait until the boost bump. Signed-off-by: Adam Emerson <aemerson@redhat.com>
| * | | | | | | | | | | | | | | mgr: Work around bug in Boost MPI/PythonAdam Emerson6 days1-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Thanks to the maintainers of the Arch User Repository Ceph PKGBUILD for this fix. Signed-off-by: Adam Emerson <aemerson@redhat.com>
* | | | | | | | | | | | | | | | Merge pull request #61284 from shraddhaag/wip-shraddhaag-fix-rados-monthrashLaura Flores5 days1-0/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | | | | rados/monthrash: add PG_DEGRADED to ignorelist
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-18 20:45:37 +0100