----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35876/#review89832 -----------------------------------------------------------
Thisremoves whitespace from both before making the comparison. Can you explain how this works in the case you mentioned? ambari-agent/src/main/python/ambari_agent/Facter.py (line 384) <https://reviews.apache.org/r/35876/#comment142650> semicolon is not needed. variable should be called ip_address_by_ifname - Alejandro Fernandez On June 25, 2015, 2:49 p.m., Di Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35876/ > ----------------------------------------------------------- > > (Updated June 25, 2015, 2:49 p.m.) > > > Review request for Ambari and Alejandro Fernandez. > > > Bugs: AMBARI-11854 > https://issues.apache.org/jira/browse/AMBARI-11854 > > > Repository: ambari > > > Description > ------- > > In a cluster with nodes that has multiple network interfaces. Ambari-agent > fails to start due to one or more active network interface did not bind to an > IP address. This happens when the eth0 network interface does not have an IP > address. > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/Facter.py 94a0ab1 > ambari-agent/src/test/python/ambari_agent/TestHardware.py 61456c5 > > Diff: https://reviews.apache.org/r/35876/diff/ > > > Testing > ------- > > 1. unit test > 2. manually run the script on a cluster where the eth0 network interface does > not have an IP address assigned. > > > Thanks, > > Di Li > >
