Skip to content
This repository was archived by the owner on Feb 8, 2024. It is now read-only.

Commit f1d4643

Browse files
committed
CORTX-33789: Hare Config stage fail with Motr Clients
Problem: If num_client_inst is specified a non-zero value, deployment is failing. Issue is with search key. For data nodes we used 'io':'motr', it returns the list of machine IDs which includes both data node and motr client nodes, because of which deployment was failing. Solution: Separated search key for data node and motr client node. Signed-off-by: pavankrishnat <[email protected]>
1 parent 150112e commit f1d4643

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

provisioning/miniprov/hare_mp/store.py

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,15 @@ def get_machine_ids_for_service(self, service_type: str) -> List[str]:
9292
service_type will be like Const.SERVICE_MOTR_IO.value,
9393
Const.SERVICE_S3_SERVER.value etc
9494
"""
95+
if service_type == 'io':
96+
return self.get_machine_ids_for_attribute('num_cvg')
97+
"""
98+
presently, For service_type: motr_client, both client and data pods
99+
are returned by the conf store search query. But data pods are
100+
filtered out while creating node in _create_node().
101+
"""
95102
types = {
96-
'io': 'motr',
103+
'motr_client': 'motr',
97104
'rgw_s3': 'rgw',
98105
'agent': 'csm'
99106
}
@@ -121,7 +128,7 @@ def get_machine_ids_for_component(self, comp_type: str) -> List[str]:
121128
return self.get_machine_ids_for_attribute('name', comp_type)
122129

123130
def get_machine_ids_for_attribute(self, attr_type: str,
124-
name: str) -> List[str]:
131+
name: str = None) -> List[str]:
125132
"""
126133
Return a list of pod's machine_id for the machines consisting
127134
the attribute name under the given attribute type.
@@ -173,7 +180,7 @@ def get_data_nodes(self) -> List[str]:
173180
return self.get_machine_ids_for_service(Const.SERVICE_MOTR_IO.value)
174181

175182
def search_val(self, parent_key: str, search_key: str,
176-
search_val: str) -> List[str]:
183+
search_val: str = None) -> List[str]:
177184
"""
178185
Searches a given key value under the given parent key.
179186
"""

0 commit comments

Comments
 (0)