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

(Updated April 14, 2015, 2:21 p.m.)


Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Tom 
Beerbower.


Bugs: AMBARI-10472
    https://issues.apache.org/jira/browse/AMBARI-10472


Repository: ambari


Description
-------

* Fixed code that was always assuming "current stack" operation.  This includes 
distributing the bits.
* Added compatible stacks endpoint to show (duh) compatible stack versions in 
one request.  This will be used exclusively to show all the versions a cluster 
may move to (say, HDP-2.2 to HDP-2.3).  This will require UI work, filed 
separately.
* Added upgrade pack definitions that are copied from 2.2.  We'll make changes 
as we get new runbook details.
* I could have spent another 3 days writing tests, so for now it's just for the 
compatibility checks and verification that all other tests continue to function 
properly.  They'll get filled out as we move further into the cycle.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java
 07e70ab 
  
ambari-server/src/main/java/org/apache/ambari/server/api/resources/StackVersionResourceDefinition.java
 0b26308 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/CompatibleRepositoryVersionService.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/StacksService.java
 dd7ec10 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProvider.java
 eb1a818 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CompatibleRepositoryVersionResourceProvider.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/OperatingSystemResourceProvider.java
 b656def 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RepositoryVersionResourceProvider.java
 ada207a 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java
 e928e8e 
  
ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/DistributeRepositoriesActionListener.java
 19c1aad 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/RepositoryVersionEntity.java
 23c121e 
  ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java 
787f5d6 
  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
 8e89cad 
  
ambari-server/src/main/java/org/apache/ambari/server/state/stack/UpgradePack.java
 b10f3f7 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml 
PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml 
PRE-CREATION 
  
ambari-server/src/test/java/org/apache/ambari/server/api/query/QueryImplTest.java
 88e62ff 
  
ambari-server/src/test/java/org/apache/ambari/server/api/resources/StackVersionResourceDefinitionTest.java
 518b3cf 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
 7c2cdcd 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterControllerImplTest.java
 147dc49 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/CompatibleRepositoryVersionResourceProviderTest.java
 PRE-CREATION 
  
ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_to_new_stack.xml
 PRE-CREATION 

Diff: https://reviews.apache.org/r/33175/diff/


Testing
-------

Manual API testing against a live cluster.

Automated:
Tests run: 2854, Failures: 0, Errors: 0, Skipped: 16

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 21:44.208s
[INFO] Finished at: Tue Apr 14 13:17:17 EDT 2015
[INFO] Final Memory: 41M/686M
[INFO] ------------------------------------------------------------------------


Thanks,

Nate Cole

Reply via email to