Bug 694440 - KeyError: 'disableSsh'
Summary: KeyError: 'disableSsh'
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: pykickstart
Version: 5.7
Hardware: Unspecified
OS: Unspecified
urgent
urgent
Target Milestone: beta
: ---
Assignee: Chris Lumens
QA Contact: Release Test Team
URL:
Whiteboard:
Depends On:
Blocks: rhel57rtt
TreeView+ depends on / blocked
 
Reported: 2011-04-07 11:20 UTC by Alexander Todorov
Modified: 2011-04-12 07:48 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-04-12 07:48:04 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Alexander Todorov 2011-04-07 11:20:42 UTC
Description of problem:
Traceback when starting anaconda:

  File "/usr/bin/anaconda", line 942, in ?
    instClass.setInstallData(anaconda)
  File "/usr/lib/anaconda/kickstart.py", line 960, in setInstallData
    self.ksparser.readKickstart(self.file)
  File "/usr/lib/python2.4/site-packages/pykickstart/parser.py", line 1209, in readKickstart
    self.handleCommand(lineno, args)
  File "/usr/lib/anaconda/kickstart.py", line 848, in handleCommand
    self.handler.handlers[cmd](cmdArgs)
  File "/usr/lib/anaconda/kickstart.py", line 200, in doFirewall
    dict["ports"], disableSsh=dict["disableSsh"])
KeyError: 'disableSsh'


Version-Release number of selected component (if applicable):
anaconda 11.1.2.229

How reproducible:
always

Steps to Reproduce:
1. Run automated install in Beaker (need updates.img for bug #693655)
2.
3.
  
Actual results:
traceback

Expected results:
install completes


Additional info:

Comment 1 Ales Kozumplik 2011-04-07 12:09:02 UTC
Chris I think this just needs a new kickstart build following commit 8b2008a5a540dd2b1508a57033043ff353695dc0.

Comment 2 Chris Lumens 2011-04-07 14:13:39 UTC
Yeah you need pykickstart-0.43.9 which I just finished building and will be working on the errata shortly.  I don't know how you want to handle this bug, then, but I'm not inclined to put it on the errata since there's really nothing to be done code-wise.

Comment 3 Ales Kozumplik 2011-04-07 14:26:15 UTC
I agree, no need for this in the errata.

Comment 4 Chris Lumens 2011-04-07 14:52:49 UTC
Filed:  https://errata.devel.redhat.com/errata/stateview/11077

Comment 5 Ludek Smid 2011-04-12 02:17:26 UTC
Alex, can you CLOSE CURRENTRELEASE this bug once it's verified?


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