ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nate Cole" <nc...@hortonworks.com>
Subject Review Request 33340: Add Prerequisite Check for Merged Configs
Date Sun, 19 Apr 2015 14:28:15 GMT

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

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


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


Repository: ambari


Description
-------

* Added prereq check for merged configs
* Fixed host check to handle when a version is specified, to check the proper version


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/checks/CheckDescription.java 0db235f

  ambari-server/src/main/java/org/apache/ambari/server/checks/ConfigurationMergeCheck.java
PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/checks/HostsRepositoryVersionCheck.java
0db7e2e 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/PreUpgradeCheckResourceProvider.java
ee8cefce 
  ambari-server/src/main/java/org/apache/ambari/server/orm/dao/RepositoryVersionDAO.java db5e956

  ambari-server/src/main/java/org/apache/ambari/server/orm/entities/RepositoryVersionEntity.java
dd5ac0a 
  ambari-server/src/main/java/org/apache/ambari/server/state/ConfigMergeHelper.java PRE-CREATION

  ambari-server/src/main/java/org/apache/ambari/server/state/stack/PrereqCheckStatus.java
3384b85 
  ambari-server/src/test/java/org/apache/ambari/server/checks/ConfigurationMergeCheckTest.java
PRE-CREATION 
  ambari-server/src/test/java/org/apache/ambari/server/checks/HostsRepositoryVersionCheckTest.java
96151af 

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


Testing
-------

Manual cluster testing + Automated:

Tests run: 2867, Failures: 0, Errors: 0, Skipped: 17

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 22:56.903s
[INFO] Finished at: Sun Apr 19 09:40:46 EDT 2015
[INFO] Final Memory: 40M/780M
[INFO] ------------------------------------------------------------------------


Thanks,

Nate Cole


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