]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
Merge pull request #36890 from sebastian-philipp/cephadm-extend-ceph.conf
authorJoshua Schmid <jschmid@suse.de>
Fri, 18 Sep 2020 08:56:24 +0000 (10:56 +0200)
committerGitHub <noreply@github.com>
Fri, 18 Sep 2020 08:56:24 +0000 (10:56 +0200)
mgr/cephadm: Add extra-ceph-conf

1  2 
src/pybind/mgr/cephadm/inventory.py
src/pybind/mgr/cephadm/module.py
src/pybind/mgr/cephadm/schedule.py
src/pybind/mgr/cephadm/services/cephadmservice.py
src/pybind/mgr/cephadm/services/iscsi.py
src/pybind/mgr/cephadm/services/osd.py
src/pybind/mgr/cephadm/tests/test_cephadm.py
src/pybind/mgr/cephadm/tests/test_migration.py

Simple merge
Simple merge
Simple merge
index 70d4be20f55dc023b2bcbdd781b875df2596e1db,9cfbda3e25a4dc875cd002bf306ce494a69b7c76..8acc4f7fa38cb5da37c9c2a6f584b09e9fcbd6eb
@@@ -353,10 -333,10 +361,10 @@@ class MonService(CephService)
                      'public_network is set but does not look like a CIDR network: \'%s\'' % network)
              extra_config += 'public network = %s\n' % network
  
-         daemon_spec.extra_config = {'config': extra_config}
+         daemon_spec.ceph_conf = extra_config
          daemon_spec.keyring = keyring
  
 -        return self.mgr._create_daemon(daemon_spec)
 +        return daemon_spec
  
      def _check_safe_to_destroy(self, mon_id: str) -> None:
          ret, out, err = self.mgr.check_mon_command({
index 1ce858ee5358483c5c0b2c6552e8ef0b15f29571,2db98458c1c3cf77e17f8dccde6ea5f3124be73a..57f3e632a439860b227c03090e286211e7db9083
@@@ -68,9 -68,9 +68,9 @@@ class IscsiService(CephService)
          igw_conf = self.mgr.template.render('services/iscsi/iscsi-gateway.cfg.j2', context)
  
          daemon_spec.keyring = keyring
-         daemon_spec.extra_config = {'iscsi-gateway.cfg': igw_conf}
+         daemon_spec.extra_files = {'iscsi-gateway.cfg': igw_conf}
  
 -        return self.mgr._create_daemon(daemon_spec)
 +        return daemon_spec
  
      def config_dashboard(self, daemon_descrs: List[DaemonDescription]):
          def get_set_cmd_dicts(out: str) -> List[dict]:
index 87ea6a8482a3e719f8dda86715d86d89129d3488,0d94696ae57659aff776f2ada6ace725888a7517..432384319656425563a8b4eaf91af173e75f5415
@@@ -637,11 -624,8 +631,11 @@@ class OSD
              return None
          for date_field in ['drain_started_at', 'drain_stopped_at', 'drain_done_at', 'process_started_at']:
              if inp.get(date_field):
-                 inp.update({date_field: datetime.strptime(inp.get(date_field, ''), DATEFMT)})
+                 inp.update({date_field: str_to_datetime(inp.get(date_field, ''))})
          inp.update({'remove_util': ctx})
 +        if 'nodename' in inp:
 +            hostname = inp.pop('nodename')
 +            inp['hostname'] = hostname
          return cls(**inp)
  
      def __hash__(self):
index 08e0abd5a61e78157d61861d16e9238bfd677b91,13647c7ceeec807dbdc6145e6bc77fa85e5a097d..ac2011f1b3a664e5b433a423f3159cedc46f277b
@@@ -1,13 -1,11 +1,14 @@@
  import json
  from datetime import datetime
  
 +import pytest
 +
  from ceph.deployment.service_spec import PlacementSpec, ServiceSpec, HostPlacementSpec
  from cephadm import CephadmOrchestrator
- from cephadm.inventory import SPEC_STORE_PREFIX, DATEFMT
+ from cephadm.inventory import SPEC_STORE_PREFIX
+ from cephadm.utils import DATEFMT
  from cephadm.tests.fixtures import _run_cephadm, cephadm_module, wait, with_host
 +from orchestrator import OrchestratorError
  from tests import mock