You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Jonathan Hurley <jh...@hortonworks.com> on 2014/05/09 23:16:08 UTC

Review Request 21284: All Services Fail To Deploy Due To Agent Parsing Exception

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

Review request for Ambari and Sid Wagle.


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


Repository: ambari


Description
-------

New code added breaks all agent parsing when the Ambari Server is not hosted on a cluster host.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/utils/StageUtils.java 767a97d 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/params.py 7c5ac6e 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/shared_initialization.py c588ffc 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/params.py 43f375d 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/shared_initialization.py 393f1b1 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/package/scripts/hive.py 276410b 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/package/scripts/params.py 1f21780 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/NAGIOS/package/files/check_namenodes_ha.sh a0b348e 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/NAGIOS/package/files/check_nodemanager_health.sh 5b2a93f 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/NAGIOS/package/files/check_templeton_status.sh 6aa944b 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/NAGIOS/package/files/check_webui.sh 2be3388 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/package/scripts/oozie.py 4ab267a 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/OOZIE/package/scripts/params.py ae4da20 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/WEBHCAT/package/files/templetonSmoke.sh 6ec9a0f 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/params.py eeb21e8 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/shared_initialization.py 9ac5667 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/params.py 861ed55 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/shared_initialization.py 3bad0e7 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/package/scripts/hive.py 1445b24 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/package/scripts/params.py 5ab1753 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/package/files/check_namenodes_ha.sh a0b348e 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/package/files/check_nodemanager_health.sh 5b2a93f 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/package/files/check_templeton_status.sh 6aa944b 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/package/files/check_webui.sh a40fdf1 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/package/scripts/oozie.py 6feb7b1 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/package/scripts/params.py 59451aa 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/WEBHCAT/package/files/templetonSmoke.sh f27503b 
  ambari-server/src/test/java/org/apache/ambari/server/utils/TestStageUtils.java 7bfe605 
  ambari-server/src/test/python/stacks/1.3.2/HIVE/test_hive_client.py 98f6321 
  ambari-server/src/test/python/stacks/1.3.2/HIVE/test_hive_metastore.py 1366520 
  ambari-server/src/test/python/stacks/1.3.2/HIVE/test_hive_server.py 700bb66 
  ambari-server/src/test/python/stacks/1.3.2/configs/default.hbasedecom.json 9295c69 
  ambari-server/src/test/python/stacks/1.3.2/configs/default.json 84dcc8d 
  ambari-server/src/test/python/stacks/1.3.2/configs/default.non_gmetad_host.json 57ec6d9 
  ambari-server/src/test/python/stacks/1.3.2/configs/secured.json 44f6506 
  ambari-server/src/test/python/stacks/1.3.2/configs/secured_no_jce_name.json b824d40 
  ambari-server/src/test/python/stacks/1.3.2/hooks/before-INSTALL/test_before_install.py f68da7b 
  ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_client.py b210e3b 
  ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_metastore.py cbb6da3 
  ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_server.py f1486d3 
  ambari-server/src/test/python/stacks/2.0.6/configs/default.hbasedecom.json c45918d 
  ambari-server/src/test/python/stacks/2.0.6/configs/default.json 3561b4b 
  ambari-server/src/test/python/stacks/2.0.6/configs/default.non_gmetad_host.json 0a5ccae 
  ambari-server/src/test/python/stacks/2.0.6/configs/flume_target.json cb81436 
  ambari-server/src/test/python/stacks/2.0.6/configs/ha_default.json f152ed7 
  ambari-server/src/test/python/stacks/2.0.6/configs/ha_secured.json 6f60980 
  ambari-server/src/test/python/stacks/2.0.6/configs/secured.json ffa1c55 
  ambari-server/src/test/python/stacks/2.0.6/configs/secured_no_jce_name.json f403cde 
  ambari-server/src/test/python/stacks/2.0.6/hooks/before-INSTALL/test_before_install.py e70debb 
  ambari-server/src/test/python/stacks/2.1/HIVE/test_hive_metastore.py 0e56eb0 
  ambari-server/src/test/python/stacks/2.1/configs/default.json d80c28d 
  ambari-server/src/test/python/stacks/2.1/configs/secured.json 68b04d0 

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


Testing
-------

If you try to deploy a new cluster on a clean database, you'll encounter the error.

File "/usr/lib/python2.6/site-packages/ambari_agent/CustomServiceOrchestrator.py", line 113, in runCommand
json_path = self.dump_command_to_json(command)
File "/usr/lib/python2.6/site-packages/ambari_agent/CustomServiceOrchestrator.py", line 209, in dump_command_to_json
command'clusterHostInfo' = manifestGenerator.decompressClusterHostInfo(command'clusterHostInfo')
File "/usr/lib/python2.6/site-packages/ambari_agent/manifestGenerator.py", line 116, in decompressClusterHostInfo
indexes = convertRangeToList(v)
File "/usr/lib/python2.6/site-packages/ambari_agent/manifestGenerator.py", line 57, in convertRangeToList
raise AgentException.AgentException("Broken data in given range, expected - ""m-n"" or ""m"", got : " + str(r))
AgentException: 'Broken data in given range, expected - m-n or m, got : -1’
The command being sent is
{hs_host=[2], namenode_host=[1], snamenode_host=[2], zookeeper_hosts=[0-2], ganglia_server_host=[1], nm_hosts=[0], ganglia_monitor_hosts=[0-2], all_hosts=[c6403.ambari.apache.org, c6401.ambari.apache.org, c6402.ambari.apache.org], rm_host=[2], app_timeline_server_hosts=[2], slave_hosts=[0], ambari_server_host=[-1], nagios_server_host=[1], all_ping_ports=[8670:0-2]}

Notice the ambari-server-host which was added in that commit; it’s value is β€œ-1” which would not parse correctly in manifestGenerator.py


Thanks,

Jonathan Hurley