ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitro Lisnichenko" <dlysniche...@hortonworks.com>
Subject Review Request 31371: Blank current stack version on the host added via addHostWizard
Date Tue, 24 Feb 2015 19:21:44 GMT

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

Review request for Ambari, Alejandro Fernandez, Nate Cole, and Tom Beerbower.


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


Repository: ambari


Description
-------

If cluster has both CURRENT and OUT_OF_SYNC host version, and we add a new host, only OUT_OF_SYNC
host version is created (creating CURRENT host version is skipped)


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/StackVersionResponse.java
005b8a1 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackVersionResourceProvider.java
ca66819 
  ambari-server/src/main/java/org/apache/ambari/server/stack/ComponentModule.java c86073f

  ambari-server/src/main/java/org/apache/ambari/server/stack/ConfigurationDirectory.java 7f21aaa

  ambari-server/src/main/java/org/apache/ambari/server/stack/ConfigurationInfo.java d00b135

  ambari-server/src/main/java/org/apache/ambari/server/stack/ConfigurationModule.java 90e7ed3

  ambari-server/src/main/java/org/apache/ambari/server/stack/ServiceDirectory.java 9b32a97

  ambari-server/src/main/java/org/apache/ambari/server/stack/ServiceModule.java 8b94101 
  ambari-server/src/main/java/org/apache/ambari/server/stack/StackDefinitionModule.java b35afb2

  ambari-server/src/main/java/org/apache/ambari/server/stack/StackDirectory.java d38cee9 
  ambari-server/src/main/java/org/apache/ambari/server/stack/StackManager.java 33f1447 
  ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java 4181293 
  ambari-server/src/main/java/org/apache/ambari/server/stack/Validable.java e8b5f24 
  ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java b1ee121 
  ambari-server/src/main/java/org/apache/ambari/server/state/StackInfo.java 4b6898a 
  ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java b2ba43b

  ambari-server/src/main/java/org/apache/ambari/server/state/stack/ConfigurationXml.java ca33601

  ambari-server/src/main/java/org/apache/ambari/server/state/stack/RepositoryXml.java eff063b

  ambari-server/src/main/java/org/apache/ambari/server/state/stack/ServiceMetainfoXml.java
56dac8a 
  ambari-server/src/main/java/org/apache/ambari/server/state/stack/StackMetainfoXml.java 55b1b9b

  ambari-server/src/main/resources/properties.json cdb2bd6 
  ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java
0d944ab 
  ambari-views/examples/weather-view/pom.xml 6845620 

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


Testing
-------

manual tests
Unit test coverage is pending


Thanks,

Dmitro Lisnichenko


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