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

test/librbd/test_notify.py: don't disable feature in slave #14751

Merged
merged 1 commit into from Apr 27, 2017

Conversation

Projects
None yet
3 participants
@trociny
Contributor

trociny commented Apr 24, 2017

On jewel it will have stolen the exclusive lock. Instead, ensure that
object map and fast diff are already disabled on the clone before the
start of the test.

Fixes: http://tracker.ceph.com/issues/19716
Signed-off-by: Mykola Golub mgolub@mirantis.com

Mykola Golub
test/librbd/test_notify.py: don't disable feature in slave
On jewel it will have stolen the exclusive lock. Instead, ensure that
object map and fast diff are already disabled on the clone before the
start of the test.

Fixes: http://tracker.ceph.com/issues/19716
Signed-off-by: Mykola Golub <mgolub@mirantis.com>

@trociny trociny added rbd tests labels Apr 24, 2017

@trociny trociny requested a review from dillaman Apr 24, 2017

@dillaman

lgtm

@dillaman

This comment has been minimized.

Contributor

dillaman commented Apr 25, 2017

retest this please

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Apr 27, 2017

Jenkins retest this please

@dillaman dillaman merged commit edc88b1 into ceph:master Apr 27, 2017

3 of 4 checks passed

default Build triggered. sha1 is merged.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
arm build successfully built on arm
Details

@trociny trociny deleted the trociny:wip-19716 branch Apr 27, 2017

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