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 19185: Rolling restart should also handle clients on the same machine as the restarting component
Date Mon, 17 Mar 2014 17:39:11 GMT

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

Ship it!


BTW, do ambari really has to store contigurationTags for the client components, if they are
updated automatic
- We need this for hosts with Client only hosts with no co-hosted Master / Slave.

- Sid Wagle


On March 17, 2014, 3:21 p.m., Dmytro Sen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19185/
> -----------------------------------------------------------
> 
> (Updated March 17, 2014, 3:21 p.m.)
> 
> 
> Review request for Ambari and Sid Wagle.
> 
> 
> Bugs: AMBARI-5065
>     https://issues.apache.org/jira/browse/AMBARI-5065
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> 3-host cluster, c6401 has nn, c6402 and c6403 have datanodes + hdfs clients
> 1. create host group in Hdfs
> 2. put one host in the group (say c6403) that has datanode and hdfs client
> 3. change datanode max java heap size so it only affects the datanode
> 4. you'll see a restart is needed only on c6403 as expected (correct) for datanode and
hdfs client
> 5. Issue a rolling restart and only on hosts that need configs (so just one host)
> 6. the works fine and it only restarts datanode c6403 as expected (correct)
> problem: hdfs client is not marked as restarted so i still see "1 HDFS Client" component
requiring a restart on that host.
> Need to handle clients on the machines as well as part of rolling restart.
> Solution:
> Update client tags on Master/Slave component update in-memory and in individual files.
> 
> 
> Diffs
> -----
> 
>   ambari-agent/src/main/python/ambari_agent/ActionQueue.py 549651a 
>   ambari-agent/src/main/python/ambari_agent/ActualConfigHandler.py 024c575 
>   ambari-agent/src/main/python/ambari_agent/LiveStatus.py 6f9f4db 
>   ambari-agent/src/test/python/ambari_agent/TestActualConfigHandler.py e62dd1a 
>   ambari-agent/src/test/python/ambari_agent/TestLiveStatus.py 96a21b9 
> 
> Diff: https://reviews.apache.org/r/19185/diff/
> 
> 
> Testing
> -------
> 
> Ran 284 tests in 11.538s
> 
> OK
> 
> 
> Thanks,
> 
> Dmytro Sen
> 
>


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