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

Directory fragmentation tests #1275

Merged
merged 4 commits into from Dec 7, 2016
Merged

Directory fragmentation tests #1275

merged 4 commits into from Dec 7, 2016

Conversation

jcsp
Copy link
Contributor

@jcsp jcsp commented Nov 17, 2016

This goes with ceph/ceph#12022

John Spray added 4 commits November 17, 2016 23:50
The success log message was in the wrong
place (and unreachable)

Signed-off-by: John Spray <john.spray@redhat.com>
This got lost in the multi-filesystem
refactor.  It's just a debug convenience.

Signed-off-by: John Spray <john.spray@redhat.com>
This makes it much easier to associate e.g. slow
requests messages with which test was running.

Signed-off-by: John Spray <john.spray@redhat.com>
Fixes: http://tracker.ceph.com/issues/17853
Signed-off-by: John Spray <john.spray@redhat.com>
@jcsp
Copy link
Contributor Author

jcsp commented Nov 17, 2016

@fullerdj this is the infamous if ino['ino'] == dir_inono: code! Naming suggestions welcome :-p

@fullerdj
Copy link
Contributor

well, we use inodeno elsewhere. I think the infamous name works fine too.

@batrick
Copy link
Member

batrick commented Nov 21, 2016

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

@jcsp jcsp merged commit a3c3e78 into ceph:master Dec 7, 2016
@jcsp jcsp deleted the wip-17853 branch December 7, 2016 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants