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

Thu Jun 26 09:54:08 UTC 2014
Christoph.Galuschka <tigalch at tigalch.org>

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 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.
> 
>  Gianluca
> 
>  _______________________________________________
>  CentOS-devel mailing list
>  CentOS-devel at centos.org
>  http://lists.centos.org/mailman/listinfo/centos-devel
> 


--
Christoph Galuschka
CentOS-QA member | IRC: tigalch
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.centos.org/pipermail/centos-devel/attachments/20140626/253a6056/attachment-0007.html>