summaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* crimson/os/seastore: hide the complexity of extent indirectionYingxin Cheng2024-11-288-114/+162
| | | | | | | | | Make it easier for TM::read_pin() users to consume extent without worrying about the indirections. This basically reverts 9cdcd06caadef6cdca64a974c6525ba300c1e289 Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
* crimson/os/seastore/transaction_manager: fix to use direct key/length to get ↵Yingxin Cheng2024-11-281-7/+15
| | | | | | absent extent Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
* crimson/os/seastore/cache: adjust logsYingxin Cheng2024-11-282-39/+42
| | | | | | Mostly convert length to the hex format. Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
* crimson/os/seastore: misc cleanups and assertsYingxin Cheng2024-11-282-20/+19
| | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
* crimson/os/seastore: clarify zero length is only possible for rootYingxin Cheng2024-11-282-6/+6
| | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
* crimson/os/seastore/cached_extent: clarify invarients in the cstorsYingxin Cheng2024-11-281-20/+54
| | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
* crimson/os/seastore/cache: cleanup, use the same method to create extentsYingxin Cheng2024-11-283-5/+4
| | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
* crimson/os/seastore/cached_extent: unify extent ptr creationYingxin Cheng2024-11-283-24/+22
| | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
* Merge pull request #60721 from anuradhagadge/fix_issue_68748_add_multiple_lableAdam King2024-11-271-3/+4
|\ | | | | | | | | cephadm: Add multiple labels in single command to host Reviewed-by: Adam King <adking@redhat.com>
| * Add multiple labels in single command to hostAnuradha Gadge2024-11-131-3/+4
| | | | | | | | Signed-off-by: Anuradha Gadge <anuradha.gadge@ibm.com>
* | Merge pull request #60747 from mohit84/crimson_device_classMatan Breizman2024-11-269-0/+69
|\ \ | | | | | | | | | | | | crimson: Set device class during spawn of a crimson osd Reviewed-by: Matan Breizman <mbreizma@redhat.com>
| * | crimson: Set device class during spawn of a crimson osdMohit Agrawal2024-11-199-0/+69
| | | | | | | | | | | | | | | | | | | | | | | | Implement a wrapper for different backend storage to set device_class during spawn of a process. Fixes: https://tracker.ceph.com/issues/66627 Signed-off-by: Mohit Agrawal <moagrawa@redhat.com>
* | | Merge pull request #60756 from ArbitCode/wip-raja-rgw-fix-68211Casey Bodley2024-11-251-4/+10
|\ \ \ | | | | | | | | | | | | | | | | rgw:Wrong x-rgw-quota-bucket-objects header when bucket quota is disabled Reviewed-by: Casey Bodley <cbodley@redhat.com>
| * | | rgw:Wrong x-rgw-quota-bucket-objects header when bucket quota is disabledRaja Sharma2024-11-221-4/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit introduces to get quota settings if bucket quota is enabled Fixes: https://tracker.ceph.com/issues/68211 Signed-off-by: Raja Sharma <raja@ibm.com>
* | | | python-common/ceph/deployment: Add key encryption support to nvmeof ↵Gil Bregman2024-11-251-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | configuration Fixes https://tracker.ceph.com/issues/69036 Signed-off-by: Gil Bregman <gbregman@il.ibm.com>
* | | | mgr/cephadm: Add key encryption support to nvmeof configurationGil Bregman2024-11-252-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | Fixes https://tracker.ceph.com/issues/69036 Signed-off-by: Gil Bregman <gbregman@il.ibm.com>
* | | | Merge pull request #60775 from cbodley/wip-rgw-access-key-active-jsonCasey Bodley2024-11-221-1/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | rgw: RGWAccessKey::decode_json() preserves default value of 'active' Reviewed-by: Shilpa Jagannath <smanjara@redhat.com>
| * | | | rgw: RGWAccessKey::decode_json() preserves default value of 'active'Casey Bodley2024-11-191-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | squid added the new "bool active" member that defaults to true, but RGWAccessKey::decode_json() sets it to false when "active" isn't present this is an issue for multisite when a squid zone replicates user metadata from a master zone running reef Fixes: https://tracker.ceph.com/issues/68985 Signed-off-by: Casey Bodley <cbodley@redhat.com>
* | | | | Merge pull request #58645 from vshankar/wip-omap-commit-header-sizeIgor Golikov2024-11-221-0/+4
|\ \ \ \ \ | | | | | | | | | | | | mds: account for header size during omap commit
| * | | | | mds: account for header size during omap commitVenky Shankar2024-11-181-0/+4
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fnode_t is set in the omap header during directory commit operation which isn't accounted when tracking operation size. Fixes: http://tracker.ceph.com/issues/67597 Signed-off-by: Venky Shankar <vshankar@redhat.com>
* | | | | Merge pull request #60488 from jmolmo/exporter_manage_sigtermJuan Miguel Olmo2024-11-225-27/+97
|\ \ \ \ \ | |_|_|/ / |/| | | | exporter: SIGTERM/SIGINT/SIGHUP management
| * | | | exporter: SIGTERM/SIGINT/SIGHUP managementJuan Miguel Olmo Martínez2024-11-155-27/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ceph exporter manages SIGINT, SIGTERM and SIGHUP signals Fixes: https://tracker.ceph.com/issues/68721 Signed-off-by: Juan Miguel Olmo Martínez <jolmomar@redhat.com>
* | | | | Merge pull request #60787 from ronen-fr/wip-rf-reservStatRonen Friedman2024-11-221-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | osd/scrub: fixing reservations dump format Reviewed-by: Nitzan Mordechai <nmordech@redhat.com>
| * | | | | osd/scrub: fixing reservations dump formatRonen Friedman2024-11-211-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Headers were missing in the faulty dump output. This commit fixes the section type to 'object section'. Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
* | | | | | Merge pull request #60253 from MaxKellermann/includesYuri Weinstein2024-11-2252-16/+100
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | common: add missing include, reduce header dependencies Reviewed-by: Ronen Friedman <rfriedma@redhat.com> Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
| * | | | | | include/str_list: remove unnecessary #includeMax Kellermann2024-11-041-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/error_code: remove unused includesMax Kellermann2024-11-042-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/perf_counters: forward-declare class utime_tMax Kellermann2024-11-043-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | And add a few missing includes. Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/DecayCounter: forward-declare ceph::FormatterMax Kellermann2024-11-041-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/Thread: include cleanupMax Kellermann2024-11-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This header uses `std::string`, but not `std::system_error`. Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/Journald: add missing includeMax Kellermann2024-11-041-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/StackStringStream: remove unused includesMax Kellermann2024-11-041-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/{ceph_time,Throttle}: use <iosfwd>Max Kellermann2024-11-042-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/pick_address: add missing includesMax Kellermann2024-11-041-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/options: add missing include for strict_si_cast() and std::cerrMax Kellermann2024-11-042-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/strtol: add missing includeMax Kellermann2024-11-041-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/ceph_argparse: add missing forward declarationMax Kellermann2024-11-041-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/buffer: add missing include for std::cerrMax Kellermann2024-11-041-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/Graylog: add missing includeMax Kellermann2024-11-041-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/elist: add missing includesMax Kellermann2024-11-041-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | common/admin_socket: add missing includesMax Kellermann2024-11-041-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | tools/radosacl: add missing include for std::cerrMax Kellermann2024-11-041-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
| * | | | | | test: add missing include for std::coutMax Kellermann2024-11-0430-2/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
* | | | | | | Merge pull request #60523 from ljflores/wip-tracker-68657Laura Flores2024-11-221-17/+23
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | mgr/balancer: optimize 'balancer status detail'
| * | | | | | | mgr/balancer: tie `update_pg_upmap_activity` to a configurableLaura Flores2024-11-011-2/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This addition gives users the option of enabling/disabling this feature. They can do so by running: ceph config set mgr mgr/balancer/update_pg_upmap_activity <True|False> The feature is off by default. Signed-off-by: Laura Flores <lflores@ibm.com>
| * | | | | | | mgr/balancer: optimize 'balancer status detail'Laura Flores2024-11-011-17/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before, we were updating the balancer status by iterating through all pg upmap entires. This was affecting the loading time of other mgr modules on clusters with a large number of pgs (600+). This can be optimized by simply pulling from the incremental. Fixes: https://tracker.ceph.com/issues/68657 Signed-off-by: Laura Flores <lflores@ibm.com>
* | | | | | | | Merge pull request #60542 from adk3798/sign-generated-rgw-certsAdam King2024-11-214-12/+45
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/cephadm: sign generated RGW certs Reviewed-by: Redouane Kachach <rkachach@ibm.com>
| * | | | | | | | mgr/cephadm: sign generated RGW certsAdam King2024-10-294-12/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the "generate_cert" field would just cause cephadm to generate self-signed certificates. This was an issue when trying to sync the secondary site in a multisite situation, resulting in ``` SL peer certificate or SSH remote key was not OK req_data->error_buf=SSL certificate problem: self-signed certificate request failed: (2200) Unknown error 2200 ``` This change makes it so the certificate are signed by cephadm's root CA cert so that users may grab that cert via "ceph orch cert-store get cert cephadm_root_ca_cert" and set that as a trusted CA cert on their secondary cluster. Additionally, we now generate a cert per RGW daemon so that we can include the hostname/addr of the node we are deploying the RGW daemon on in the cert. Signed-off-by: Adam King <adking@redhat.com>
* | | | | | | | | Merge pull request #60303 from adk3798/handle-failed-daemon-startAdam King2024-11-216-4/+84
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cephadm: handle "systemctl start" failures during deployment better Reviewed-by: John Mulligan <jmulligan@redhat.com>
| * | | | | | | | | cephadm: handle "systemctl start" failures during deployment betterAdam King2024-10-246-4/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously it was assumed when the deploy command fails whatever daemon we were trying to deploy does not exist on the host. However, in the specific case where deploy fails trying to start the daemon's systemd unit this is not the case. This leads us to both cleanup the keyring for the daemon and also causes us to not trigger a refresh of the daemons on the host which can make cephadm attempt to deploy another daemon instead of just reporting the existing one as failed. To get around this we need to handle that specific failure as a success in the mgr module's deploy workflow so that we refresh the daemons and report the failure as intended. https://tracker.ceph.com/issues/68536 Signed-off-by: Adam King <adking@redhat.com>
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-19 12:31:43 +0100