-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47014/
-----------------------------------------------------------
(Updated May 5, 2016, 5:55 p.m.)
Review request for Ambari, Daniel Gergely, Laszlo Puskas, Sandor Magyari,
Srimanth Gunturi, and Sid Wagle.
Changes
-------
Added unit test results.
Bugs: AMBARI-16270
https://issues.apache.org/jira/browse/AMBARI-16270
Repository: ambari
Description
-------
When the first host for a host group registers with the cluster and is assigned
to host group Ambari checks if there is any config group to be created for that
host group. If it has to than create a config group and assigning to it all
hosts that are expected for the parent host group. However it not checks is all
of the expected hosts of the host group have already been associated with the
cluster thus the config group creation fails as config groups can have only
hosts added that already are associated with the cluster.
This has been changed to add only those hosts to config group that already have
been associated with the cluster and update the created config group with the
remaing hosts as these become available and register with the cluster.
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/topology/AmbariContext.java
cf1a6ac
ambari-server/src/test/java/org/apache/ambari/server/topology/AmbariContextTest.java
1613d11
Diff: https://reviews.apache.org/r/47014/diff/
Testing (updated)
-------
Manual testing done.
Unit test results:
Results :
Tests run: 3568, Failures: 0, Errors: 0, Skipped: 36
Thanks,
Sebastian Toader
