-
Notifications
You must be signed in to change notification settings - Fork 6k
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: In plugins 'module' classes need not to be called "Module" anymore. #18526
Conversation
c3b023f
to
c4d900e
Compare
the failures are addressed by #18533 |
need to rebase after 213cbd6 is merged |
6df4742
to
2a063a8
Compare
@jcsp mind taking a look? |
Looks good, let's also update the docs in this PR, in plugins.rst where it says "create a class |
Let's run the rados/mgr suite on this before merging, as it has several new tests since 213cbd6 merged |
Fixes: http://tracker.ceph.com/issues/17454 Signed-off-by: Kefu Chai <kchai@redhat.com> Signed-off-by: bhavishyagopesh <bhavishyagopesh@gmail.com>
@jcsp the tests failed: http://pulpito.ceph.com/kchai-2017-11-02_08:11:58-rados-wip-17454-kefu-distro-basic-mira/ but the master branch also failed http://pulpito.ceph.com/kchai-2017-11-02_09:53:26-rados-master-distro-basic-mira/ strange enough, when i enable the "selftest" module in my testbed, the ceph-mgr does respawn itself and "active_gid" changes. |
hmm, passed for me |
Fixes: http://tracker.ceph.com/issues/17454
Signed-off-by: bhavishyagopesh bhavishyagopesh@gmail.com
Signed-off-by: Kefu Chai kchai@redhat.com