----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62848/#review187444 -----------------------------------------------------------
Ship it! Ship It! - Nate Cole On Oct. 9, 2017, 3:06 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62848/ > ----------------------------------------------------------- > > (Updated Oct. 9, 2017, 3:06 p.m.) > > > Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, and Nate Cole. > > > Bugs: AMBARI-22166 > https://issues.apache.org/jira/browse/AMBARI-22166 > > > Repository: ambari > > > Description > ------- > > After deleting a service, you can't revert a PATCH or MAINT upgrade. This is > because we iterate over all prior services when creating the upgrade context > for the reversion. > > STR: > - Install ZK, HDFS, and Storm > - Upgrade ZK and Storm using a PATCH or MAINT VDF > - Remove Storm > - Revert the prior patch > > ``` > 09 Oct 2017 10:02:59,142 ERROR [ambari-client-thread-22258] > AbstractResourceProvider:285 - Caught AmbariException when creating a resource > org.apache.ambari.server.ServiceNotFoundException: Service not found, > clusterName=cl1, serviceName=ATLAS > at > org.apache.ambari.server.state.cluster.ClusterImpl.getService(ClusterImpl.java:863) > at > org.apache.ambari.server.state.UpgradeContext.<init>(UpgradeContext.java:403) > at > org.apache.ambari.server.state.UpgradeContext$$FastClassByGuice$$6e2e5fe5.newInstance(<generated>) > at > com.google.inject.internal.cglib.reflect.$FastConstructor.newInstance(FastConstructor.java:40) > at > com.google.inject.internal.DefaultConstructionProxyFactory$1.newInstance(DefaultConstructionProxyFactory.java:60) > at > com.google.inject.internal.ConstructorInjector.construct(ConstructorInjector.java:85) > at > com.google.inject.internal.ConstructorBindingImpl$Factory.get(ConstructorBindingImpl.java:254) > at > com.google.inject.internal.InjectorImpl$4$1.call(InjectorImpl.java:978) > at > com.google.inject.internal.InjectorImpl.callInContext(InjectorImpl.java:1024) > at com.google.inject.internal.InjectorImpl$4.get(InjectorImpl.java:974) > at > com.google.inject.assistedinject.FactoryProvider2.invoke(FactoryProvider2.java:632) > at com.sun.proxy.$Proxy31.create(Unknown Source) > ``` > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeContext.java > 9f31ab8a62 > > ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeContextTest.java > 5cf9a4c859 > > > Diff: https://reviews.apache.org/r/62848/diff/2/ > > > Testing > ------- > > Manual reversion of a PATCH after removing a service. > > [INFO] Results: > [INFO] > [WARNING] Tests run: 4915, Failures: 0, Errors: 0, Skipped: 34 > > > Thanks, > > Jonathan Hurley > >
