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

qa: simplify TestJournalRepair #15096

Merged
merged 1 commit into from May 16, 2017

Conversation

Projects
None yet
2 participants
@jcsp
Contributor

jcsp commented May 15, 2017

This was sending lots of metadata ops to MDSs to persuade
them to migrate some subtrees, but that was flaky. Use
the shiny new rank pinning functionality instead.

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

John Spray
qa: simplify TestJournalRepair
This was sending lots of metadata ops to MDSs to persuade
them to migrate some subtrees, but that was flaky.  Use
the shiny new rank pinning functionality instead.

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

@jcsp jcsp requested a review from batrick May 16, 2017

@jcsp

This comment has been minimized.

Contributor

jcsp commented May 16, 2017

retest this please (jenkins)

@batrick

This comment has been minimized.

Member

batrick commented May 16, 2017

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>

@jcsp jcsp merged commit 60f9046 into ceph:master May 16, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@jcsp jcsp deleted the jcsp:wip-journalrepair-test branch May 16, 2017

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