Bug 883191 - "This URL refers to a mirror" should read "This URL refers to a mirror list" (Installation Location spoke)
Summary: "This URL refers to a mirror" should read "This URL refers to a mirror list" ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: anaconda
Version: 18
Hardware: All
OS: All
unspecified
high
Target Milestone: ---
Assignee: Anaconda Maintenance Team
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F18Blocker, F18FinalBlocker
TreeView+ depends on / blocked
 
Reported: 2012-12-04 00:46 UTC by Adam Williamson
Modified: 2012-12-06 07:29 UTC (History)
8 users (show)

Fixed In Version: anaconda-18.35-1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-06 07:29:30 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Adam Williamson 2012-12-04 00:46:36 UTC
See https://bugzilla.redhat.com/show_bug.cgi?id=868558#c31 and down - the basic bug there is now fixed, you can use a mirror list URL, but the checkbox that was added is labelled wrongly: it reads 'This URL refers to a mirror', not 'This URL refers to a mirror list' as it should. Not only will people are entering mirror list URLs not check the box (as they should), but possibly people who are entering direct mirror URLs will check the box (as they shouldn't) - this bad label could break both cases.

Proposing as a blocker - it's only a UI bug, but it's a significant one, as it can cause everyone who enters a URL to do the wrong thing.

Comment 1 Fedora Update System 2012-12-05 00:57:14 UTC
pykickstart-1.99.22-1.fc18, anaconda-18.35-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/FEDORA-2012-19380/pykickstart-1.99.22-1.fc18,anaconda-18.35-1.fc18

Comment 2 Robyn Bergeron 2012-12-05 08:35:01 UTC
It's unclear to me from reading this ticket and 868558 as to whether or not things are labelled backwards (and thus, upon entering a mirror URL, clicking the button to indicate that it is a mirror, when the button is supposed to only be clicked for mirror lists), if the button should be clicked regardless of it being a mirrorlist or direct mirror URL (which I can see happening either way - clicking yes in both cases, or saying that a mirrorlist is not a mirror in not clicking).... or if it is a change from what was asked at this stage in F17 that is causing confusion.  (Since we previously had "this URL refers to a mirrorlist")

If it really and truly means "is this a mirror", and not mirror list, then that might be worthwhile documenting as a change from previously expected behavior.

If it is really being done it to differentiate between mirrors and mirror lists, and that alone, then it seems more logical (to me, and here is your grain of salt) to isolate the item that cannot possibly construed as one subset of the other. I can see people entering a mirror list URL and clicking the box, thinking, "it's sort of a glorified supermirror, right?" - but entering a direct mirror URL is much less likely to be confused with the more specific "This URL refers to a mirrorlist."

If people consider "my own personal repo" to be "a mirror" and we only want direct mirror URLs to get clicked then that brings in an extra universe of fun in wording it as "mirror" vs. "mirrorlist" in that way. I'm not sure that that's common thought though.  

But like I said - I'm unclear from bz comments as to if the buttons are actually implementing things incorrectly in certain cases, or if it's just confusing and likely to lead to disaster.  (And if that level of disaster continues to be as adamw described in comment 31 of 868558.)

Comment 3 Adam Williamson 2012-12-05 09:08:44 UTC
The box means what it meant before. You're meant to check it if the URL is a 'mirror list', not if the URL is a 'mirror'. It is currently directing you to do exactly the opposite of what you actually ought to do. If the URL you are entering is a direct mirror URL you should *not* check the box. If the URL you are entering is a mirror list URL you *should* check the box. The text 'This URL refers to a mirror' gives you precisely the wrong idea.

Comment 4 Robyn Bergeron 2012-12-05 10:04:14 UTC
+1 blocker - "The installer must be able to use all supported local and remote package source options" - it is not usable if requesting the wrong type of input.

Comment 5 Kamil Páral 2012-12-05 10:22:56 UTC
Fixed in anaconda 18.35.

Comment 6 Kamil Páral 2012-12-05 10:23:53 UTC
(I mean I verified it in anaconda 18.35 and it looks OK now)

Comment 7 Fedora Update System 2012-12-06 07:29:35 UTC
pykickstart-1.99.22-1.fc18, anaconda-18.35-1.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.