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

librbd: remove duplicate read_only test in librbd::async_flatten #5856

Merged
merged 1 commit into from Nov 5, 2015

Conversation

Projects
None yet
5 participants
@runsisi
Copy link
Contributor

runsisi commented Sep 9, 2015

ictx->read_only has been tested at line 2838, so the same test at line 2847 can be removed.

Signed-off-by: runsisi runsisi@hust.edu.cn

librbd: remove duplicate read_only test in librbd::async_flatten
Signed-off-by: runsisi <runsisi@hust.edu.cn>
@dillaman

This comment has been minimized.

Copy link
Contributor

dillaman commented Sep 9, 2015

LGTM

@jdurgin

This comment has been minimized.

Copy link
Member

jdurgin commented Sep 10, 2015

lgtm, will merge after infernalis feature freeze

dillaman added a commit that referenced this pull request Nov 5, 2015

Merge pull request #5856 from runsisi/wip-misc-remove-dup-test
librbd: remove duplicate read_only test in librbd::async_flatten

Reviewed-by: Jason Dillaman <dillaman@redhat.com>

@dillaman dillaman merged commit c17a75a into ceph:master Nov 5, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.