[AMBARI-16378] stackadvisor uses getHostsForSlaveComponent with wrong parameter name

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: trunk
• Fix Version/s: 2.4.0
• Component/s: ambari-server
• Assignee: Lav Jain
• Reporter: Lav Jain
• Created: 09/May/16 18:42
• Updated: 14/Jul/16 17:49
• Resolved: 10/May/16 17:59

Description

  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

The correct parameter name is comonentName (instead of component)

Issue Links

• is duplicated by: AMBARI-16434 NFSGateway and Phoenix are getting selected by default
• links to: Review Board