Bug 471658

Summary: anaconda loader should not bring up network if ks=hd:
Product: [Fedora] Fedora Reporter: Warren Togami <wtogami>
Component: anacondaAssignee: David Cantrell <dcantrell>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: 10CC: 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 Flags
vnc-and-non_remote_ks.patch
none
0001-Do-not-bring-up-network-for-non-remote-kickstart-loc.patch none

Description Warren Togami 2008-11-14 21:18:35 UTC
preupgrade wrote something like this into /boot/grub/grub.conf:

title Upgrade to Rawhide
        kernel /upgrade/vmlinuz preupgrade repo=hd:UUID=b9215874-6e70-4fc9-846b-a7d3c18d2572:/var/cache/yum/preupgrade stage2=hd:UUID=d74b467d-5b0b-47e3-beac-b883ccd7bf74:/upgrade/install.img ks=hd:UUID=d74b467d-5b0b-47e3-beac-b883ccd7bf74:/upgrade/ks.cfg
        initrd /upgrade/initrd.img

anaconda during the loader demands that I choose an interface to bring up, either eth0 or wlan0, despite anaconda not needing networking.  So perhaps it shouldn't ask to choose an interface if ks=hd:?

preupgrade could still require networking if there was insufficient space to write /boot/upgrade/install.img.  It would have stage2=http:// instead of stage2=hd:  So make sure this and other legitimate cases where it needs networking do not break.

Comment 1 Will Woods 2008-11-14 22:41:43 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.

Comment 2 David Cantrell 2008-11-14 23:14:32 UTC
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.

Comment 3 David Cantrell 2008-11-15 00:56:55 UTC
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?

Comment 4 Chris Lumens 2008-11-17 20:45:36 UTC
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.

Comment 5 David Cantrell 2008-11-17 20:52:23 UTC
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.

Comment 6 David Cantrell 2008-11-17 22:12:14 UTC
Patch in anaconda-11.4.1.59-1.

Comment 7 Bug Zapper 2008-11-26 05:24:04 UTC
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