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
client: always refresh mds feature bits on session open #54030
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you rephrase commit title to:
client: always refresh mds feature bits on session open
done. |
in a cluster with multiple active MDSs, is it possible that different MDSs are on different versions? |
If you mean different ceph versions, then yes. |
and the client must be maintaining some table (or some sort of data structure) to keep records of the MDSs and their feature bits right? So that client doesn't get baffled with what and how to communicate with mds.a, mds.b and mds.c if all three are on different ceph versions with possibly different set of feature bits to offer. |
yes - those are maintained per (mds) session. |
Can we test this out apart from the upgrade tests? Maybe add a dummy feature bit to the MDS and expect client to not crash? |
This yaml
always reproduces this, so I think we can depend on that. |
@batrick @dparmar18 fixed and updated. please check. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
jenkins test make check |
i hope this includes #53734 which found out this issue |
Fixes: http://tracker.ceph.com/issues/63188 Signed-off-by: Venky Shankar <vshankar@redhat.com>
rebase and updated for jenkins tests... |
jenkins test api |
Fixes: http://tracker.ceph.com/issues/63188
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. "pacific"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test dashboard cephadm
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox
jenkins test windows