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

librados: set the flag CEPH_OSD_FLAG_FULL_TRY of Op in the right place. #14193

Merged
merged 1 commit into from Apr 1, 2017

Conversation

Projects
None yet
3 participants
@liupan1111
Contributor

liupan1111 commented Mar 28, 2017

Improve the incompletely bug fix for my code added in #12627.
In 12627, I set CEPH_OSD_FLAG_FULL_TRY flag in _prepare_osd_op, which was too late for pause check. But because of the bug ( now fixed in #13425), I thought OSD was full, but indeed it was not, therefore, my last change didn't work, which case I didn't realize that at that time.

librados: set the flag CEPH_OSD_FLAG_FULL_TRY of Op in the right place.
Signed-off-by: Pan Liu <liupan1111@gmail.com>
@liupan1111

This comment has been minimized.

Contributor

liupan1111 commented Mar 28, 2017

@dillaman , please help take a look, thanks.

@tchaikov

This comment has been minimized.

@tchaikov tchaikov merged commit e2b7cc0 into ceph:master Apr 1, 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

@liupan1111 liupan1111 deleted the liupan1111:wip-fix-delete-full branch Apr 2, 2017

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