mds: ensure snapclient is synced before corruption check#55412
mds: ensure snapclient is synced before corruption check#55412
Conversation
|
I thought there was an outstanding ticket for this (as I've seen it before) but I cannot locate it. |
|
@batrick Could you explain a bit more about why this warning shows up due to snap client not synchorized with snap server? (and also include it in the commit message) |
During resolve, orphan dirfrags are purged which may result in dentry commits. If the snapclient is not sync'd (which normally happens as the last step of up:resolve), then the snapclient indicates the last sequence is 1 which is always less than a dentry's legitimate "first" value. Fixes: https://tracker.ceph.com/issues/64058 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Please see the commit message and https://tracker.ceph.com/issues/64058#note-2 |
Makes more sense now that the context is set :) |
|
https://pulpito.ceph.com/?branch=wip-vshankar-testing-20240307.013758 (test run review pending) |
Fixes: https://tracker.ceph.com/issues/64058
Contribution Guidelines
To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an
xbetween the brackets:[x]. Spaces and capitalization matter when checking off items this way.Checklist
Show available Jenkins commands
jenkins retest this pleasejenkins test classic perfjenkins test crimson perfjenkins test signedjenkins test make checkjenkins test make check arm64jenkins test submodulesjenkins test dashboardjenkins test dashboard cephadmjenkins test apijenkins test docsjenkins render docsjenkins test ceph-volume alljenkins test ceph-volume toxjenkins test windowsjenkins test rook e2e