ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sid Wagle" <swa...@hortonworks.com>
Subject Re: Review Request 21403: Store cluster installed state as part of the Cluster resource
Date Thu, 15 May 2014 15:59:42 GMT

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

Ship it!


Ship It!

- Sid Wagle


On May 15, 2014, 2:38 p.m., Jonathan Hurley wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21403/
> -----------------------------------------------------------
> 
> (Updated May 15, 2014, 2:38 p.m.)
> 
> 
> Review request for Ambari, Nate Cole and Sid Wagle.
> 
> 
> Bugs: AMBARI-5749
>     https://issues.apache.org/jira/browse/AMBARI-5749
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Clients have requested to have the state of cluster provisioning defined in a more structured
manner. They currently use the /persist endpoint to store the state of cluster provisioning,
which if it becomes corrupted, can place the Ambari Web Client in a bad state.
> 
> We have now exposed a "provisioning_state" property on the cluster resource so allow
the client to mark a cluster as being either INIT or INSTALLED. There are no in-between states
and once a cluster is marked as INSTALLED, it cannot be reverted back to INIT.
> 
> - New tests added to cover provisioned_state lifecycle states
> - Database scripts updated
> - Database upgrade steps added
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
b2151f5 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/ClusterRequest.java
1eaf7c7 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/ClusterResponse.java
557cf6e 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterResourceProvider.java
8b6c39a 
>   ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ClusterEntity.java
b9e18cf 
>   ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ClusterStateEntity.java
d955a95 
>   ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java 8955347 
>   ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
4717498 
>   ambari-server/src/main/java/org/apache/ambari/server/upgrade/SchemaUpgradeHelper.java
51d64d3 
>   ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog161.java
8b4a199 
>   ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql ce01146 
>   ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql c0472bd 
>   ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 5792556 
>   ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql 8c07e63 
>   ambari-server/src/main/resources/properties.json a529950 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
f2dd4c9 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/ClusterRequestTest.java
c1ce0f3 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/ClusterResponseTest.java
67924a1 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AbstractResourceProviderTest.java
6f4a74c 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterResourceProviderTest.java
f8bb351 
>   ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterTest.java
24bb6d1 
>   ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog161Test.java
415951f 
> 
> Diff: https://reviews.apache.org/r/21403/diff/
> 
> 
> Testing
> -------
> 
> Results :
> 
> Tests run: 1607, Failures: 0, Errors: 0, Skipped: 13
> 
> ----------------------------------------------------------------------
> Ran 203 tests in 0.493s
> 
> OK
> ----------------------------------------------------------------------
> Total run:554
> Total errors:0
> Total failures:0
> OK
> AMBARI_CONF_DIR is not set, using default /etc/ambari-server/conf
> log
> Log not found
> log
> Cluster primary/cluster OS type is aaa11 and local/current OS type is aaa11
> Cluster primary/cluster OS type is zzz_x77 and local/current OS type is ddd33
> WARNING: Can not find ambari.properties.rpmsave file from previous version, skipping
import of settings
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 13:04.560s
> [INFO] Finished at: Thu May 15 10:37:14 EDT 2014
> [INFO] Final Memory: 22M/123M
> [INFO] ------------------------------------------------------------------------
> 
> 
> File Attachments
> ----------------
> 
> Reverting ClusterStateEntity to a pre-State nature.
>   https://reviews.apache.org/media/uploaded/files/2014/05/15/a35e7641-4030-4d1c-b18b-a9fbd3e32b6f__AMBARI-5749.patch
> 
> 
> Thanks,
> 
> Jonathan Hurley
> 
>


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