Bug 877347

Summary: Return success even if giving invalid options for oo-admin-ctl-app -c force-destroytest
Product: OKD Reporter: Rony Gong 🔥 <qgong>
Component: PodAssignee: Dan McPherson <dmcphers>
Status: CLOSED CURRENTRELEASE QA Contact: libra bugs <libra-bugs>
Severity: low Docs Contact:
Priority: unspecified    
Version: 2.x   
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: devenv_2490 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-19 19:26:51 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:

Description Rony Gong 🔥 2012-11-16 10:21:29 UTC 
Description of problem:
command: oo-admin-ctl-app -c force-destroytest  -l qgong+1 -a qphp
will retrun success, should show fail for invalid option

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

How reproducible:
always

Steps to Reproduce:
1.Create app
2.Use invalid -c option "force-destroytest"  to this app 
oo-admin-ctl-app -c force-destroytest  -l qgong+1 -a qphp
3.
  
Actual results:
[root@ip-10-122-63-9 subsys]# oo-admin-ctl-app -c force-destroytest  -l qgong+1 -a qphp
Success

Expected results:
Should show fail for invalid option

Additional info:
Comment 1 Rony Gong 🔥 2012-11-19 02:45:17 UTC 
Verified on devenv_stage_254
[root@ip-10-202-193-65 openshift]# oo-admin-ctl-app -c forec-destroy  -l qgong -a jbosstest
Command must be one of: (start|stop|force-stop|restart|status|destroy|force-destroy|removegear)
== Synopsis

oo-admin-ctl-app: Control user applications

== Usage

oo-admin-ctl-app OPTIONS

Options:
-l|--rhlogin <rhlogin>
    Red Hat login (RHN or OpenShift login with OpenShift access) (required)
-a|--app     <application>
    Application name  (alphanumeric) (required)
-c|--command <command>
    (start|stop|force-stop|restart|status|destroy|force-destroy|removegear) (required)
-b|--bypass
    Ignore warnings
-h|--help
    Show Usage info