[ 
https://issues.apache.org/jira/browse/AMBARI-16378?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15278972#comment-15278972
 ] 

Hudson commented on AMBARI-16378:
---------------------------------

FAILURE: Integrated in Ambari-trunk-Commit #4817 (See 
[https://builds.apache.org/job/Ambari-trunk-Commit/4817/])
AMBARI-16378. stackadvisor uses getHostsForSlaveComponent with wrong (smohanty: 
[http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=07dbee41a2be7cc65daa7efd6e89a2ec31df7c55])
* ambari-server/src/main/resources/stacks/stack_advisor.py


> stackadvisor uses getHostsForSlaveComponent with wrong parameter name
> ---------------------------------------------------------------------
>
>                 Key: AMBARI-16378
>                 URL: https://issues.apache.org/jira/browse/AMBARI-16378
>             Project: Ambari
>          Issue Type: Bug
>          Components: ambari-server
>    Affects Versions: trunk
>            Reporter: Lav Jain
>            Assignee: Lav Jain
>             Fix For: trunk
>
>         Attachments: AMBARI-16378.patch
>
>
> {code}
>   def getHostsForSlaveComponent(self, services, hosts, component, hostsList, 
> hostsComponentsMap, freeHosts):
>     componentName = component["StackServiceComponents"]["component_name"]
>     if component["StackServiceComponents"]["cardinality"] == "ALL":
>       return hostsList
>     componentIsPopulated = self.isComponentHostsPopulated(component)
>     if componentIsPopulated:
>       return component["StackServiceComponents"]["hostnames"]
>     hostsForComponent = []
>     if self.isSlaveComponent(component):
>       cardinality = str(component["StackServiceComponents"]["cardinality"])
>       if self.isComponentUsingCardinalityForLayout(component) and cardinality:
>         # cardinality types: 1+, 1-2, 1
> {code}
> The correct parameter name is comonentName (instead of component)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to