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/BaseMgrStandbyModule: parse prefix properly #26920
Conversation
Fixes: http://tracker.ceph.com/issues/38705 Signed-off-by: Sage Weil <sage@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -65,7 +65,7 @@ ceph_get_module_option(BaseMgrStandbyModule *self, PyObject *args) | |||
{ | |||
char *what = nullptr; | |||
char *prefix = nullptr; | |||
if (!PyArg_ParseTuple(args, "ss:ceph_get_module_option", &what)) { | |||
if (!PyArg_ParseTuple(args, "ss:ceph_get_module_option", &what, &prefix)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh man, I complained about the Active-Standby mismatching interfaces, and overlooked the big thing here 🤦♂️
Nevertheless, I've filed a separate issue for refactoring this, as the amount of duplicate code in this area is high: http://tracker.ceph.com/issues/38717
When mgr/selftest/testkey = foo and mgr/selftest/x/testkey is not set, then get_localized() should return foo. Signed-off-by: Sage Weil <sage@redhat.com>
* refs/pull/26920/head: qa/tasks/mgr/test_module_selftest: fix localized value test mgr/BaseMgrStandbyModule: parse prefix properly Reviewed-by: Volker Theile <vtheile@suse.com> Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Fixes: http://tracker.ceph.com/issues/38705
Signed-off-by: Sage Weil sage@redhat.com