Bug 870467

Summary: spacecmd interactive mode exits if wrong argument is entered
Product: [Community] Spacewalk Reporter: David Juran <djuran>
Component: ServerAssignee: Aron Parsons <aronparsons>
Status: CLOSED CURRENTRELEASE QA Contact: Red Hat Satellite QA List <satqe-list>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.8CC: jpazdziora, mmello, msuchy
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: spacecmd-1.9.3-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-03-03 22:36:53 UTC Type: Bug
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: 917805    

Description David Juran 2012-10-26 14:53:59 UTC
Description of problem:
If you in spacecmd interactive mode type an invalid parameter, e.g. snippet_update -n foo, the interactvie mode exits

Version-Release number of selected component (if applicable):
spacecmd-1.8.15-1


Steps to Reproduce:
1. spacecmd
2. snippet_update -n foo

  
Actual results:

interactive mode exits and you get thrown out into your shell.

Comment 1 Miroslav Suchý 2012-10-29 13:53:47 UTC
Forwarding to upstream.

Comment 2 Michael Mráka 2012-10-31 11:02:52 UTC
Reassigning to spacecmd maintainer.

Comment 3 Steven Hardy 2013-02-21 11:49:18 UTC
Reassigning to Aron Parsons, spacecmd maintainer

Comment 4 Aron Parsons 2013-03-03 22:36:53 UTC
commit aa6d1c678db182185d0b7e0bd789719cd371eb79

This fixes the same issue throughout the whole application.  We needed to subclass OptionParser to change the error behavior.

Comment 5 Jan Pazdziora 2013-03-04 08:09:12 UTC
Aron, we prefer to set the state to MODIFIED when changes were done for the bugzilla, fill the Fixed In Version (I did that now) and leave it for the release nanny to close the bugzilla when the new Spacewalk release is released.