Merge PR #33014 into master

* refs/pull/33014/head:
	mgr/dashboard: pass a list of drive_group to create_osds

Reviewed-by: Sebastian Wagner <swagner@suse.com>
This commit is contained in:
Sage Weil 2020-01-31 23:28:44 -06:00
commit 4813b4132b
2 changed files with 3 additions and 3 deletions

View File

@ -95,7 +95,7 @@ class OsdManager(ResourceManager):
@wait_api_result
def create(self, drive_group):
return self.api.create_osds(drive_group)
return self.api.create_osds([drive_group])
class OrchClient(object):

View File

@ -872,7 +872,7 @@ class Orchestrator(object):
#assert not (service_name and service_id)
raise NotImplementedError()
def create_osds(self, drive_group):
def create_osds(self, drive_groups):
# type: (DriveGroupSpec) -> Completion
"""
Create one or more OSDs within a single Drive Group.
@ -882,7 +882,7 @@ class Orchestrator(object):
finer-grained OSD feature enablement (choice of backing store,
compression/encryption, etc).
:param drive_group: DriveGroupSpec
:param drive_groups: a list of DriveGroupSpec
:param all_hosts: TODO, this is required because the orchestrator methods are not composable
Probably this parameter can be easily removed because each orchestrator can use
the "get_inventory" method and the "drive_group.host_pattern" attribute