[rhel7/f20/master 2/2] Updates to boot-options.txt document (#831512)

Anne Mulhern amulhern at redhat.com
Mon Nov 4 14:13:27 UTC 2013


----- Original Message -----
> From: "Vratislav Podzimek" <vpodzime at redhat.com>
> To: "anaconda patch review" <anaconda-patches at lists.fedorahosted.org>
> Sent: Monday, November 4, 2013 1:44:14 AM
> Subject: Re: [rhel7/f20/master 2/2] Updates to boot-options.txt document	(#831512)
> 
> On Fri, 2013-11-01 at 14:55 -0400, mulhern wrote:
> > Related: rhbz#831512
> > 
> > The goal is to have the information in this document placed in the
> > Installation Guide instead so that this document will eventually be
> > deleted. Currently it is serving as a reference.
> What about the wiki page we have? I'm pretty sure people are used to
> getting information from there. Could we somehow generate the wiki page
> from this asciidoc document by using pandoc or sowething like that?

I think the plan is to encourage them to go to the official docs a little more
for official user information.

The wiki should also be cleaned up just a bit and Will can eventually take down
the more obsolete document that he has posted. But that's a little in the future.


> >  
> >  [[kickstart]]
> >  Kickstart
> >  ---------
> >  
> >  === inst.ks ===
> > -Give the location of a kickstart file to be used to automate the install.
> > Uses
> > -the same formats as `inst.repo`.
> > +Give the location of a kickstart file to be used to automate the install.
> > +Locations may be specified using any of the formats allowed for
> > +<<inst.repo,`inst.repo`>>.
> If this is true, I'm really surprised we support getting kickstart from
> an NFS mount. Could you please check it?

I don't know enough about this to be surprised ;> But it definitely defaults
to nfs:auto specification if no value is given.
  
> > +[[inst.text]]
> >  === inst.text ===
> > -Force the installer to use a very limited text-based UI. Unless you're
> > using a
> > -kickstart this probably isn't a good idea; you should use VNC instead.
> > +Run the installer using a very limited text-based UI. Unless you're using
> > a
> > +kickstart file this probably isn't a good idea; you should use VNC
> > instead.
> I believe what we now have in the text-based UI is not really *very*
> limited. Could we drop that one word?

With pleasure!

> >  
> > +[[inst.updates]]
> >  === inst.updates ===
> >  Give the location of an `updates.img` to be applied to the installer
> >  runtime.
> > -Takes the same arguments as `inst.repo` and `inst.ks`.
> > +Locations may be specified using any of the formats allowed for
> > `inst.repo`.
> The same question/surprise as in the inst.ks case.

The same answer here...the option parsing code allows nfs.

> >  
> >  === inst.autostep ===
> >  Normally, if the information needed by a given installer screen has
> > @@ -313,30 +315,28 @@ This is mostly used with the `autoscreenshot`
> > kickstart command to get
> >  automated screenshots of the installer.
> I'm not sure we still support either of autostep and autoscreenshot
> options.

Ok, I think that's so. Gone!

> 
> >  
> >  === inst.debug ===
> > -Use `inst.debug=1` to add a "debug" button to the UI, which allows
> > dropping
> > -into a Python debugger.
> > +Use `inst.debug=1` to allow dropping into a Python debugger via an
> > +additional button in the GUI.
> This is no longer valid.

Gone!

> 
> >  
> >  === inst.traceback ===
> >  Intentionally raise an exception to make the installer crash with a
> >  traceback.
> > -This is for testing our crash-handling code. _Don't use it unless you want
> > the
> > -installer to crash._
> This option as well as the "functionality" was dropped.

Gone!

> 
> Other than those few comments this patch and the other one both look
> good to me.
> 
> --
> Vratislav Podzimek
> 
> Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic
> 
> _______________________________________________
> anaconda-patches mailing list
> anaconda-patches at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches
> 


More information about the anaconda-patches mailing list