Hi,

thanks for keeping the eyes open. This is allready at bugs.centos.org at issue 6957. Will try to check that for C7.

Thanks and cheers Christoph

Gianluca Cecchi gianluca.cecchi@gmail.com hat am 26. Juni 2014 um 11:50 geschrieben:

Hello, 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".

### Added by IPA Installer ### serverhttp://0.rhel.pool.ntp.org serverhttp://1.rhel.pool.ntp.org serverhttp://2.rhel.pool.ntp.org

The rpm involved is ipa-server-3.0.0-37.el6.x86_64 and the file involved is /usr/lib/python2.6/site-packages/ipaserver/install/ntpinstance.py

where there is this if conditional statement

     # We use the OS variable to point it towards either the rhel
     # or fedora pools. Other distros should be added in the future
     # or we can get our own pool.
     os = ""
     if ipautil.file_exists("/etc/fedora-release"):
         os = "fedora"
     elif ipautil.file_exists("/etc/redhat-release"):
         os = "rhel"

So I think this should be changed in something like

     os = ""
     if ipautil.file_exists("/etc/fedora-release"):
         os = "fedora"
     elif ipautil.file_exists("/etc/centos-release"):
         os = "centos"

What do you think? Possibly valid for rhel 7 too, but not already tested.

Gianluca

---

CentOS-devel mailing list CentOS-devel@centos.org http://lists.centos.org/mailman/listinfo/centos-devel

-- Christoph Galuschka CentOS-QA member | IRC: tigalch

Gianluca Cecchi gianluca.cecchi@gmail.com hat am 26. Juni 2014 um 11:50 geschrieben:

Hello, 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".

### Added by IPA Installer ### serverhttp://0.rhel.pool.ntp.org serverhttp://1.rhel.pool.ntp.org serverhttp://2.rhel.pool.ntp.org

The rpm involved is ipa-server-3.0.0-37.el6.x86_64 and the file involved is /usr/lib/python2.6/site-packages/ipaserver/install/ntpinstance.py

where there is this if conditional statement

     # We use the OS variable to point it towards either the rhel
     # or fedora pools. Other distros should be added in the future
     # or we can get our own pool.
     os = ""
     if ipautil.file_exists("/etc/fedora-release"):
         os = "fedora"
     elif ipautil.file_exists("/etc/redhat-release"):
         os = "rhel"

So I think this should be changed in something like

     os = ""
     if ipautil.file_exists("/etc/fedora-release"):
         os = "fedora"
     elif ipautil.file_exists("/etc/centos-release"):
         os = "centos"

What do you think? Possibly valid for rhel 7 too, but not already tested.

I just checked with C7-QA, and that issue is there as well. I filed the bug #7257 against C7-QA to track this.

Thanks

Christoph

-- Christoph Galuschka CentOS-QA member | IRC: tigalch