ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Hurley" <jhur...@hortonworks.com>
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:05:26 GMT

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

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


Review request for Ambari, Dmytro Sen and Nate Cole.


Changes
-------

Updating Python test output.


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 
  ambari-agent/src/test/python/ambari_agent/TestRegistration.py c4e074d 

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


Testing (updated)
-------

----------------------------------------------------------------------
Ran 305 tests in 5.993s

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] 550 resources included (use -debug for more details)
Warning:  org.apache.xerces.jaxp.SAXParserImpl$JAXPSAXParser: Property 'http://www.oracle.com/xml/jaxp/properties/entityExpansionLimit'
is not recognized.
Warning:  org.apache.xerces.parsers.SAXParser: Feature 'http://javax.xml.XMLConstants/feature/secure-processing'
is not recognized.
Warning:  org.apache.xerces.parsers.SAXParser: Property 'http://www.oracle.com/xml/jaxp/properties/entityExpansionLimit'
is not recognized.
[INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0 approved: 408 licence.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 8.384s
[INFO] Finished at: Wed May 07 15:55:39 EDT 2014
[INFO] Final Memory: 7M/81M
[INFO] ------------------------------------------------------------------------


Thanks,

Jonathan Hurley


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