Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mds: properly reconnect client caps after loading inodes #21769

Merged
merged 1 commit into from May 11, 2018

Conversation

ukernel
Copy link
Contributor

@ukernel ukernel commented May 2, 2018

Commit e43c02d "mds: filter out blacklisted clients when importing
caps" makes MDCache::process_imported_caps() ignore clients that are
not in MDCache::rejoin_imported_session_map. The map does not contain
clients from which mds has received reconnect messages. This causes
some client caps (corresponding inodes were not in cache when mds was
in reconnect state) to get dropped.

Signed-off-by: "Yan, Zheng" zyan@redhat.com

Commit e43c02d "mds: filter out blacklisted clients when importing
caps" makes MDCache::process_imported_caps() ignore clients that are
not in MDCache::rejoin_imported_session_map. The map does not contain
clients from which mds has received reconnect messages. This causes
some client caps (corresponding inodes were not in cache when mds was
in reconnect state) to get dropped.

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
@batrick
Copy link
Member

batrick commented May 4, 2018

Note to self: cherry-pick to mimic.

@batrick
Copy link
Member

batrick commented May 7, 2018

Probably causes: http://tracker.ceph.com/issues/21777#note-7

@ukernel
Copy link
Contributor Author

ukernel commented May 9, 2018

this patch reveals the bug,

@batrick batrick merged commit 48f60e7 into ceph:master May 11, 2018
batrick added a commit that referenced this pull request May 11, 2018
* refs/pull/21769/head:
	mds: properly reconnect client caps after loading inodes

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
@ukernel ukernel deleted the wip-23518-fix branch May 14, 2018 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants