-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60850/
-----------------------------------------------------------
(Updated July 13, 2017, 4:27 p.m.)
Review request for Ambari, Alejandro Fernandez, Nate Cole, and Sid Wagle.
Bugs: AMBARI-21436
https://issues.apache.org/jira/browse/AMBARI-21436
Repository: ambari
Description
-------
When upgrading from a stack which contains a component with doesn't exist in
the target stack, the upgrade will run into a problem while attempting to
restart that component.
There should be a pre-requisite check which guarantees that missing
components/services are removed before upgrading.
Also, we need to fix the following exception when distributing a repo without
the components:
{code}
Caused by: org.apache.ambari.server.StackAccessException: Stack data,
stackName=HDP, stackVersion=2.6, serviceName=HBASE,
componentName=HBASE_REST_SERVER
at
org.apache.ambari.server.api.services.AmbariMetaInfo.getComponent(AmbariMetaInfo.java:307)
at
org.apache.ambari.server.state.host.HostImpl.hasComponentsAdvertisingVersions(HostImpl.java:1176)
at
org.apache.ambari.server.controller.internal.ClusterStackVersionResourceProvider.createHostVersions(ClusterStackVersionResourceProvider.java:495)
at
org.apache.ambari.server.controller.internal.ClusterStackVersionResourceProvider.createResourcesAuthorized(ClusterStackVersionResourceProvider.java:447)
... 93 more
{code}
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/checks/CheckDescription.java
2be42fc77d
ambari-server/src/main/java/org/apache/ambari/server/checks/ComponentsExistInRepoCheck.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/state/Host.java
241659a111
ambari-server/src/main/java/org/apache/ambari/server/state/host/HostImpl.java
328fe221e2
ambari-server/src/test/java/org/apache/ambari/server/checks/ComponentExistsInRepoCheckTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/60850/diff/2/
Changes: https://reviews.apache.org/r/60850/diff/1-2/
Testing
-------
PENDING
Thanks,
Jonathan Hurley
