----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47014/ -----------------------------------------------------------
(Updated May 5, 2016, 4:44 p.m.) Review request for Ambari, Daniel Gergely, Laszlo Puskas, Sandor Magyari, Srimanth Gunturi, and Sid Wagle. Changes ------- Separated the assert into 3 asserts in the unit test. 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 (updated) ----- 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 ------- Manual testing done. Unit tests are in progress. Thanks, Sebastian Toader