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: check for luminous MDS before sending FLUSH_MDLOG #15805

Merged
merged 1 commit into from Jun 22, 2017

Conversation

Projects
None yet
3 participants
@jcsp
Contributor

jcsp commented Jun 21, 2017

This is needed to make it safe to run newer clients
on pre-luminous MDS.

Signed-off-by: John Spray john.spray@redhat.com

John Spray
client: check for luminous MDS before sending FLUSH_MDLOG
This is needed to make it safe to run newer clients
on pre-luminous MDS.

Signed-off-by: John Spray <john.spray@redhat.com>

@jcsp jcsp requested a review from liewegas Jun 21, 2017

@jcsp

This comment has been minimized.

Contributor

jcsp commented Jun 21, 2017

@liewegas this is the one we need to merge before doing an RC

@theanalyst

This comment has been minimized.

Member

theanalyst commented Jun 22, 2017

@jcsp @liewegas is this good to go or does it need a qe or so?

@jcsp jcsp merged commit 53fc7a5 into ceph:master Jun 22, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
arm64 make check arm64 make check succeeded
Details
make check make check succeeded
Details

@jcsp jcsp deleted the jcsp:wip-mds-flush-upgrade branch Jun 22, 2017

@jcsp

This comment has been minimized.

Contributor

jcsp commented Jun 22, 2017

This passed an fs suite

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