Bug 471658
Summary: | anaconda loader should not bring up network if ks=hd: | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Warren Togami <wtogami> | ||||||
Component: | anaconda | Assignee: | David Cantrell <dcantrell> | ||||||
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | medium | ||||||||
Version: | 10 | CC: | anaconda-maint-list, wwoods | ||||||
Target Milestone: | --- | ||||||||
Target Release: | --- | ||||||||
Hardware: | All | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2009-02-16 20:32:46 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: | 438944 | ||||||||
Attachments: |
|
Description
Warren Togami
2008-11-14 21:18:35 UTC
I assume that the upgrade would proceed as normal if you configured the interface anyway? Not really a blocker, if that's the case. I'm working on a patch right now. It's not a blocker, from my point of view. Like Will said, the installation will proceed once the interface is up, the user may just be left wondering why they had to configure the interface. If the patch is simple, I'll consider it for F-10. I'll post here once I have more to report. Thanks. Created attachment 323673 [details]
vnc-and-non_remote_ks.patch
I think this will fix the issue. Remove the redundant call to kickstartNetworkUp() in loader. If the user specifies vnc, that requires loader to configure networking because we can't present the network configuration UI until we have a display (catch 22 with vnc). So, if they specify vnc *and* the kickstart parameter is a non-network source, set the LOADER_FLAGS_EARLY_NETWORKING flag, which is what I did for the syslog= parameter to make sure we bring up networking in loader.
Chris, can you comment?
I made a couple comments elsewhere that I'll sum up here for completness. (1) The test in the first hunk for !strncmp(ksFile, "ks=", 3) will throw out the possibility of it just being "ks" which can happen under the NFS case. (2) Don't we want to break the tie between VNC and network case, and then change it to be isKickstartFileRemote(loaderData->ksFile) instead? With those modifications, it looks good to go ahead and commit. Created attachment 323796 [details]
0001-Do-not-bring-up-network-for-non-remote-kickstart-loc.patch
Updated patch. Sending to the devel list for review.
Patch in anaconda-11.4.1.59-1. This bug appears to have been reported against 'rawhide' during the Fedora 10 development cycle. Changing version to '10'. More information and reason for this action is here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping |