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

common: LibCephFS.GetPoolId failure #5887

Merged
merged 1 commit into from Oct 6, 2015

Conversation

Projects
None yet
4 participants
@Abhishekvrshny
Copy link
Contributor

Abhishekvrshny commented Sep 11, 2015

crypto: fix unbalanced ceph::crypto::init/ceph::crypto:shutdown
we may create a CephContext without calling common_init_finish(), then
delete the CephContext. In this case, ceph::crypto:init() is not called,
so CephContext::~CephContext() should not call ceph::crypto::shutdown().

Fixes: #12598
Signed-off-by: Yan, Zheng <zyan@redhat.com>
(cherry picked from commit 98a85ec)

@Abhishekvrshny Abhishekvrshny added this to the hammer milestone Sep 12, 2015

@Abhishekvrshny Abhishekvrshny assigned ghost Sep 12, 2015

@Abhishekvrshny Abhishekvrshny added common and removed core labels Sep 12, 2015

@ghost

This comment has been minimized.

Copy link

ghost commented Oct 2, 2015

in gitbuilders as wip-pr-5887

ghost pushed a commit that referenced this pull request Oct 2, 2015

Merge pull request #5887: LibCephFS.GetPoolId failure
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Oct 3, 2015

Merge pull request #5887: LibCephFS.GetPoolId failure
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Oct 3, 2015

Merge pull request #5887: LibCephFS.GetPoolId failure
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Oct 3, 2015

Merge pull request #5887: LibCephFS.GetPoolId failure
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost

This comment has been minimized.

Copy link

ghost commented Oct 4, 2015

@liewegas it passes the rados suite ( http://pulpito.ceph.com/loic-2015-10-03_11:11:28-rados-wip-pr-5887---basic-multi and http://pulpito.ceph.com/loic-2015-10-04_15:14:57-rados-wip-pr-5887---basic-multi are a clean run, except for one failure that was fixed by #6172) and it also passed the two CephFS jobs that consistently fail if it is not applied (http://pulpito.ceph.com/loic-2015-10-03_10:10:17-fs-wip-pr-5887---basic-multi/). Do you think it is good to merge ? This is needed for v0.94.4 and I'll update the draft release notes accordingly.

liewegas added a commit that referenced this pull request Oct 6, 2015

Merge pull request #5887 from Abhishekvrshny/wip-13044-hammer
LibCephFS.GetPoolId failure

Reviewed-by: Sage Weil <sage@redhat.com>

@liewegas liewegas merged commit 0650862 into ceph:hammer Oct 6, 2015

@ghost ghost changed the title LibCephFS.GetPoolId failure fs: LibCephFS.GetPoolId failure Oct 7, 2015

@ghost ghost changed the title fs: LibCephFS.GetPoolId failure common: LibCephFS.GetPoolId failure Oct 7, 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.