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

ceph-disk: missing return when a file is used for journal #11600

Closed
wants to merge 1 commit into from

Conversation

jayaajay
Copy link
Contributor

@jayaajay jayaajay commented Oct 21, 2016

http://tracker.ceph.com/issues/17662

ceps-disk is missing a return statement when preparing the journal space when it is regular file.

@cbodley
Copy link
Contributor

cbodley commented Oct 21, 2016

thank you! could you please add Fixes: http://tracker.ceph.com/issues/17662 to your commit message? we also need a Signed-off-by: line (you can git commit --amend -s)

…ssues/17662

Signed-off-by: Jayashree Candadai <jayaajay@indiana.edu>
@ghost
Copy link

ghost commented Oct 21, 2016

good catch !

@ghost
Copy link

ghost commented Oct 21, 2016

I'll add a test for that so it stays fixed.

@ghost ghost changed the title Fix the missing return in ceph-disk ceph-disk: missing return when a file is used for journal Oct 24, 2016
@ghost
Copy link

ghost commented Oct 24, 2016

@jayaajay I amended your commit to add tests and fix another issue, could you please review it ? #11619

@ghost ghost closed this Oct 24, 2016
@jayaajay
Copy link
Contributor Author

Done!

On Oct 21, 2016, at 2:06 PM, Casey Bodley <notifications@github.commailto:notifications@github.com> wrote:

thank you! could you please add Fixes: http://tracker.ceph.com/issues/17662 to your commit message? we also need a Signed-off-by: line (you can git commit --amend -s)


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com//pull/11600#issuecomment-255435182, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKMUaYJ37qtLQpRfZWUQ2iQBCfSOCw3Iks5q2P81gaJpZM4KddBg.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants