Skip to content

squid: cephfs-journal-tool: Journal trimming issue#65602

Merged
neesingh-rh merged 2 commits intoceph:squidfrom
kotreshhr:wip-73130-squid
Jan 19, 2026
Merged

squid: cephfs-journal-tool: Journal trimming issue#65602
neesingh-rh merged 2 commits intoceph:squidfrom
kotreshhr:wip-73130-squid

Conversation

@kotreshhr
Copy link
Contributor

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


backport of #65577
parent tracker: https://tracker.ceph.com/issues/69708

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

Validates that the cephfs-journal-tool reset
doesn't reset the trim position so that the
journal trim takes care of trimming the older
unused journal objects helping to recover the
space in metadata pool.

Fixes: https://tracker.ceph.com/issues/69708
Signed-off-by: Kotresh HR <khiremat@redhat.com>
(cherry picked from commit 7e85556)
If the fs had to go through journal recovery and reset,
the cephfs-journal-tool resets the journal trim position
because of which the old unused journal objects just stay
forever in the metadata pool. The patch fixes the issue.
Now, the old stale journal objects are trimmed during the
regular trimming cycle helping to recover space in the
metadata pool.

Fixes: https://tracker.ceph.com/issues/69708
Signed-off-by: Kotresh HR <khiremat@redhat.com>
(cherry picked from commit 4f9a926)
@kotreshhr kotreshhr added this to the squid milestone Sep 19, 2025
@kotreshhr kotreshhr added the cephfs Ceph File System label Sep 19, 2025
@github-actions github-actions bot added the tests label Sep 19, 2025
@joscollin
Copy link
Member

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

Copy link
Contributor

@neesingh-rh neesingh-rh left a comment

Choose a reason for hiding this comment

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

@neesingh-rh neesingh-rh merged commit d7af23e into ceph:squid Jan 19, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cephfs Ceph File System tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants