ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vitalyi Brodetskyi" <vbrodets...@hortonworks.com>
Subject Re: Review Request 30885: After upgrading Ambari 1.6.1 to 1.7, the directory of zookeeper server get changed to /hadoop/zookeeper
Date Wed, 11 Feb 2015 19:02:42 GMT

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

Ship it!


Ship It!

- Vitalyi Brodetskyi


On Лют. 11, 2015, 6:54 після полудня, Dmitro Lisnichenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30885/
> -----------------------------------------------------------
> 
> (Updated Лют. 11, 2015, 6:54 після полудня)
> 
> 
> Review request for Ambari and Vitalyi Brodetskyi.
> 
> 
> Bugs: AMBARI-9579
>     https://issues.apache.org/jira/browse/AMBARI-9579
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> PROBLEM: After upgrading Ambari 1.6.1 to 1.7, the directory of zookeeper server get changed
to /hadoop/zookeeper
> BUSINESS IMPACT: Customer is upgrading Ambari on an HA cluster. Due to this bug, customer
was not able to start HDFS after upgrade, and was mislead to perform a -formatZK to workaround
the issue.
> STEPS TO REPRODUCE: 
> 1. Deploy a Ambari 1.6.1 cluster
> (I believe this is regardless of HDP version, but I used a HDP 2.1.3)
> 2. Set zookeeper server directory to something other than /hadoop/zookeeper
> 3. Upgrade Ambari to 1.7
> 4. Observe that zookeeper directory got change back to /hadoop/zookeeper
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog170.java
a7736dd 
>   ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog170Test.java
42735c0 
> 
> Diff: https://reviews.apache.org/r/30885/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Dmitro Lisnichenko
> 
>


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