----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36738/#review93282 -----------------------------------------------------------
Ship it! Ship It! - Andrew Onischuk On July 28, 2015, 12:25 p.m., Dmitro Lisnichenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36738/ > ----------------------------------------------------------- > > (Updated July 28, 2015, 12:25 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jonathan > Hurley, and Nate Cole. > > > Bugs: AMBARI-12517 > https://issues.apache.org/jira/browse/AMBARI-12517 > > > Repository: ambari > > > Description > ------- > > We may skip sending install_packages action to hosts without versionable > components, and instantly transition these hosts to INSTALLED state. That > should allow to simplify agent-side logic for working with repo versions. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java > f3714b2 > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartBeatHandler.java > 6f34b62 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java > 972226d > > ambari-server/src/main/java/org/apache/ambari/server/events/ActionFinalReportReceivedEvent.java > 3ff5031 > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/DistributeRepositoriesActionListener.java > 5b7c2d6 > ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java > fe669bd > ambari-server/src/main/java/org/apache/ambari/server/state/Host.java > 34fb995 > > ambari-server/src/main/java/org/apache/ambari/server/state/host/HostImpl.java > e20cd25 > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > bfcef1a > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProviderTest.java > 1dfa2fb > ambari-server/src/test/python/custom_actions/TestInstallPackages.py 5ddfaa8 > > Diff: https://reviews.apache.org/r/36738/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Ambari Views ...................................... SUCCESS [3.002s] > [INFO] Ambari Metrics Common ............................. SUCCESS [1.475s] > [INFO] Ambari Server ..................................... SUCCESS > [48:41.983s] > [INFO] Ambari Agent ...................................... SUCCESS [7.346s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 48:54.423s > [INFO] Finished at: Tue Jul 28 13:13:53 EEST 2015 > [INFO] Final Memory: 61M/779M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Dmitro Lisnichenko > >
