----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54665/#review159015 -----------------------------------------------------------
Fix it, then Ship it! ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java (lines 237 - 239) <https://reviews.apache.org/r/54665/#comment229941> Agree with @Alejandro's comment. Once fixed, then I'm ok to Ship. - Nate Cole On Dec. 12, 2016, 11:31 a.m., Di Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54665/ > ----------------------------------------------------------- > > (Updated Dec. 12, 2016, 11:31 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Nate Cole, and Tim Thorpe. > > > Bugs: AMBARI-19156 > https://issues.apache.org/jira/browse/AMBARI-19156 > > > Repository: ambari > > > Description > ------- > > Install packages step (before RU/EU) fails with err "Cannot enumerate > services" if the current stack has services that have been removed in the new > stack. For example, mystack-1.0 has Spark, but mystakc-2.0 only has Spark2, > where Spark's metainfo.xml in mystack-2.0 has the <deleted>true</deleted> > section. > > Workaround is to remove Spark before attempting the install packages. > > The error is below: > 02 Dec 2016 12:53:43,928 ERROR [ambari-client-thread-27] > BaseManagementHandler:61 - Caught a system exception while attempting to > create a resource: Cannot enumerate services > org.apache.ambari.server.controller.spi.SystemException: Cannot enumerate > services > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java > 23a4a27 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java > d417ec2 > ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java > ce606de > ambari-server/src/main/java/org/apache/ambari/server/state/StackInfo.java > ba5cb42 > > ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java > 8263f72 > > ambari-server/src/test/java/org/apache/ambari/server/stack/StackModuleTest.java > 0b7d0ff > > Diff: https://reviews.apache.org/r/54665/diff/ > > > Testing > ------- > > 1. unit tests > 2. patched a cluster with the fixes, update new stack to remove Spark ( > leaving only Spark 2), run Install Packages, verify it didn't fail with the > "Cannot enumerate services" error. > > > Thanks, > > Di Li > >