-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59369/
-----------------------------------------------------------
(Updated May 23, 2017, 2:44 p.m.)
Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and
Jonathan Hurley.
Changes
-------
Update for tests. _Most_ tests are passing - stabilization will be in a
coming-soon patch.
Bugs: AMBARI-21059
https://issues.apache.org/jira/browse/AMBARI-21059
Repository: ambari
Description
-------
With the advent of {{PATCH}} and {{CLUSTER}} upgrades, there is no longer the
notion of a cluster being bound to a specific stack. Instead, the stack ID is
being pushed down to the service and component level.
Currently, upgrades tries to set the cluster desired stack ID on finalization.
Eventually there will be a complete decouple of the cluster from a singular
stack id. This jira covers a majority of cases in code. Until configs are
refactored, we can't remove the associated stack id in its entirety.
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/annotations/ExperimentalFeature.java
1d5ba0e9e9
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/Stage.java
574afa1c86
ambari-server/src/main/java/org/apache/ambari/server/agent/HeartBeatHandler.java
36015282ea
ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatMonitor.java
a77ed75c99
ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatProcessor.java
17e1f9c138
ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java
c655c6286f
ambari-server/src/main/java/org/apache/ambari/server/checks/AbstractCheckDescriptor.java
2fc1787e17
ambari-server/src/main/java/org/apache/ambari/server/checks/ComponentsInstallationCheck.java
70a061cc79
ambari-server/src/main/java/org/apache/ambari/server/checks/HostsMasterMaintenanceCheck.java
1e87319d84
ambari-server/src/main/java/org/apache/ambari/server/checks/HostsRepositoryVersionCheck.java
a4cea3105f
ambari-server/src/main/java/org/apache/ambari/server/checks/RangerPasswordCheck.java
a55a1481d6
ambari-server/src/main/java/org/apache/ambari/server/checks/ServiceCheckValidityCheck.java
4ccdc0a3d9
ambari-server/src/main/java/org/apache/ambari/server/checks/ServicesUpCheck.java
273bdaac68
ambari-server/src/main/java/org/apache/ambari/server/checks/YarnTimelineServerStatePreservingCheck.java
d8dba96094
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariActionExecutionHelper.java
9fb77e8d6b
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java
397c1c29e1
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementController.java
96bab85b47
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
e373f81f6b
ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java
8a5731bcbe
ambari-server/src/main/java/org/apache/ambari/server/controller/ServiceRequest.java
66c1a93081
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java
4ad01a54a7
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
6447888dde
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ComponentResourceProvider.java
24ef41a767
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ServiceResourceProvider.java
664ba42f34
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackDefinedPropertyProvider.java
8972ca220f
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
6027ce76f6
ambari-server/src/main/java/org/apache/ambari/server/controller/logging/LoggingSearchPropertyProvider.java
c69d00ba46
ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/AMSPropertyProvider.java
c4c2ddcbae
ambari-server/src/main/java/org/apache/ambari/server/controller/utilities/state/DefaultServiceCalculatedState.java
264ba030f7
ambari-server/src/main/java/org/apache/ambari/server/controller/utilities/state/HBaseServiceCalculatedState.java
d44515cdb4
ambari-server/src/main/java/org/apache/ambari/server/controller/utilities/state/HDFSServiceCalculatedState.java
89d4004f4f
ambari-server/src/main/java/org/apache/ambari/server/controller/utilities/state/HiveServiceCalculatedState.java
0643c94ce1
ambari-server/src/main/java/org/apache/ambari/server/controller/utilities/state/OozieServiceCalculatedState.java
4d0cf92c87
ambari-server/src/main/java/org/apache/ambari/server/controller/utilities/state/YARNServiceCalculatedState.java
24c460226c
ambari-server/src/main/java/org/apache/ambari/server/metadata/RoleCommandOrder.java
40ec0a1e80
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/ClusterDAO.java
38175709db
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/RepositoryVersionDAO.java
a2472b6a8c
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/RepositoryVersionEntity.java
47abde4660
ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/AutoSkipFailedSummaryAction.java
7a99f09ff0
ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java
cf2844bca6
ambari-server/src/main/java/org/apache/ambari/server/state/Clusters.java
aa53564661
ambari-server/src/main/java/org/apache/ambari/server/state/ConfigFactory.java
d6cd99786c
ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java
05b50ab915
ambari-server/src/main/java/org/apache/ambari/server/state/ConfigImpl.java
0e40254487
ambari-server/src/main/java/org/apache/ambari/server/state/ServiceComponentHost.java
9a35bcc97f
ambari-server/src/main/java/org/apache/ambari/server/state/ServiceComponentImpl.java
3c8ef3530c
ambari-server/src/main/java/org/apache/ambari/server/state/ServiceImpl.java
6bb0ffb25b
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
e4ac23e132
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java
bdc4f90f89
ambari-server/src/main/java/org/apache/ambari/server/state/configgroup/ConfigGroupImpl.java
a74e2a28d8
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ServiceCheckGrouping.java
dbfce48e8b
ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
e08b1f9ba5
ambari-server/src/main/java/org/apache/ambari/server/upgrade/AbstractUpgradeCatalog.java
cb1295961d
ambari-server/src/main/java/org/apache/ambari/server/upgrade/FinalUpgradeCatalog.java
38ad5baaec
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog200.java
edf107a9f4
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog210.java
7b7681c0fc
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog212.java
90854dd346
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog2121.java
0487cd7024
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog220.java
9cf7bbde4b
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog221.java
c235cf8e08
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog222.java
f0f92537c0
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java
8488795eac
ambari-server/src/main/java/org/apache/ambari/server/view/ViewRegistry.java
0125d54ac1
ambari-server/src/test/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapperTest.java
89ec32b543
ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java
76de02c3e7
ambari-server/src/test/java/org/apache/ambari/server/checks/HostsMasterMaintenanceCheckTest.java
7b7d8171a4
ambari-server/src/test/java/org/apache/ambari/server/checks/RangerPasswordCheckTest.java
91b3296332
ambari-server/src/test/java/org/apache/ambari/server/checks/ServiceCheckValidityCheckTest.java
996f349c0c
ambari-server/src/test/java/org/apache/ambari/server/checks/ServicesUpCheckTest.java
1368b8d9d6
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerImplTest.java
0735d5aa40
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
d1d819fbb9
ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java
8cfe258eae
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProviderTest.java
7b3837ef68
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ComponentResourceProviderTest.java
03e3e6639b
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/HostResourceProviderTest.java
b075b71598
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ServiceResourceProviderTest.java
c82c884e7a
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackDefinedPropertyProviderTest.java
4d44576c5b
ambari-server/src/test/java/org/apache/ambari/server/controller/logging/LoggingSearchPropertyProviderTest.java
1f2322cc4e
ambari-server/src/test/java/org/apache/ambari/server/controller/metrics/RestMetricsPropertyProviderTest.java
66e62a0e07
ambari-server/src/test/java/org/apache/ambari/server/controller/metrics/timeline/AMSPropertyProviderTest.java
24fd47bd05
ambari-server/src/test/java/org/apache/ambari/server/events/EventsTest.java
710e4e7cd6
ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/HostVersionOutOfSyncListenerTest.java
4ca20701f7
ambari-server/src/test/java/org/apache/ambari/server/metadata/RoleCommandOrderTest.java
d2cc3454c1
ambari-server/src/test/java/org/apache/ambari/server/metadata/RoleGraphTest.java
303ee89a33
ambari-server/src/test/java/org/apache/ambari/server/orm/OrmTestHelper.java
469e8c8212
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/AutoSkipFailedSummaryActionTest.java
fbad1b1d4c
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/ComponentVersionCheckActionTest.java
b06117b47e
ambari-server/src/test/java/org/apache/ambari/server/stageplanner/TestStagePlanner.java
70631475ec
ambari-server/src/test/java/org/apache/ambari/server/state/ConfigGroupTest.java
4437e6062e
ambari-server/src/test/java/org/apache/ambari/server/state/ConfigHelperTest.java
1709da8846
ambari-server/src/test/java/org/apache/ambari/server/state/ServiceComponentTest.java
4c9ffcc605
ambari-server/src/test/java/org/apache/ambari/server/state/alerts/AlertEventPublisherTest.java
f5f4e10590
ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterDeadlockTest.java
e7516e6f9b
ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterImplTest.java
ec5eef02c0
ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClustersDeadlockTest.java
022cf1f96a
ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClustersTest.java
d59d1d5c09
ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ConcurrentServiceConfigVersionTest.java
c643b2f969
ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ServiceComponentHostConcurrentWriteDeadlockTest.java
0678a71869
ambari-server/src/test/java/org/apache/ambari/server/state/services/RetryUpgradeActionServiceTest.java
f996aac0d7
ambari-server/src/test/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostTest.java
d5c1b1a68c
ambari-server/src/test/java/org/apache/ambari/server/upgrade/AbstractUpgradeCatalogTest.java
4dd7fd9bb1
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog200Test.java
1649078a85
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog210Test.java
6c2e9f7c95
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog211Test.java
c705d89045
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog212Test.java
896602b54c
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog220Test.java
fc754a06fe
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog221Test.java
f4b3897dee
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog222Test.java
82ba1498de
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog240Test.java
70673f8114
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java
3cb2c47075
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java
c949ca2762
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalogHelper.java
6b28846ca3
ambari-server/src/test/java/org/apache/ambari/server/view/ViewRegistryTest.java
eda232b29c
ambari-web/app/controllers/wizard/step8_controller.js 7e318e0e62
Diff: https://reviews.apache.org/r/59369/diff/2/
Changes: https://reviews.apache.org/r/59369/diff/1-2/
Testing
-------
Manual: Install cluster; add service; run service checks; deploy both patch and
full version; full and patch RU/EU.
Automated pending unit test fixes.
Thanks,
Nate Cole