Skip to content

squid: mds: check for snapshots on parent snaprealms#63019

Merged
joscollin merged 4 commits intoceph:squidfrom
batrick:wip-71094-squid
Sep 23, 2025
Merged

squid: mds: check for snapshots on parent snaprealms#63019
joscollin merged 4 commits intoceph:squidfrom
batrick:wip-71094-squid

Conversation

@batrick
Copy link
Member

@batrick batrick commented Apr 28, 2025

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


backport of #62833
parent tracker: https://tracker.ceph.com/issues/70912

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

batrick added 4 commits April 28, 2025 15:36
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
(cherry picked from commit 89d3fb0)
This allows false/true as a value.

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
(cherry picked from commit 58e5c94)
The code was incorrectly only looking at snapshots in the current snaprealm.

Fixes: https://tracker.ceph.com/issues/70912
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
(cherry picked from commit b8fd8e7)
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
(cherry picked from commit 05ca22f)
@batrick batrick added this to the v16.2.15 milestone Apr 28, 2025
@batrick batrick added the cephfs Ceph File System label Apr 28, 2025
@joscollin
Copy link
Member

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

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.

This needs a rerun separately as charmap is not involved in QA somehow.

@joscollin
Copy link
Member

This needs a rerun separately as charmap is not involved in QA somehow.

https://pulpito.ceph.com/jcollin-2025-09-23_02:29:10-fs:functional-wip-jcollin-testing-20250912.020127-squid-distro-default-smithi/

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 5373fc7 into ceph:squid Sep 23, 2025
10 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