<!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>
    
  </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>
    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> 
  </blockquote> 
  <p>I just checked with C7-QA, and that issue is there as well. I filed the bug #7257 against C7-QA to track this.</p> 
  <p>Thanks</p> 
  <p>Christoph</p> 
  <blockquote style="position: relative; margin-left: 0px; padding-left: 10px; border-left: solid 1px blue;" type="cite"> 
   <div>
     
   </div> 
  </blockquote> 
  <div id="ox-signature">
   --
   <br />Christoph Galuschka
   <br />CentOS-QA member | IRC: tigalch
  </div>
 
</body></html>