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

tasks/cephfs: reproducer for #16022 #1018

Merged
merged 1 commit into from Jun 10, 2016

Conversation

Projects
None yet
2 participants
@jcsp
Copy link
Contributor

commented May 26, 2016

Even though check_subs was broken, the multiclient
test was still passing because it didn't test that
clients survived a failover in the multi-filesystem
case.

Fixes: http://tracker.ceph.com/issues/16022
Signed-off-by: John Spray john.spray@redhat.com

John Spray
tasks/cephfs: reproducer for #16022
Even though check_subs was broken, the multiclient
test was still passing because it didn't test that
clients survived a failover in the multi-filesystem
case.

Fixes: http://tracker.ceph.com/issues/16022
Signed-off-by: John Spray <john.spray@redhat.com>

@jcsp jcsp added the cephfs label May 26, 2016

@jcsp

This comment has been minimized.

Copy link
Contributor Author

commented May 26, 2016

I've tested locally that this fails without "MDSMonitor.cc: fix mdsmap. subscriptions" and passes with it.

@jcsp

This comment has been minimized.

@jcsp jcsp assigned jcsp and ukernel and unassigned jcsp May 27, 2016

@jcsp

This comment has been minimized.

Copy link
Contributor Author

commented May 27, 2016

@ukernel could you review please?

@jcsp

This comment has been minimized.

Copy link
Contributor Author

commented Jun 3, 2016

@ukernel ping, please could you review this?

@ukernel ukernel merged commit 1b76c6b into master Jun 10, 2016

1 check passed

default Build finished.
Details

@ukernel ukernel deleted the wip-16022 branch Jun 10, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.