-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21116/
-----------------------------------------------------------
Review request for Ambari, Dmytro Sen and Nate Cole.
Bugs: AMBARI-5686
https://issues.apache.org/jira/browse/AMBARI-5686
Repository: ambari
Description
-------
The iptables check in the Agent was done by spawning a new process for iptables
and checking the result against a pre-determined string value. This did not
work on CentOS 5.x as the string values were different and the -S option was
not available. Instead, we are now using the return code from the "service"
command to determine the state of iptables.
Diffs
-----
ambari-agent/src/main/python/ambari_agent/HostInfo.py 8e0cf90
ambari-agent/src/test/python/ambari_agent/TestHostInfo.py 627261d
Diff: https://reviews.apache.org/r/21116/diff/
Testing
-------
----------------------------------------------------------------------
Ran 304 tests in 3.968s
OK
[INFO]
[INFO] --- apache-rat-plugin:0.10:check (default) @ ambari-agent ---
[INFO] 51 implicit excludes (use -debug for more details).
[INFO] Exclude: src/examples/*
[INFO] Exclude: src/test/python/dummy*.txt
[INFO] Exclude: src/test/python/ambari_agent/dummy_files/*
[INFO] Exclude: src/test/python/ambari_agent/dummy*.txt
[INFO] Exclude: src/main/python/ambari_agent/imports.txt
[INFO] Exclude: src/main/puppet/modules/stdlib/**
[INFO] Exclude: **/*.erb
[INFO] Exclude: **/*.json
[INFO] 548 resources included (use -debug for more details)
[INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0
approved: 407 licence.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 6.073s
[INFO] Finished at: Tue May 06 11:55:02 EDT 2014
[INFO] Final Memory: 7M/81M
[INFO] ------------------------------------------------------------------------
Thanks,
Jonathan Hurley
