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

hammer: fs: double decreased the count to trim caps which will cause failing to respond to cache pressure #8804

Merged
merged 1 commit into from Aug 3, 2016

Conversation

smithfarm
Copy link
Contributor

Signed-off-by: Zhi Zhang zhangz.david@outlook.com
(cherry picked from commit d8a3f6d)
@smithfarm smithfarm self-assigned this Apr 28, 2016
@smithfarm smithfarm added this to the hammer milestone Apr 28, 2016
@smithfarm smithfarm added bug-fix core cephfs Ceph File System and removed core labels Apr 28, 2016
smithfarm added a commit that referenced this pull request Jun 1, 2016
…caps which will cause failing to respond to cache pressure

Reviewed-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit that referenced this pull request Jun 27, 2016
…caps which will cause failing to respond to cache pressure

Reviewed-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit that referenced this pull request Jul 18, 2016
…caps which will cause failing to respond to cache pressure

Reviewed-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit that referenced this pull request Jul 24, 2016
…caps which will cause failing to respond to cache pressure

Reviewed-by: Nathan Cutler <ncutler@suse.com>
@smithfarm
Copy link
Contributor Author

@gregsfortytwo This PR passed a cephfs integration run with two failures. Both failures are reproducible in both hammer-backports and hammer itself. One failure is due to http://tracker.ceph.com/issues/11258 ("cephfs-java ftruncate unit test failure") which is fixed in master, but the hammer backport is non-trivial. The second failure is probably an issue with some of the tests. For details, see http://tracker.ceph.com/issues/15895#note-21

Do you think this PR can be merged?

@smithfarm
Copy link
Contributor Author

@jcsp OK to merge?

@jcsp jcsp merged commit 8f51f14 into ceph:hammer Aug 3, 2016
@smithfarm smithfarm deleted the wip-15512-hammer branch August 8, 2016 08:25
@smithfarm smithfarm changed the title hammer: Double decreased the count to trim caps which will cause failing to respond to cache pressure hammer: fs: double decreased the count to trim caps which will cause failing to respond to cache pressure Aug 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix cephfs Ceph File System
Projects
None yet
2 participants