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

mgr: shutdown py_modules in Mgr::shutdown() #14078

Merged
merged 1 commit into from Mar 29, 2017

Conversation

Projects
None yet
2 participants
@tchaikov
Contributor

tchaikov commented Mar 22, 2017

Fixes: http://tracker.ceph.com/issues/19258
Signed-off-by: Kefu Chai kchai@redhat.com

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Mar 22, 2017

@tchaikov tchaikov changed the title from mgr: shutdown py_modules in Mgr::shutdown() to [DNM] mgr: shutdown py_modules in Mgr::shutdown() Mar 22, 2017

@tchaikov tchaikov changed the title from [DNM] mgr: shutdown py_modules in Mgr::shutdown() to mgr: shutdown py_modules in Mgr::shutdown() Mar 29, 2017

mgr: shutdown py_modules in Mgr::shutdown()
Fixes: http://tracker.ceph.com/issues/19258
Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov

This comment has been minimized.

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Mar 29, 2017

the failed one is the rebuild-monstore test. i think the reason of the failure is that the rebuild test didn't take the ceph-mgr into consideration, and hence failed to add its keyring and caps to the monstore when rebuilding the monstore.

@tchaikov tchaikov requested review from liewegas and jcsp Mar 29, 2017

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Mar 29, 2017

retest this please (jenkins agent issue)

@tchaikov tchaikov merged commit 76d467c into ceph:master Mar 29, 2017

2 of 3 checks passed

default Build started sha1 is merged.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details

@tchaikov tchaikov deleted the tchaikov:wip-19258 branch Mar 29, 2017

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Mar 29, 2017

@jcsp blessed this PR over irc.

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