Bug 441837 - pykickstart - option --bootproto: invalid choice: 'query' (choose from 'dhcp', 'bootp', 'static')
pykickstart - option --bootproto: invalid choice: 'query' (choose from 'dhcp'...
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: pykickstart (Show other bugs)
All Linux
low Severity medium
: rc
: ---
Assigned To: Chris Lumens
Alexander Todorov
Depends On:
Blocks: 401531
  Show dependency treegraph
Reported: 2008-04-10 10:26 EDT by James Laska
Modified: 2013-09-02 02:24 EDT (History)
3 users (show)

See Also:
Fixed In Version: RHBA-2008-0459
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-05-21 10:30:13 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
screen capture of the error mentioned in comment #0 (25.40 KB, image/png)
2008-04-10 10:51 EDT, Alexander Todorov
no flags Details

  None (edit)
Description James Laska 2008-04-10 10:26:32 EDT
Description of problem:

In testing bug#401531 ... it appears that the new "network --bootproto=query"
syntax is not recognized as valid by pykickstart.

The installation will stop and prompt to reboot in stage#2 as seen below: 


Version-Release number of selected component (if applicable):
anaconda- (RHEL5.2-Server-20080409.nightly)

How reproducible:

Steps to Reproduce:
1. Build iso file with a kickstart on the iso that contains:

network --bootproto=query

2. Boot iso image with: linux ks=cdrom
3. Answer any loader questions to proceed to stage#2
Actual results:
pykickstart doesn't like kickstart syntax

Dialog stating ...
option --bootproto: invalid choice: 'query' (choose from 'dhcp', 'bootp', 'static')


Expected results:

pykickstart should be happy
Comment 1 James Laska 2008-04-10 10:37:40 EDT
This blocks end-2-end verification of bug#401531.

While this is late in the cycle, if we desire to support the hotfix pushed out
against 5.1 (officially fixed in #401531), I think we'll need to take this in.
Comment 2 Alexander Todorov 2008-04-10 10:51:00 EDT
Created attachment 302001 [details]
screen capture of the error mentioned in comment #0
Comment 3 Chris Lumens 2008-04-10 10:53:58 EDT
This patch should fix it:

diff --git a/pykickstart/parser.py b/pykickstart/parser.py
index a37baa0..c4a0e82 100644
--- a/pykickstart/parser.py
+++ b/pykickstart/parser.py
@@ -581,7 +581,7 @@ class KickstartHandlers:
     def doNetwork(self, args):
         op = KSOptionParser(lineno=self.lineno)
         op.add_option("--bootproto", dest="bootProto", default="dhcp",
-                      choices=["dhcp", "bootp", "static"])
+                      choices=["dhcp", "bootp", "static", "query"])
         op.add_option("--class", dest="dhcpclass")
         op.add_option("--device", dest="device")
         op.add_option("--essid", dest="essid")
Comment 7 James Laska 2008-04-15 15:53:14 EDT
VERIFIED proposed patch in updates image
(http://devserv.devel.redhat.com/~jlaska/updates-441837.img (also includes fix
for bug#442020)).
Comment 8 Chris Lumens 2008-04-15 18:06:28 EDT
This will be fixed in pykickstart-0.43-1.
Comment 12 errata-xmlrpc 2008-05-21 10:30:13 EDT
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.


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