openstack/deb-sahara
Code Issues Proposed changes

Fix missed service message in MapR plugin

Browse Source 
Now, in the case of presence of required
service with wrong version, dependency version
is added to the message.

Change-Id: Id77c4da715de4d0090228ad62ca1dfde7f7d45fa
Closes-Bug: #1497956
This commit is contained in:
Artem Osadchyi 2015-09-21 14:02:28 +03:00
parent 57523a684f
commit 9e97bfbd77
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
sahara/plugins/mapr/util/validation_utils.py
View File

@ -164,11 +164,15 @@ def on_same_node(component, dependency):
return ft.partial(validate, component=component, dependency=dependency)
def depends_on(service, required_by=None):
def depends_on(service, required_by):
def validate(cluster_context, service, required_by):
if not cluster_context.is_present(service):
service_name = service.ui_name
if service.version:
service_name += " %s" % service.version
raise e.RequiredServiceMissingException(
service.ui_name, required_by.ui_name)
service_name, required_by.ui_name)
return ft.partial(validate, service=service, required_by=required_by)