> On Nov. 8, 2016, 3:54 a.m., Alejandro Fernandez wrote: > > Does this need to be tested on CentOS 6? If everything passes, +1
I tested on CentOS 6.7 with ntpd installed, and it works. - Masahiro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53506/#review155181 ----------------------------------------------------------- On Nov. 6, 2016, 10:23 a.m., Masahiro Tanaka wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53506/ > ----------------------------------------------------------- > > (Updated Nov. 6, 2016, 10:23 a.m.) > > > Review request for Ambari, Andrew Onischuk, Sumit Mohanty, and Sid Wagle. > > > Bugs: AMBARI-18761 > https://issues.apache.org/jira/browse/AMBARI-18761 > > > Repository: ambari > > > Description > ------- > > I use CentOS 7 as a host OS. When the host check runs, it shows warning. The > warning says "The following services should be up: Service ntpd." > I think "chrony" is a default NTP client and server on CentOS7. So We should > check if there are ntpd or chronyd on a host. > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/HostInfo.py 3871e72 > ambari-agent/src/test/python/ambari_agent/TestHostInfo.py 0c0a3b3 > > Diff: https://reviews.apache.org/r/53506/diff/ > > > Testing > ------- > > ``` > +1 overall. Here are the results of testing the latest attachment > http://issues.apache.org/jira/secure/attachment/12837535/AMBARI-18761.3.patch > against trunk revision . > +1 @author. The patch does not contain any @author tags. > +1 tests included. The patch appears to include 1 new or modified test files. > +1 javac. The applied patch does not increase the total number of javac > compiler warnings. > +1 release audit. The applied patch does not increase the total number of > release audit warnings. > +1 core tests. The patch passed unit tests in . > Test results: > https://builds.apache.org/job/Ambari-trunk-test-patch/9154//testReport/ > Console output: > https://builds.apache.org/job/Ambari-trunk-test-patch/9154//console > This message is automatically generated. > ``` > > Manual test: Installed Ambari2.4.1 on centos7, replaced HostInfo.py and > recheck host. > > > Thanks, > > Masahiro Tanaka > >
