ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Hurley" <>
Subject Re: Review Request 21116: iptables host check warning shows on centos 5.9 even if iptables are stopped
Date Wed, 07 May 2014 20:04:06 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated May 7, 2014, 4:04 p.m.)

Review request for Ambari, Dmytro Sen and Nate Cole.


Some linux distributions don't use the 'service iptables status" model. Updated the code to
detect the linux type and choose a different command. Note that some of the commands don't
report the result on the result code, so the stdout is used in these cases.

Manually tested the Python file on: CentOS5.9, CentOS6.4
Manually tested the commands on Ubuntu, SUSE Enterprise

Bugs: AMBARI-5686

Repository: ambari


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

  ambari-agent/src/main/python/ambari_agent/ 8e0cf90 
  ambari-agent/src/test/python/ambari_agent/ 627261d 
  ambari-agent/src/test/python/ambari_agent/ c4e074d 



Ran 304 tests in 3.968s

[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] ------------------------------------------------------------------------
[INFO] Total time: 6.073s
[INFO] Finished at: Tue May 06 11:55:02 EDT 2014
[INFO] Final Memory: 7M/81M
[INFO] ------------------------------------------------------------------------


Jonathan Hurley

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