openstack/deb-sahara
Code Issues Proposed changes

Made processes names case sensitive

Browse Source 
Current approach "store name in any case, but search in lowercase" was
done to work with HDP uppercase processes. Since process manipulation
logic was moved to plugins, replaced approach to case sensitive use.

Closes-Bug: #1282806
Change-Id: I4a8d65b750135a8521f26c6e6c2d9311f3281b14
This commit is contained in:
Andrew Lazarev 2014-02-14 13:16:58 -08:00
parent 463b402ec3
commit 1b48f5b56a
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sahara/plugins/general/utils.py
View File

@ -22,7 +22,7 @@ from sahara.plugins.general import exceptions as ex
def get_node_groups(cluster, node_process=None):
return [ng for ng in cluster.node_groups
if (node_process is None or
node_process in [n.lower() for n in ng.node_processes])]
node_process in ng.node_processes)]
def get_instances_count(cluster, node_process=None):

4
sahara/plugins/hdp/ambariplugin.py
View File

@ -140,10 +140,10 @@ class AmbariPlugin(p.ProvisioningPluginBase):
"cluster_configs": cluster_configs})
def get_oozie_server(self, cluster):
return u.get_instance(cluster, "oozie_server")
return u.get_instance(cluster, "OOZIE_SERVER")
def validate_edp(self, cluster):
oo_count = u.get_instances_count(cluster, 'oozie_server')
oo_count = u.get_instances_count(cluster, 'OOZIE_SERVER')
if oo_count != 1:
raise ex.InvalidComponentCountException('oozie', '1', oo_count)