Bug 457583

Summary: modifying default repo to specify different URL fails
Product: [Fedora] Fedora Reporter: Bill Nottingham <notting>
Component: anacondaAssignee: Jeremy Katz <katzj>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: low    
Version: rawhideCC: atodorov, benedictus, clumens, dcantrell, jlaska, rvokal
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard: NEEDSRETESTING
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-10-15 16:01:07 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 438943    
Attachments:
Description Flags
patch to simplify repo handling
none
traceback!!
none
set of patches to fix none

Description Bill Nottingham 2008-08-01 16:18:23 UTC
Description of problem:

I boot the F10 Alpha installer. It's by default pointed at a HTTP mirrorlist.

I edit the repo, change it to
http://reducto.install.bos.redhat.com/pungi/f10-alpha.7/10-Alpha/x86_64/os/

It accepts it. If I try to continue the install, it fails downloading the metadata.

Version-Release number of selected component (if applicable):

F10-Alpha.7

How reproducible:

100%

Additional info:

If I try to re-edit, the same mirrorlist is the starting point, not whatever I
edited it to before.

Comment 1 Chris Lumens 2008-08-29 17:44:11 UTC
I've made a good bit of progress on this - it now takes the new URL, creates a new repo with that URL to take the place of the old one, and then attempts to download the new repodata.  I then hit a traceback due to problems with the primarydb being empty.

Attaching a patch against rawhide that simplifies a lot of the repo handling (worked on with input from Seth) and the traceback I'm getting.  Also reassigning to Jeremy, who might have a clue how to fix this.

Comment 2 Chris Lumens 2008-08-29 17:45:33 UTC
Created attachment 315382 [details]
patch to simplify repo handling

Comment 3 Chris Lumens 2008-08-29 17:46:12 UTC
Created attachment 315383 [details]
traceback!!

Comment 4 Chris Lumens 2008-08-29 17:59:46 UTC
*** Bug 458998 has been marked as a duplicate of this bug. ***

Comment 5 Jeremy Katz 2008-09-11 19:49:36 UTC
Created attachment 316478 [details]
set of patches to fix

Here's a set of patches which looks like it fixes things, although I want to hold off until post-beta to avoid destabilizing things

Basically, we move actual package selection later so that we don't have packages in the tsInfo referencing a repo which has gone away.

Comment 6 Jeremy Katz 2008-09-29 14:24:25 UTC
And pushed

Comment 7 Jesse Keating 2008-10-03 23:39:52 UTC
Marking for re-testing.

Comment 8 Alexander Todorov 2008-10-28 07:40:12 UTC
*** Bug 468656 has been marked as a duplicate of this bug. ***

Comment 9 Andy Lindeberg 2009-06-08 19:13:08 UTC
*** Bug 494869 has been marked as a duplicate of this bug. ***