Bug 1028633

Summary: The utility oo-devel-node app-create shouldn't restrict the order
Product: OpenShift Online Reporter: Peter Ruan <pruan>
Component: ContainersAssignee: Dan McPherson <dmcphers>
Status: CLOSED CURRENTRELEASE QA Contact: libra bugs <libra-bugs>
Severity: low Docs Contact:
Priority: low    
Version: 2.xCC: chunchen, dmcphers
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-01-30 00:50:02 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:

Description Peter Ruan 2013-11-08 22:28:11 UTC
Description of problem:
  The order of the argument matters when running oo-devel-node app-create.  Why should it?

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

How reproducible:
always.

Steps to Reproduce:
1. oo-devel-node app-create --with-namespace case247269 --with-app-name case247269 --with-app-uuid 999999999999999999999999 --with-container-uuid 999999999999999999999999
2.
3.

Actual results:
[0mMissing --with-container-uuid UUID

Usage: oo-devel-node app-create --with-container-uuid UUID --with-namespace NAMESPACE \
--with-app-uuid UUID --with-app-name NAME --with-secret-token TOKEN \
[--with-uid UID] [--with-container-name NAME] \
[--with-quota-blocks BLOCKS] [--with-quota-files FILES] 

Expected results:
order of the argument shouldn't matter.

Additional info:

Comment 1 Dan McPherson 2014-01-20 17:29:28 UTC
https://github.com/openshift/origin-server/pull/4524

This seems to be the default behavior of commander and the use case isn't interesting enough to work around.  I added a better error message but not going to make another change to fix fully.

Comment 3 chunchen 2014-01-21 03:11:04 UTC
It's fixed, verified on devenv_4247, please refer to the following results:

# oo-devel-node app-create --with-namespace case247269 --with-app-name case247269 --with-app-uuid 999999999999999999999999 --with-container-uuid 999999999999999999999999

Missing --with-container-uuid UUID as the first option

Usage: oo-devel-node app-create --with-container-uuid UUID --with-namespace NAMESPACE \
     --with-app-uuid UUID --with-app-name NAME --with-secret-token TOKEN \
     [--with-uid UID] [--with-container-name NAME] \
     [--with-quota-blocks BLOCKS] [--with-quota-files FILES]