]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: add parsing for config on osd specs 47164/head
authorLuis Domingues <domingues.luis@protonmail.ch>
Tue, 19 Jul 2022 09:04:34 +0000 (11:04 +0200)
committerLuis Domingues <domingues.luis@protonmail.ch>
Tue, 19 Jul 2022 13:57:27 +0000 (15:57 +0200)
Cephadm, while parsing spec files, can parse ceph configuration
for almost all the services, except for OSDs, where it fails
with a nasty "unexpected keyword argument config".

This commit fixes this issue.

Signed-off-by: Luis Domingues <domingues.luis@protonmail.ch>
src/python-common/ceph/deployment/drive_group.py

index 6f96b2f6de8725438f45e9271b3993e2d144dda1..b451d664ec4b565696b9bc28b8794ac0df1cd05f 100644 (file)
@@ -150,7 +150,7 @@ class DriveGroupSpec(ServiceSpec):
         "data_devices", "db_devices", "wal_devices", "journal_devices",
         "data_directories", "osds_per_device", "objectstore", "osd_id_claims",
         "journal_size", "unmanaged", "filter_logic", "preview_only", "extra_container_args",
-        "data_allocate_fraction", "method", "crush_device_class",
+        "data_allocate_fraction", "method", "crush_device_class", "config",
     ]
 
     def __init__(self,
@@ -178,10 +178,12 @@ class DriveGroupSpec(ServiceSpec):
                  data_allocate_fraction=None,  # type: Optional[float]
                  method=None,  # type: Optional[OSDMethod]
                  crush_device_class=None,  # type: Optional[str]
+                 config=None,  # type: Optional[Dict[str, str]]
                  ):
         assert service_type is None or service_type == 'osd'
         super(DriveGroupSpec, self).__init__('osd', service_id=service_id,
                                              placement=placement,
+                                             config=config,
                                              unmanaged=unmanaged,
                                              preview_only=preview_only,
                                              extra_container_args=extra_container_args)