Project

General

Profile

Bug #6884

Centos mirror in "Installation Media" doesn't work for versions without $minor

Added by Jorick Astrego about 4 years ago. Updated about 1 month ago.

Status:
Closed
Priority:
Low
Category:
Unattended installations
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

http://mirror.centos.org/centos/$major.$minor/os/$arch, doesn't work for Centos 7 (or 6, or 5)

D, [2014-08-01T14:41:13.820310 #3745] DEBUG -- : Starting task (total: 0): /usr/bin/wget --timeout=10 --tries=3 --no-check-certificate -nv -c "http://mirror.centos.org/centos/7./os/x86_64/images/pxeboot/vmlinuz" -O "/var/lib/tftpboot/boot/Centos7-7-x86_64-vmlinuz" 
D, [2014-08-01T14:41:13.914386 #3745] DEBUG -- : Starting task (total: 0): /usr/bin/wget --timeout=10 --tries=3 --no-check-certificate -nv -c "http://mirror.centos.org/centos/7./os/x86_64/images/pxeboot/initrd.img" -O "/var/lib/tftpboot/boot/Centos7-7-x86_64-initrd.img" 
E, [2014-08-01T14:41:14.053902 #3745] ERROR -- : [4851] http://mirror.centos.org/centos/7./os/x86_64/images/pxeboot/vmlinuz:

E, [2014-08-01T14:41:14.054012 #3745] ERROR -- : [4851] 2014-08-01 14:41:14 ERROR 404: Not Found.

E, [2014-08-01T14:41:14.066278 #3745] ERROR -- : [5334] http://mirror.centos.org/centos/7./os/x86_64/images/pxeboot/initrd.img:

E, [2014-08-01T14:41:14.066342 #3745] ERROR -- : [5334] 2014-08-01 14:41:14 ERROR 404: Not Found.

So created a new Centos 7 mirror with:
http://mirror.centos.org/centos/$major/os/$arch

But there must be some cleaner way.


Related issues

Related to Foreman - Bug #10841: Foreman adds Operating systems no matter whatResolved2015-06-16
Related to Foreman - Refactor #11019: Operatingsystem model limitationsNew2015-07-06
Related to Foreman - Bug #13104: CentOS minor version numbering issue - preventing buildClosed2016-01-10

Associated revisions

Revision 7560679b (diff)
Added by Daniel Lobato Garcia over 3 years ago

Fixes #6884 - Installation media without $minor generate broken $version

$version is generated with the major version and a dot, producing paths
like foo.bar/x64/7. which don't exist. For CentOS at least we want to generate
a path without the dot. The default values for these media have been changed too.

Revision 33d3ee13 (diff)
Added by Daniel Lobato Garcia over 3 years ago

Fixes #6884 - Installation media without $minor generate broken $version

$version is generated with the major version and a dot, producing paths
like foo.bar/x64/7. which don't exist. For CentOS at least we want to generate
a path without the dot. The default values for these media have been changed too.

(cherry picked from commit 7560679b9849b0fc5c168d5047bd3469d719b45a)

Conflicts:
db/seeds.d/10-installation_media.rb

History

#1 Updated by Dominic Cleal about 4 years ago

  • Description updated (diff)
  • Category set to Unattended installations

#2 Updated by Marek Hulán almost 4 years ago

Confirming, for some reason they don't have 7.0/ directory but either 7 or 7.0.1406 (see http://mirror.centos.org/centos/)

#3 Updated by Ronald van Zantvoort almost 4 years ago

Wouldn't it be nicer to fill $release with "#{major}#{('.' + minor) unless minor.empty?}" (which is already in the code anyway, just disabled for RedHat family)?

#4 Updated by Pasi Karkkainen over 3 years ago

Ronald van Zantvoort wrote:

Wouldn't it be nicer to fill $release with "#{major}#{('.' + minor) unless minor.empty?}" (which is already in the code anyway, just disabled for RedHat family)?

Yes, indeed.

This bug is still a problem.. in a fresh install of Foreman 1.7.0 you still have the broken CentOS 7.0 install media url specified as a default.

#5 Updated by The Foreman Bot over 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2107 added
  • Pull request deleted ()

#6 Updated by Daniel Lobato Garcia over 3 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#7 Updated by Dominic Cleal over 3 years ago

  • Assignee set to Daniel Lobato Garcia
  • Legacy Backlogs Release (now unused) set to 28

#8 Updated by Mathieu Parent over 3 years ago

Couldn't this be fixed in 1.7 too?

#9 Updated by Dominic Cleal over 3 years ago

  • Legacy Backlogs Release (now unused) changed from 28 to 32

#10 Updated by Marek Hulán about 3 years ago

  • Related to Bug #10841: Foreman adds Operating systems no matter what added

#11 Updated by Dominic Cleal about 3 years ago

#12 Updated by Michael Deffenbaugh over 2 years ago

  • Related to Bug #13104: CentOS minor version numbering issue - preventing build added

Also available in: Atom PDF