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
osd: misc FileStore fixes #7059
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
xiexingguo
force-pushed
the
xxg-wip-14187
branch
2 times, most recently
from
January 1, 2016 01:42
5ecef24
to
fa51a93
Compare
please rebase? |
xiexingguo
force-pushed
the
xxg-wip-14187
branch
from
January 4, 2016 02:59
fa51a93
to
295029d
Compare
Shall return a negative error code instead. Fixes: ceph#14187 Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Shall return a negative return code instead, otherwise caller will unable to handle it. Fixes: ceph#14194 Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Fixes: ceph#14188 Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
xiexingguo
force-pushed
the
xxg-wip-14187
branch
3 times, most recently
from
January 4, 2016 05:37
989bf27
to
fa2bbfa
Compare
Closed
Fixes: ceph#14186 Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@liewegas Rebased and some more small fixes, please review. |
lgtm! |
xiexingguo
force-pushed
the
xxg-wip-14187
branch
2 times, most recently
from
January 5, 2016 02:48
a045abf
to
2334e5d
Compare
Although currently lfn_close() is a noop, but these cases will be problematic if someday we want to do some real tidy up work. Fixes: ceph#14192 Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
liewegas
added a commit
that referenced
this pull request
Jan 11, 2016
osd: misc FileStore fixes Reviewed-by: Sage Weil <sage@redhat.com>
ghost
changed the title
os: fix return code if unable to sync
osd: misc FileStore fixes
Feb 10, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Shall return a negative error code instead.
Fixes: #14187
Signed-off-by: xie xingguo xie.xingguo@zte.com.cn