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

kraken: tests: test_notify.py: assert(not image.is_exclusive_lock_owner()) on line 147 #14833

Merged
merged 1 commit into from Jul 5, 2017

Conversation

smithfarm
Copy link
Contributor

@smithfarm smithfarm self-assigned this Apr 27, 2017
@smithfarm smithfarm added this to the kraken milestone Apr 27, 2017
@smithfarm smithfarm changed the title kraken: [test] test_notify.py: assert(not image.is_exclusive_lock_owner()) on line 147 kraken: tests: test_notify.py: assert(not image.is_exclusive_lock_owner()) on line 147 Apr 27, 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>
(cherry picked from commit e009e1b)
@smithfarm
Copy link
Contributor Author

@dillaman This PR passed an rbd run at http://tracker.ceph.com/issues/19009#note-56 (there were some failures that required a fix - applied via #16113). Please review/approve the PR if you agree it can be merged.

Copy link

@dillaman dillaman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@smithfarm smithfarm merged commit e13f546 into ceph:kraken Jul 5, 2017
@smithfarm smithfarm deleted the wip-19794-kraken branch July 5, 2017 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants