ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Speidel" <jspei...@hortonworks.com>
Subject Review Request 33649: Blueprints containing HA components or implicit MYSQL_SERVER components fail to provision
Date Wed, 29 Apr 2015 00:09:27 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33649/
-----------------------------------------------------------

Review request for Ambari, Mahadev Konar and Robert Nettleton.


Bugs: AMBARI-10811
    https://issues.apache.org/jira/browse/AMBARI-10811


Repository: ambari


Description
-------

When provisioning a cluster using a blueprint that contains HA components or an existing MYSQL_SERVER
or implicit MYSQL_SERVER (not explicitly specified in blueprint but required by dependency)
the cluster fails to deploy.
The request remains in the PENDING state and at 0% complete.

Output like the following is logged to ambari-server.out:
TopologyManager.ConfigureClusterTask: Entering
Caught an error from Config Processor: java.lang.IllegalArgumentException: Unable to determine
required host groups for component. Component 'MYSQL_SERVER' is not mapped to any host group
or is mapped to multiple groups.

This is a regression as a result of https://issues.apache.org/jira/browse/AMBARI-10750
The problem was that the blueprint config processor was not dealing with these topologies
properly when trying to determine which host groups needed to be resolved prior to updating
configuration.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java
95e9807 
  ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterConfigurationRequest.java
1bffbf2 
  ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java 3e1b565

  ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
34b239b 

Diff: https://reviews.apache.org/r/33649/diff/


Testing
-------

Functional Testing:
- Tested that the provided topologies don't result in the above error

Unit Testing:
- added new test
- currently running entire unit test suite and will update review with results prior to merging


Thanks,

John Speidel


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message