<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"><html xmlns="http://www.w3.org/1999/xhtml"><head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
</head><body style="">
<div>
Hi,
</div>
<div>
 
</div>
<div>
thanks for keeping the eyes open.
</div>
<div>
This is allready at bugs.centos.org at issue 6957. Will try to check that for C7.
</div>
<div>
 
</div>
<div>
Thanks and cheers
</div>
<div>
Christoph
</div>
<div>
 
</div>
<blockquote style="position: relative; margin-left: 0px; padding-left: 10px; border-left: solid 1px blue;" type="cite">
Gianluca Cecchi <gianluca.cecchi@gmail.com> hat am 26. Juni 2014 um 11:50 geschrieben:
<br />
<br />
<div dir="ltr">
Hello,
<div>
I'm configuring ipa server on CentOS 6.5 and I notice that when the ipa-server-install command configures ntp.conf it puts inside "rhel" for pool of ntp servers instead of "centos".
</div>
<div>
 
</div>
<div>
<div>
### Added by IPA Installer ###
</div>
<div>
server
<a href="http://0.rhel.pool.ntp.org">0.rhel.pool.ntp.org</a>
</div>
<div>
server
<a href="http://1.rhel.pool.ntp.org">1.rhel.pool.ntp.org</a>
</div>
<div>
server
<a href="http://2.rhel.pool.ntp.org">2.rhel.pool.ntp.org</a>
</div>
</div>
<div>
 
</div>
<div>
 
</div>
<div>
The rpm involved is ipa-server-3.0.0-37.el6.x86_64
</div>
<div>
and the file involved is 
</div>
<div>
/usr/lib/python2.6/site-packages/ipaserver/install/ntpinstance.py
</div>
<div>
 
</div>
<div>
where there is this if conditional statement
</div>
<div>
 
</div>
<div>
<div>
        # We use the OS variable to point it towards either the rhel
</div>
<div>
        # or fedora pools. Other distros should be added in the future
</div>
<div>
        # or we can get our own pool.
</div>
<div>
        os = ""
</div>
<div>
        if ipautil.file_exists("/etc/fedora-release"):
</div>
<div>
            os = "fedora"
</div>
<div>
        elif ipautil.file_exists("/etc/redhat-release"):
</div>
<div>
            os = "rhel"
</div>
</div>
<div>
 
</div>
<div>
So I think this should be changed in something like
</div>
<div>
 
</div>
<div>
<div>
        os = ""
</div>
<div>
        if ipautil.file_exists("/etc/fedora-release"):
</div>
<div>
            os = "fedora"
</div>
<div>
        elif ipautil.file_exists("/etc/centos-release"):
</div>
<div>
            os = "centos"
</div>
</div>
<div>
 
</div>
<div>
What do you think?
</div>
<div>
Possibly valid for rhel 7 too, but not already tested.
</div>
<div>
 
</div>
<div>
Gianluca
</div>
<div>
 
</div>
</div> _______________________________________________
<br />CentOS-devel mailing list
<br />CentOS-devel@centos.org
<br />http://lists.centos.org/mailman/listinfo/centos-devel
</blockquote>
<div>
<br /> 
</div>
<div id="ox-signature">
--
<br />Christoph Galuschka
<br />CentOS-QA member | IRC: tigalch
</div>
</body></html>