Skip to content

squid: mds: wrong snap check for directory with parent snaps#65824

Open
joscollin wants to merge 4 commits intoceph:squidfrom
joscollin:wip-72283-squid
Open

squid: mds: wrong snap check for directory with parent snaps#65824
joscollin wants to merge 4 commits intoceph:squidfrom
joscollin:wip-72283-squid

Conversation

@joscollin
Copy link
Member

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


backport of #63524
parent tracker: https://tracker.ceph.com/issues/71462

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

This test is checking for failure conditions.

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
Fixes: https://tracker.ceph.com/issues/71462
(cherry picked from commit c428149)
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
Fixes: https://tracker.ceph.com/issues/71462
(cherry picked from commit 7678dbf)
If the parent directory has snapshots but the child was created after, then we
should be able to modify its charmap.

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
Fixes: https://tracker.ceph.com/issues/71462
(cherry picked from commit 659e426)
To look if the directory's first is beyond the last snap. This matches the behavior of lssnaps.

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
Fixes: https://tracker.ceph.com/issues/71462
(cherry picked from commit c22db4e)
@joscollin joscollin added this to the squid milestone Oct 8, 2025
@joscollin joscollin added the cephfs Ceph File System label Oct 8, 2025
@github-actions github-actions bot added the tests label Oct 8, 2025
@joscollin
Copy link
Member Author

jenkins test make check

5 similar comments
@joscollin
Copy link
Member Author

jenkins test make check

@joscollin
Copy link
Member Author

jenkins test make check

@joscollin
Copy link
Member Author

jenkins test make check

@joscollin
Copy link
Member Author

jenkins test make check

@joscollin
Copy link
Member Author

jenkins test make check

@joscollin
Copy link
Member Author

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

@joscollin
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants