]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/orchestrator: remove `(add|test|remove)_stateful_service_rule` 27043/head
authorSebastian Wagner <sebastian.wagner@suse.com>
Tue, 5 Mar 2019 16:39:57 +0000 (17:39 +0100)
committerSebastian Wagner <sebastian.wagner@suse.com>
Tue, 19 Mar 2019 09:28:56 +0000 (10:28 +0100)
Reasons are:

1. No one implemented this till now
2. The distinction between DriveGroups end `tateful_service_rule` is not clear

Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
(cherry picked from commit e267c9000ce248be206264a0d8acb4ab8fa3296f)

src/pybind/mgr/orchestrator.py

index 36cf2cca56ceae630fc94ace6acf272ec1c171ff..3adaaedb583ce30637b778fb896ba5c548dd15fd 100644 (file)
@@ -374,37 +374,6 @@ class Orchestrator(object):
         """
         raise NotImplementedError()
 
-    def add_stateful_service_rule(self, service_type, stateful_service_spec,
-                                  placement_spec):
-        """
-        Stateful service rules serve two purposes:
-         - Optionally delegate device selection to the orchestrator
-         - Enable the orchestrator to auto-assimilate new hardware if it
-           matches the placement spec, without any further calls from ceph-mgr.
-
-        To create a confidence-inspiring UI workflow, use test_stateful_service_rule
-        beforehand to show the user where stateful services will be placed
-        if they proceed.
-        """
-        raise NotImplementedError()
-
-    def test_stateful_service_rule(self, service_type, stateful_service_spec,
-                                   placement_spec):
-        """
-        See add_stateful_service_rule.
-        """
-        raise NotImplementedError()
-
-    def remove_stateful_service_rule(self, service_type, id_):
-        """
-        This will remove the *rule* but not the services that were
-        created as a result.  Those should be converted into statically
-        placed services as if they had been created with add_stateful_service,
-        so that they can be removed with remove_stateless_service
-        if desired.
-        """
-        raise NotImplementedError()
-
 
 class UpgradeSpec(object):
     # Request to orchestrator to initiate an upgrade to a particular