Skip to content

squid: mds: add MDS asok command for dumping stray directories#62517

Merged
joscollin merged 5 commits intoceph:squidfrom
salieri11:wip-70631-squid
Sep 23, 2025
Merged

squid: mds: add MDS asok command for dumping stray directories#62517
joscollin merged 5 commits intoceph:squidfrom
salieri11:wip-70631-squid

Conversation

@salieri11
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/70631


backport of #60684
parent tracker: https://tracker.ceph.com/issues/56442

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

@salieri11 salieri11 requested a review from a team as a code owner March 26, 2025 11:25
@salieri11 salieri11 added this to the squid milestone Mar 26, 2025
@salieri11 salieri11 added the cephfs Ceph File System label Mar 26, 2025
@salieri11 salieri11 self-assigned this Mar 26, 2025
@salieri11 salieri11 requested a review from a team May 11, 2025 09:59
salieri11 added 5 commits May 11, 2025 10:00
the rename makes the code consistent and allows to call the
`MDSContext::finish` method from the `C_MDS_DumpStrayDirCtx::finish` method,
which fortifies the safety of the new dump stray folder command.

Signed-off-by: Igor Golikov <igolikov@ibm.com>
Fixes: http://tracker.ceph.com/issues/56442
(cherry picked from commit df404e0)
Signed-off-by: Igor Golikov <igolikov@ibm.com>
Fixes: https://tracker.ceph.com/issues/56442
(cherry picked from commit 801951e)
Signed-off-by: Igor Golikov <igolikov@ibm.com>
Fixes: https://tracker.ceph.com/issues/56442
(cherry picked from commit ee2beec)
Fixes: https://tracker.ceph.com/issues/56442
Signed-off-by: Igor Golikov <igolikov@ibm.com>
(cherry picked from commit f858fa9)
close object section only if it was opened

Signed-off-by: Igor Golikov <igolikov@ibm.com>
Fixes: https://tracker.ceph.com/issues/70707
@salieri11
Copy link
Contributor Author

jenkins please retest this

@salieri11
Copy link
Contributor Author

jenkins test make check

@joscollin
Copy link
Member

This PR is under test in https://tracker.ceph.com/issues/72883.

@salieri11
Copy link
Contributor Author

@joscollin , the PR was updated with the bug fix, as discussed

Copy link
Member

@joscollin joscollin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joscollin joscollin merged commit 2661c22 into ceph:squid Sep 23, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants