summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Add Abhishek Kane into peoplemap, githubmap, organizationmap, mailmapabhishek-kane3 days4-0/+4
| | | | Signed-off-by: abhishek-kane <abhishek.kane@ibm.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
| * | | | | | | | rados/monthrash: add PG_DEGRADED to ignorelistShraddha Agrawal9 days1-0/+1
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Issue: mon/test_mon_osdmap_prune.sh is failing due to PG_DEGRADED state. Cause: We are writing an objects to a newly created PG before it has enough OSDs in acting set to create enough replicas. This is expected behaviour, thus the solution is to add the warning to ignorelist. Fixes: https://tracker.ceph.com/issues/69288 Signed-off-by: Shraddha Agrawal <shraddha.agrawal000@gmail.com>
* | | | | | | | Merge pull request #61311 from ↵Yuri Weinstein5 days2-3/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | zdover23/wip-doc-2025-01-10-releases-remove-quincy-from-active doc/releases: remove Quincy from "Active" Reviewed by: Laura Flores <lflores@redhat.com> Reviewed-by: Yuri Weinstein <yweinste@redhat.com> Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
| * | | | | | | | qa/tests: removed quincy from crontabYuri Weinstein9 days1-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Yuri Weinstein <yweinste@redhat.com>
| * | | | | | | | doc/releases: remove Quincy from "Active"Zac Dover9 days1-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove "Quincy" from the list of "Active Releases" in doc/releases/index.rst. Signed-off-by: Zac Dover <zac.dover@proton.me>
* | | | | | | | | Merge pull request #61208 from ↵Adam King5 days1-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Yonatan-Zaken/restore_selinux_context_when_writing_remote_files_on_host mgr/cephadm: Restore client file(s) default SELinux context type Reviewed-by: Adam King <adking@redhat.com> Reviewed-by: John Mulligan <jmulligan@redhat.com>
| * | | | | | | | | mgr/cephadm: Restore client file(s) default SELinux context typezaken13 days1-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When cephadm writes client files to hosts, the new files are created in /tmp/cephadm-<fsid> and then moved to their target directory. This makes the client files receive the usr_tmp_t context, which is different then their original context. mv -Z will set the SELinux security context type of the target file based on the SELinux policy rules for the target directory. Fixes: https://tracker.ceph.com/issues/69395 Signed-off-by: zaken <yonatan.zaken.ext@nokia.com>
* | | | | | | | | | Merge pull request #61099 from rkachach/fix_issue_69252Adam King5 days2-3/+13
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/cephadm: fixing logic in cert-store save_cert method Reviewed-by: Adam King <adking@redhat.com> Reviewed-by: John Mulligan <jmulligan@redhat.com>
| * | | | | | | | | | mgr/cephadm: fixing logic in cert-store save_cert methodRedouane Kachach2024-12-162-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the previous code was not saving the certificates correctly, it just overwritten the same entry reducing all the certificates to only one. The new logic traverse and save all the certificates from the cert dictionary. Fixes: https://tracker.ceph.com/issues/69252 Signed-off-by: Redouane Kachach <rkachach@ibm.com>
* | | | | | | | | | | Merge pull request #61035 from ↵Adam King5 days1-2/+14
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ShwetaBhosale1/fix_issue_69192_getting_error_if_yaml_file_has_trailing_tab mgr/cephadm: mgr orchestrator module raise exception if there is trailing tab in yaml file Reviewed-by: Adam King <adking@redhat.com>
| * | | | | | | | | | | mgr/cephadm: mgr orchestrator module raise exception if there is trailing ↵Shweta Bhosale12 days1-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tab in yaml file Fixes: https://tracker.ceph.com/issues/69192 Signed-off-by: Shweta Bhosale <Shweta.Bhosale1@ibm.com>
* | | | | | | | | | | | Merge pull request #60888 from ↵Adam King5 days7-22/+72
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ShwetaBhosale1/fix_issue_69070_only_bind_ports_on_network_parameter_spec_for_alertmanager mgr/cephadm: Add only_bind_ports_on_network spec parameter for Alertmanager Reviewed-by: Adam King <adking@redhat.com>
| * | | | | | | | | | | | mgr/cephadm: Add only_bind_ports_on_network spec parameter for AlertmanagerShweta Bhosale2024-12-037-22/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/69070 Signed-off-by: Shweta Bhosale <Shweta.Bhosale1@ibm.com>
* | | | | | | | | | | | | Merge pull request #60828 from rkachach/fix_prometheus_targets_discovery_mtlsAdam King5 days4-0/+41
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/cephadm: fixing config files for prometheus and alertmanager Reviewed-by: Adam King <adking@redhat.com>
| * | | | | | | | | | | | | mgr/cephadm: fixing config files for prometheus and alertmanagerRedouane Kachach2024-11-294-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when mgmt-gateway is enabled mTLS is enforced and all the communication peers (monitoring daemons in this case) have to provide a valid certificate (signed by cephadm Root CA). Fixes: https://tracker.ceph.com/issues/69018 Signed-off-by: Redouane Kachach <rkachach@ibm.com>
* | | | | | | | | | | | | | Merge pull request #60690 from ↵Adam King5 days7-2/+149
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ShwetaBhosale1/fix_issue_68695_ceph_orch_commands_does_not_work_for_deployment_name_osd mgr/cephadm: ceph orch <start/stop/restart> commands should work for service name osd and command to updated service name for osd Reviewed-by: Adam King <adking@redhat.com>
| * | | | | | | | | | | | | | ceph orch <start/stop/restart> commands should work for service name osd and ↵Shweta Bhosale2024-12-055-1/+123
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | added a new command to updated service name for osd Fixes: https://tracker.ceph.com/issues/68695 Signed-off-by: Shweta Bhosale <Shweta.Bhosale1@ibm.com>
| * | | | | | | | | | | | | | Documentation changes for 'ceph orch osd set-spec-affinity' commandShweta Bhosale2024-12-052-1/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/68695 Signed-off-by: Shweta Bhosale <Shweta.Bhosale1@ibm.com>
* | | | | | | | | | | | | | | Merge pull request #58604 from adk3798/cephadm-rgw-no-multisite-trafficAdam King5 days4-0/+57
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/cephadm: allow disabling rgw_run_sync_thread through spec Reviewed-by: Redouane Kachach <rkachach@ibm.com>
| * | | | | | | | | | | | | | | doc/cephadm: documentation for RGW spec "disable_multisite_sync_traffic" optionAdam King2024-10-301-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Adam King <adking@redhat.com>
| * | | | | | | | | | | | | | | mgr/cephadm: allow disabling rgw_run_sync_thread through specAdam King2024-10-303-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is desirable for an RGW service that you don't want to send multisite sync data so the RGW daemons can focus more on handling IO. Note they may still receive mulsitie sync data unless they are removed from the zone and zonegroup replication endpoints Signed-off-by: Adam King <adking@redhat.com>
* | | | | | | | | | | | | | | | Merge pull request #61272 from laimis9133/mainAnthony D'Atri5 days1-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | doc/radosgw/config-ref: fix lc worker thread tuning