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: [ FAILED ] TestLibRBD.ExclusiveLockTransition #5241

Merged
1 commit merged into from Jul 28, 2015

Conversation

Projects
None yet
4 participants
@theanalyst
Member

theanalyst commented Jul 14, 2015

librbd: prevent object map updates from being interrupted
Object map updates were being canceled in-flight when the exclusive lock
is released.  This resulted in an ERESTART error code bubbling up to
AioRequest.

Fixes: 12165
Backport: hammer
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit 590cdc9)

Conflicts:
	src/librbd/ObjectMap.h
conflict due to a variable `m_snap_id' which was introduced in PR #4140
which is dropped as we are not backporting that feature

@theanalyst theanalyst self-assigned this Jul 14, 2015

@theanalyst theanalyst added this to the hammer milestone Jul 14, 2015

ghost pushed a commit that referenced this pull request Jul 15, 2015

Merge pull request #5241: [ FAILED ] TestLibRBD.ExclusiveLockTransition
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 15, 2015

Merge pull request #5241: [ FAILED ] TestLibRBD.ExclusiveLockTransition
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 16, 2015

Merge pull request #5241: [ FAILED ] TestLibRBD.ExclusiveLockTransition
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 19, 2015

Merge pull request #5241: [ FAILED ] TestLibRBD.ExclusiveLockTransition
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 19, 2015

Merge pull request #5241: [ FAILED ] TestLibRBD.ExclusiveLockTransition
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 19, 2015

Merge pull request #5241: [ FAILED ] TestLibRBD.ExclusiveLockTransition
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 19, 2015

Merge pull request #5241: [ FAILED ] TestLibRBD.ExclusiveLockTransition
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Jul 19, 2015

Merge pull request #5241: [ FAILED ] TestLibRBD.ExclusiveLockTransition
Reviewed-by: Loic Dachary <ldachary@redhat.com>

@theanalyst theanalyst added rbd and removed core labels Jul 28, 2015

@ghost

This comment has been minimized.

ghost commented Jul 28, 2015

@jdurgin does this hammer backport looks good to merge ? It went through a rbd suite ( http://tracker.ceph.com/issues/11990#rbd ).

@jdurgin

This comment has been minimized.

Member

jdurgin commented Jul 28, 2015

👍

ghost pushed a commit that referenced this pull request Jul 28, 2015

Loic Dachary
Merge pull request #5241 from theanalyst/wip-12238-hammer
[  FAILED  ] TestLibRBD.ExclusiveLockTransition

Reviewed-by: Josh Durgin <jdurgin@redhat.com>

@ghost ghost merged commit 7fd31b1 into ceph:hammer Jul 28, 2015

@ghost ghost changed the title from [ FAILED ] TestLibRBD.ExclusiveLockTransition to librbd: [ FAILED ] TestLibRBD.ExclusiveLockTransition Aug 4, 2015

This issue was closed.

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