Take II: [master/rhel7-branch] Auto-load ks.cfg if OEMDRV volume available. (#1057271)

Adam Williamson awilliam at redhat.com
Tue Jul 28 22:30:48 UTC 2015


On Wed, 2015-07-22 at 15:36 -0400, Samantha N. Bueno wrote:
> This checks if there are any OEMDRV labelled storage volumes 
> available--
> if yes, load a ks.cfg file from that.
> 
> This should only happen if a user has not manually specified a ks=
> option themselves.
> 
> Additionally, this checks for a boot arg inst.ks=none, to explicitly
> disable autoloading of ks.cfg from any detected OEMDRV volumes.

I think there's an error in this, possibly when there is no 'ks=' or
'inst.ks=' at all; when you boot with an updates.img at present there
seems to be a delay to the boot process, and you can see this error
while it's stuck:

//lib/dracut/hooks/pre-trigger/50-kickstart-genrules.sh: line 34: [:
too many arguments

which is the test that was added by this commit. There aren't any
obvious consequences, but it still looks like a bug.

The error does still pop up, I think, when you boot without
inst.updates - it's just not visible for very long, so easy to miss. I
don't know what the cause of the boot delay when booting with
inst.updates is, it just seems to sit there doing nothing for a while
before deciding to go ahead and download the updates image.
-- 
Adam Williamson
Fedora QA Community Monkey
IRC: adamw | Twitter: AdamW_Fedora | XMPP: adamw AT happyassassin . net
http://www.happyassassin.net



More information about the anaconda-patches mailing list