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

(Updated March 14, 2014, 2:56 p.m.)


Review request for Ambari and Sid Wagle.


Changes
-------

If command['configurationTags'] is None then get error:
{noformat}
raceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActionQueue.py", line 
131, in process_command
    self.execute_command(command)
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActionQueue.py", line 
229, in execute_command
    configHandler.write_actual_component(command['role'], 
command['configurationTags'])
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActualConfigHandler.py", 
line 50, in write_actual_component
    self.write_client_components(tags)
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActualConfigHandler.py", 
line 58, in write_client_components
    if config_name in self.configTags[componentName] and \
TypeError: argument of type 'NoneType' is not iterable
Opening /proc/modules: No such file or directory
Traceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActionQueue.py", line 
131, in process_command
    self.execute_command(command)
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActionQueue.py", line 
229, in execute_command
    configHandler.write_actual_component(command['role'], 
command['configurationTags'])
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActualConfigHandler.py", 
line 50, in write_actual_component
    self.write_client_components(tags)
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActualConfigHandler.py", 
line 58, in write_client_components
    if config_name in self.configTags[componentName] and \
TypeError: argument of type 'NoneType' is not iterable
Traceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActionQueue.py", line 
131, in process_command
    self.execute_command(command)
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActionQueue.py", line 
229, in execute_command
    configHandler.write_actual_component(command['role'], 
command['configurationTags'])
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActualConfigHandler.py", 
line 50, in write_actual_component
    self.write_client_components(tags)
  File "/usr/lib/python2.6/site-packages/ambari_agent/ActualConfigHandler.py", 
line 58, in write_client_components
{noformat}


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 (updated)
-----

  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

Reply via email to