summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #49710 from cyx1231st/wip-seastore-cleanup-overwriteYingxin2023-01-172-329/+446
|\ | | | | | | | | | | crimson/os/seastore/object_data_handler: cleanup the overwrite path Reviewed-by: Samuel Just <sjust@redhat.com> Reviewed-by: Xuehan Xu <xxhdx1985126@gmail.com>
| * crimson/os/seastore: add FIXME to consider pending extent with absolute paddrYingxin Cheng2023-01-162-0/+3
| | | | | | | | | | | | During object data overwrite. Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
| * crimson/os/seastore/object_data_handler: cleanup the overwrite pathYingxin Cheng2023-01-161-329/+443
| | | | | | | | | | | | | | | | | | * implement factory and type for extent_to_write_t. * refactor overwrite_plan_t with explicit size calculations and defined operations. * cleanup the overwrite path accordingly. Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
* | Merge pull request #49743 from xxhdx1985126/wip-omap-fixesYingxin2023-01-173-10/+14
|\ \ | | | | | | | | | | | | crimson/os/seastore/omap_manager: minor bugs fixes Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
| * | test/crimson/seastore/test_omap_manager: keep test_omap_mappings consistent ↵Xuehan Xu2023-01-161-1/+1
| | | | | | | | | | | | | | | | | | with "rm_key_range" config Signed-off-by: Xuehan Xu <xxhdx1985126@gmail.com>
| * | crimson/os/seastore/omap_manager: fix a capture-by-reference related issueXuehan Xu2023-01-162-9/+13
|/ / | | | | | | | | | | | | Variables that are not guaranteed to be available as long as the continuations live shouldn't be captured by reference Signed-off-by: Xuehan Xu <xxhdx1985126@gmail.com>
* | Merge pull request #49725 from anthonyeleven/56521zdover232023-01-161-4/+13
|\ \ | | | | | | | | | | | | doc/releases: Clarify Quincy CephFS MDS upgrade process Reviewed-by: Zac Dover <zac.dover@gmail.com>
| * | doc/releases: Clarify Quincy CephFS MDS upgrade processAnthony D'Atri2023-01-121-4/+13
| | | | | | | | | | | | Signed-off-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
* | | Merge pull request #49747 from zdover23/wip-doc-2023-01-16-luks-versionsAnthony D'Atri2023-01-161-4/+1
|\ \ \ | | | | | | | | doc/ceph-volume: update LUKS docs
| * | | doc/ceph-volume: update LUKS docsZac Dover2023-01-151-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove references that claim that Ceph uses only LUKS version 1. https://tracker.ceph.com/issues/58354 Signed-off-by: Zac Dover <zac.dover@gmail.com>
* | | | Merge pull request #49716 from ↵zdover232023-01-161-31/+37
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | zdover23/wip-doc-2023-01-12-rbd-iscsi-initiator-linux-prompts doc/rbd: refine iscsi-initiator-linux.rbd Reviewed-by: Ilya Dryomov <idryomov@redhat.com> Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
| * | | doc/rbd: format iscsi-initiator-linux.rbd betterZac Dover2023-01-151-31/+37
| | | | | | | | | | | | | | | | | | | | | | | | Add prompts and clean up the lists in doc/rbd/iscsi-initiator-linux.rbd. Signed-off-by: Zac Dover <zac.dover@gmail.com>
* | | | Merge pull request #49745 from ↵colemitchell2023-01-151-0/+21
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | zdover23/wip-doc-2023-01-15-start-documenting-ceph-escaping-bold-characters-within-words doc/start: add RST escape character rules for bold Reviewed-by: Cole Mitchell <cole.mitchell@gmail.com>
| * | | doc/start: add RST escape character rules for boldZac Dover2023-01-151-0/+21
|/ / / | | | | | | | | | | | | | | | Explain how to escape the bold notation (**) within words in RST. Signed-off-by: Zac Dover <zac.dover@gmail.com>
* | | Merge pull request #49736 from ↵colemitchell2023-01-131-0/+32
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | zdover23/wip-doc-2023-01-14-dev-basic-workflow-git-commands doc/dev: add git branch management commands Reviewed-by: Cole Mitchell <cole.mitchell@gmail.com>
| * | | doc/dev: add git branch management commandsZac Dover2023-01-131-0/+32
|/ / / | | | | | | | | | | | | | | | | | | Add git branch deleting and search commands to the "Basic Workflow" page of the Developer Guide. Signed-off-by: Zac Dover <zac.dover@gmail.com>
* | | Merge pull request #48496 from adk3798/apply-spec-parsingAdam King2023-01-132-60/+91
|\ \ \ | | | | | | | | | | | | | | | | | | | | cephadm: only pull host info from applied spec, don't try to parse yaml Reviewed-by: John Mulligan <jmulligan@redhat.com> Reviewed-by: Redouane Kachach <rkachach@redhat.com>
| * | | cephadm: only pull host info from applied specAdam King2022-11-302-60/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We don't need to actually try to properly parse the yaml spec for our purposes, jsut pull the host info out from it. Fixes: https://tracker.ceph.com/issues/57870 Signed-off-by: Adam King <adking@redhat.com>
* | | | Merge pull request #49024 from rkachach/fix_issue_58067Adam King2023-01-133-2/+7
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | mgr/cephadm: fixing mgr port shown in ceph orch ps Reviewed-by: Adam King <adking@redhat.com>
| * | | | mgr/cephadm: fixing mgr port shown in ceph orch psRedouane Kachach2022-11-233-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/58067 Signed-off-by: Redouane Kachach <rkachach@redhat.com>
* | | | | Merge PR #49731 into mainPatrick Donnelly2023-01-131-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/49731/head: tools/cephfs: fix traceback from missing envvar Reviewed-by: Milind Changire <mchangir@redhat.com>
| * | | | | tools/cephfs: fix traceback from missing envvarPatrick Donnelly2023-01-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* | | | | | Merge pull request #48617 from adk3798/upgrade-status-helpAdam King2023-01-131-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/orchestrator: fix upgrade status help message Reviewed-by: Redouane Kachach <rkachach@redhat.com>
| * | | | | | mgr/cephadm: fix upgrade status help messageAdam King2022-10-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This help message for this command had just been copied over from the `upgrade check` command, which doesn't make sense for `upgrade status` Fixes: https://tracker.ceph.com/issues/57921 Signed-off-by: Adam King <adking@redhat.com>
* | | | | | | Merge pull request #48384 from jmolmo/Rook_PDFetcherAdam King2023-01-132-7/+52
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/rook: Device inventory Reviewed-by: Nizamudeen A <nia@redhat.com> Reviewed-by: Travis Nielsen <tnielsen@redhat.com>
| * | | | | | | mgr/rook: Device inventoryJuan Miguel Olmo Martínez2022-10-072-7/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Recovered device inventory list for host clusters Signed-off-by: Juan Miguel Olmo Martínez <jolmomar@redhat.com>
* | | | | | | | Merge pull request #48592 from adk3798/offline-upgradeAdam King2023-01-135-9/+99
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/cephadm: improve offline host handling, mostly around upgrade Reviewed-by: Redouane Kachach <rkachach@redhat.com>
| * | | | | | | | mgr/cephadm: remove host from offline_hosts list when removing hostAdam King2022-10-211-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise, it could remain in the list and cephadm could think there is an offline host in the cluster when said host has actually already been removed. Signed-off-by: Adam King <adking@redhat.com>
| * | | | | | | | mgr/cephadm: clear upgrade health error when upgrade is resumedAdam King2022-10-212-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The health error is no longer valid if the upgrade has been removed. If the issue is still present we'll hit it again. Fixes: https://tracker.ceph.com/issues/57891 Signed-off-by: Adam King <adking@redhat.com>
| * | | | | | | | mgr/cephadm: unit tests for upgrade offline host scenariosAdam King2022-10-211-1/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Adam King <adking@redhat.com>
| * | | | | | | | mgr/cephadm: update check-host to handle new HostConnectionError exceptionAdam King2022-10-212-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise we'll return a message about the host not being found and to check 'ceph orch host ls' when the actual problem is the host being offline Signed-off-by: Adam King <adking@redhat.com>
| * | | | | | | | mgr/cephadm: raise a better error on connection error during upgradeAdam King2022-10-211-1/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Right now failures to connect to a host during the upgrade result in a "failed due to an unexpected exception" error. We can do a bit better than that. Fixes: https://tracker.ceph.com/issues/57891 Signed-off-by: Adam King <adking@redhat.com>
| * | | | | | | | mgr/cephadm: introduce HostConnectionError exception type in ssh handlingAdam King2022-10-211-5/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The idea is to be able to know elsewhere that the OrchestratorError we are looking at is specifically one raised due to a failure to connnect to a host. This can hopefully allow for some more precise error handling Signed-off-by: Adam King <adking@redhat.com>
| * | | | | | | | mgr/cephadm: abort upgrade if there are offline hosts.Adam King2022-10-211-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We won't be able to complete the upgrade if there are offline hosts anyway so we might as well abort immediately. Signed-off-by: Adam King <adking@redhat.com>
* | | | | | | | | Merge pull request #48574 from adk3798/debug-log-spamAdam King2023-01-137-29/+48
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/cephadm: make logging refresh metadata to debug logs configurable Reviewed-by: Redouane Kachach <rkachach@redhat.com>
| * | | | | | | | | mgr/cephadm: make logging refresh metadata to debug logs configurableAdam King2022-10-217-29/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This output is so verbose it actually makes the debug logs difficult to use. The info logged from one host refresh can take up over 1000 lines in the logs. I also find I rarely actually need this info for debugging, so having it be something you can toggle would be nice. Fixes: https://tracker.ceph.com/issues/57909 Signed-off-by: Adam King <adking@redhat.com>
* | | | | | | | | | Merge pull request #47383 from rhcs-dashboard/ceph-exporter-cephadm-changesAdam King2023-01-1312-16/+224
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | exporter: cephadm changes Reviewed-by: Adam King <adking@redhat.com> Reviewed-by: Juan Miguel Olmo Martínez <jolmomar@redhat.com> Reviewed-by: Redouane Kachach <rkachach@redhat.com>
| * | | | | | | | | | ceph-exporter: cephadm changesAvan Thakkar2022-12-1212-16/+224
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/55046 Signed-off-by: Avan Thakkar <athakkar@redhat.com> Includes changes required for introducing ceph-exporter container in cephadm.
* | | | | | | | | | | Merge pull request #49389 from adk3798/git-archive-fixAdam King2023-01-131-6/+7
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qa/tasks/cephadm.py: fix pulling cephadm from git.ceph.com Reviewed-by: Redouane Kachach <rkachach@redhat.com>
| * | | | | | | | | | | qa/tasks/cephadm.py: fix pulling cephadm from git.ceph.comAdam King2022-12-121-6/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The old way of using the git archive doesn't seem to work anymore so trying another approach Was getting "fatal: operation not supported by protocol" when trying the git archive Fixes: https://tracker.ceph.com/issues/58222 Signed-off-by: Adam King <adking@redhat.com>
* | | | | | | | | | | | Merge pull request #49726 from rhcs-dashboard/cephadm-e2e-fixesNizamudeen A2023-01-133-8/+20
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/dashboard: cephadm dashboard e2e fixes Reviewed-by: Aashish Sharma <aasharma@redhat.com> Reviewed-by: Avan Thakkar <athakkar@redhat.com> Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
| * | | | | | | | | | | | mgr/dashboard: cephadm dashboard e2e fixesNizamudeen A2023-01-133-8/+20
| | |_|_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/58406 Signed-off-by: Nizamudeen A <nia@redhat.com>
* | | | | | | | | | | | Merge pull request #49635 from rkachach/fix_issue_58378Adam King2023-01-131-4/+10
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/mgr_util: fixing some tox issues Reviewed-by: Adam King <adking@redhat.com> Reviewed-by: John Mulligan <jmulligan@redhat.com>
| * | | | | | | | | | | | mgr/mgr_util: fixing some tox issuesRedouane Kachach2023-01-052-18/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/58378 Signed-off-by: Redouane Kachach <rkachach@redhat.com>
* | | | | | | | | | | | | Merge pull request #49417 from athanatos/sjust/backburner/wip-smp-optionSamuel Just2023-01-137-123/+419
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | crimson/osd/main: get full config before starting reactor Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
| * | | | | | | | | | | | crimson/osd/main: get full config before starting reactorSamuel Just2023-01-057-123/+419
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Deployment tooling these days relies on fetching configs from the mons. We'd like to be able to set seastar reactor params like --smp via mon config, so we need to start up a reactor and contact the mons before starting up the reactor for real. Seastar doesn't seem to like invoking app.run more than once -- a bunch of thread-locals and statics get left around and result in invalid memory accesses. To avoid all of this, we simply fork a child process to talk to the mons and send the information back over a fifo. Signed-off-by: Samuel Just <sjust@redhat.com>
* | | | | | | | | | | | | Merge pull request #49713 from rhcs-dashboard/fix-sse-daemon-nameAashish Sharma2023-01-121-15/+16
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/dashboard: rgw server side encryption daemon name fix Reviewed-by: Nizamudeen A <nia@redhat.com>
| * | | | | | | | | | | | mgr/dashboard: rgw server side encryption daemon name fixAashish Sharma2023-01-111-15/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The config values for the server side encryption currently are wrongly set to the mon daemon by default. This PR intends to fix this issue Fixes: https://tracker.ceph.com/issues/58419 Signed-off-by: Aashish Sharma <aasharma@redhat.com>
* | | | | | | | | | | | | Merge pull request #49540 from xxhdx1985126/wip-rebuild-missing-setYingxin2023-01-126-64/+175
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | crimson/osd/pg: rebuild missing set when a new interval is created Reviewed-by: Samuel Just <sjust@redhat.com> Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
| * | | | | | | | | | | | | crimson/osd/pg: rebuild missing set when a new interval is createdXuehan Xu2023-01-063-1/+92
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/58339 Signed-off-by: Xuehan Xu <xxhdx1985126@gmail.com>
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-19 12:25:34 +0100