ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jonathan Hurley <jhur...@hortonworks.com>
Subject Re: Review Request 43425: [preview] Component should support a desired version
Date Thu, 18 Feb 2016 16:14:47 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43425/#review119626
-----------------------------------------------------------




ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java
(lines 1153 - 1158)
<https://reviews.apache.org/r/43425/#comment180978>

    So if there is currently no CURRENT version of the cluster, this looks for "INSTALLING"
... but "INSTALLING" doesn't mean that the upgrade is happening. It means that the repo is
being distributed. If the cluster is ready to install on a cluster, I'd expect this to be
INSTALLED.



ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
(line 1648)
<https://reviews.apache.org/r/43425/#comment180977>

    Correct; we need to unset this on ABORT. But what about when re-setting the aborted upgrade
back to PENDING to resume it? It needs to get re-set, right?



ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/StackVersionListener.java
(lines 139 - 143)
<https://reviews.apache.org/r/43425/#comment180980>

    We can remove this, right? The host version state should never change b/c a version changes.
It now tracks the repo state, not the state of the cluster or the component.



ambari-server/src/main/java/org/apache/ambari/server/state/RepositoryVersionState.java 
<https://reviews.apache.org/r/43425/#comment180976>

    Is there any chance these exist in a database about to be upgraded? I'd hope not and that
means we can probably not worry about them on ambari upgrade.


- Jonathan Hurley


On Feb. 18, 2016, 11:10 a.m., Dmitro Lisnichenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43425/
> -----------------------------------------------------------
> 
> (Updated Feb. 18, 2016, 11:10 a.m.)
> 
> 
> Review request for Ambari, Jonathan Hurley and Nate Cole.
> 
> 
> Bugs: AMBARI-14996
>     https://issues.apache.org/jira/browse/AMBARI-14996
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> When performing an upgrade, we need to specify that a component's desired version is
getting changed. Also upgrade status is tracked based on component statuses (not host/cluster
versions)
> 
> 
> Diffs
> -----
> 
>   .gitignore a8c4aab 
>   ambari-server/src/main/java/org/apache/ambari/server/agent/HeartBeatHandler.java 248ce4b

>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java
b62f4d1 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RepositoryVersionResourceProvider.java
87731c4 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
40565c5 
>   ambari-server/src/main/java/org/apache/ambari/server/events/HostComponentVersionEvent.java
ee65d3d 
>   ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/StackVersionListener.java
74d4f4b 
>   ambari-server/src/main/java/org/apache/ambari/server/events/publishers/VersionEventPublisher.java
3a11f38 
>   ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ClusterEntity.java
2c4d695 
>   ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostComponentStateEntity.java
f92f645 
>   ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ServiceComponentDesiredStateEntity.java
65cc107 
>   ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java
03d407a 
>   ambari-server/src/main/java/org/apache/ambari/server/stack/MasterHostResolver.java
561350b 
>   ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java b430525 
>   ambari-server/src/main/java/org/apache/ambari/server/state/RepositoryVersionState.java
344f358 
>   ambari-server/src/main/java/org/apache/ambari/server/state/ServiceComponent.java 7803045

>   ambari-server/src/main/java/org/apache/ambari/server/state/ServiceComponentHost.java
f1e8d62 
>   ambari-server/src/main/java/org/apache/ambari/server/state/ServiceComponentImpl.java
4afc857 
>   ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java 5143bfa

>   ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeState.java ced1dd3

>   ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
8230fe3 
>   ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
92828af 
>   ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostSummary.java
1c36143 
>   ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql 2db745b 
>   ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 8c626f5 
>   ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql f8e4ee1 
>   ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql e3ce96c 
>   ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql 5d47c28 
>   ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 263004f 
>   ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 80feca5 
>   ambari-server/src/test/java/org/apache/ambari/server/StateRecoveryManagerTest.java
0e9b18d 
>   ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java
2a4cec8 
>   ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatMonitor.java
9c59cde 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
3e372f5 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackDefinedPropertyProviderTest.java
9b27447 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderHDP22Test.java
b0e3467 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java
035c61a 
>   ambari-server/src/test/java/org/apache/ambari/server/events/EventsTest.java ad53f70

>   ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/HostVersionOutOfSyncListenerTest.java
772d68a 
>   ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/StackVersionListenerTest.java
ae05a6b 
>   ambari-server/src/test/java/org/apache/ambari/server/events/publishers/VersionEventPublisherTest.java
071c6f0 
>   ambari-server/src/test/java/org/apache/ambari/server/orm/OrmTestHelper.java 71b89c3

>   ambari-server/src/test/java/org/apache/ambari/server/orm/dao/ClusterVersionDAOTest.java
b57db5b 
>   ambari-server/src/test/java/org/apache/ambari/server/orm/dao/HostVersionDAOTest.java
e8e5d20 
>   ambari-server/src/test/java/org/apache/ambari/server/orm/dao/UpgradeDAOTest.java 3ad2240

>   ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/ComponentVersionCheckActionTest.java
89f0ba6 
>   ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/ConfigureActionTest.java
14137cc 
>   ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/UpgradeActionTest.java
989eba2 
>   ambari-server/src/test/java/org/apache/ambari/server/state/ServiceComponentTest.java
55e7a61 
>   ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java d513448

>   ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterDeadlockTest.java
847de7d 
>   ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterTest.java
2ffcd5d 
>   ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClustersDeadlockTest.java
7cb7679 
>   ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClustersTest.java
daf8d12 
>   ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ConcurrentServiceConfigVersionTest.java
4845d6f 
>   ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ServiceComponentHostConcurrentWriteDeadlockTest.java
81399a1 
>   ambari-server/src/test/java/org/apache/ambari/server/state/host/HostTest.java 4248d13

>   ambari-server/src/test/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostTest.java
62f2cbf 
> 
> Diff: https://reviews.apache.org/r/43425/diff/
> 
> 
> Testing
> -------
> 
> Pending E2E RU/EU upgrades/downgrades on live cluster
> 
> Did not work on unit tests yet.
> 
> 
> Thanks,
> 
> Dmitro Lisnichenko
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message