[CentOS-devel] ipa-server package and sort of branding issue

> Gianluca Cecchi <gianluca.cecchi at 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 ###
>  server<http://0.rhel.pool.ntp.org>
>  server<http://1.rhel.pool.ntp.org>
>  server<http://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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.centos.org/pipermail/centos-devel/attachments/20140626/2c60afba/attachment-0003.html>