You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Vitalyi Brodetskyi <vb...@hortonworks.com> on 2017/05/03 11:45:25 UTC

Review Request 58960: Install: Recommend chronyd instead of ntpd for RHEL 7

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

Review request for Ambari, Andrew Onischuk, Dmytro Sen, and Sumit Mohanty.


Bugs: AMBARI-20917
    https://issues.apache.org/jira/browse/AMBARI-20917


Repository: ambari


Description
-------

RHEL 7 has moved away from NTPD, and is now using chronyd (https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/System_Administrators_Guide/ch-Configuring_NTP_Using_the_chrony_Suite.html). For the host checks we perform during add host we need to add logic to recommend chronyd instead of ntpd.


Diffs
-----

  ambari-agent/src/main/python/ambari_agent/HostInfo.py 4b7bfd7 


Diff: https://reviews.apache.org/r/58960/diff/1/


Testing
-------

mvn clean test


Thanks,

Vitalyi Brodetskyi


Re: Review Request 58960: Install: Recommend chronyd instead of ntpd for RHEL 7

Posted by Sumit Mohanty <sm...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58960/#review173734
-----------------------------------------------------------


Ship it!




Ship It!

- Sumit Mohanty


On May 3, 2017, 11:45 a.m., Vitalyi Brodetskyi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58960/
> -----------------------------------------------------------
> 
> (Updated May 3, 2017, 11:45 a.m.)
> 
> 
> Review request for Ambari, Andrew Onischuk, Dmytro Sen, and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-20917
>     https://issues.apache.org/jira/browse/AMBARI-20917
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> RHEL 7 has moved away from NTPD, and is now using chronyd (https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/System_Administrators_Guide/ch-Configuring_NTP_Using_the_chrony_Suite.html). For the host checks we perform during add host we need to add logic to recommend chronyd instead of ntpd.
> 
> 
> Diffs
> -----
> 
>   ambari-agent/src/main/python/ambari_agent/HostInfo.py 4b7bfd7 
> 
> 
> Diff: https://reviews.apache.org/r/58960/diff/1/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Vitalyi Brodetskyi
> 
>


Re: Review Request 58960: Install: Recommend chronyd instead of ntpd for RHEL 7

Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58960/#review173781
-----------------------------------------------------------


Ship it!




Ship It!

- Alejandro Fernandez


On May 3, 2017, 11:45 a.m., Vitalyi Brodetskyi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58960/
> -----------------------------------------------------------
> 
> (Updated May 3, 2017, 11:45 a.m.)
> 
> 
> Review request for Ambari, Andrew Onischuk, Dmytro Sen, and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-20917
>     https://issues.apache.org/jira/browse/AMBARI-20917
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> RHEL 7 has moved away from NTPD, and is now using chronyd (https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/System_Administrators_Guide/ch-Configuring_NTP_Using_the_chrony_Suite.html). For the host checks we perform during add host we need to add logic to recommend chronyd instead of ntpd.
> 
> 
> Diffs
> -----
> 
>   ambari-agent/src/main/python/ambari_agent/HostInfo.py 4b7bfd7 
> 
> 
> Diff: https://reviews.apache.org/r/58960/diff/1/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Vitalyi Brodetskyi
> 
>