> On Oct. 14, 2015, 1:36 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java, > > line 1254 > > <https://reviews.apache.org/r/39319/diff/2/?file=1098377#file1098377line1254> > > > > Does this handle hosts that only have AMS?
Yes, exactly. - Nate ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39319/#review102662 ----------------------------------------------------------- On Oct. 14, 2015, 1:21 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39319/ > ----------------------------------------------------------- > > (Updated Oct. 14, 2015, 1:21 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. > > > Bugs: AMBARI-13423 > https://issues.apache.org/jira/browse/AMBARI-13423 > > > Repository: ambari > > > Description > ------- > > When distributing bits, and the repo is targeting components for patching, > then only send the command to those hosts. > > NOTES: > - This is on a feature branch, made sure tests were functional > - Added a new repo state, a separate effort is ongoing for the transition > logic > - The branch is NOT YET merged with the trunk changes for EU. Once that > settles, that will be brought in. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/services/PersistKeyValueService.java > 9b942e5 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java > 4868ca7 > > ambari-server/src/main/java/org/apache/ambari/server/events/ActionFinalReportReceivedEvent.java > de797f3 > > ambari-server/src/main/java/org/apache/ambari/server/state/RepositoryVersionState.java > b0f85b4 > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java > e3bb320 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProviderTest.java > 1819ef9 > > Diff: https://reviews.apache.org/r/39319/diff/ > > > Testing > ------- > > Manual testing, automated pending. > > > Thanks, > > Nate Cole > >
