]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: use dict instead of defaultdict to silence mypy warning
authorKefu Chai <kchai@redhat.com>
Sun, 17 Jan 2021 08:36:35 +0000 (16:36 +0800)
committerKefu Chai <kchai@redhat.com>
Tue, 26 Jan 2021 10:02:24 +0000 (18:02 +0800)
otherwise we have:

1: cephadm/module.py:30: note: In module imported here:
1: cephadm/serve.py: note: In member "_check_for_strays" of class "CephadmServe":
1: cephadm/serve.py:395: error: Argument "default" to "get_metadata" of "MgrModule" has incompatible type "defaultdict[str, None]"; expected "Optional[Dict[str, str]]"

Signed-off-by: Kefu Chai <kchai@redhat.com>
src/pybind/mgr/cephadm/serve.py

index 3e4a62013d47abf26be20cde76fe030ca1aa330d..8a4e4601e74454323f3df63c3581bde5f34c31ff 100644 (file)
@@ -406,12 +406,11 @@ class CephadmServe:
                 for s in daemons:
                     name = '%s.%s' % (s.get('type'), s.get('id'))
                     if s.get('type') == 'rbd-mirror':
-                        defaults = defaultdict(lambda: None, {'id': None})
                         metadata = self.mgr.get_metadata(
-                            "rbd-mirror", s.get('id'), default=defaults)
-                        if metadata['id']:
+                            "rbd-mirror", s.get('id'))
+                        try:
                             name = '%s.%s' % (s.get('type'), metadata['id'])
-                        else:
+                        except (KeyError, TypeError):
                             self.log.debug(
                                 "Failed to find daemon id for rbd-mirror service %s" % (s.get('id')))