Bug 988662

Summary: [origin_runtime_192]The help messages of oo-user-var-add/list/remove are messy
Product: OpenShift Online Reporter: chunchen <chunchen>
Component: ContainersAssignee: Michal Fojtik <mfojtik>
Status: CLOSED WONTFIX QA Contact: libra bugs <libra-bugs>
Severity: low Docs Contact:
Priority: medium    
Version: 2.xCC: jhonce, mfojtik, wsun, xtian
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: 2013-09-10 15:29:05 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 chunchen 2013-07-26 05:50:24 UTC
Description of problem:
When run oo-user-var-add/list/remove -h/--help, will show "invalid option -- h" or "unrecognized option `--help'",and duplicated help messages for oo-user-var-list/remove.

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

How reproducible:
always

Steps to Reproduce:
1. SSH into the instance
2. Show help messages of oo-user-var-add
oo-user-var-add -h
oo-user-var-add --help
3. Show help messages of oo-user-var-list
oo-user-var-list -h
oo-user-var-list --help
4. Show help messages of oo-user-var-remove
oo-user-var-remove -h
oo-user-var-remove --help

Actual results:
At step 2:
/usr/bin/oo-user-var-add: unrecognized option -- h     <<<<<<<<<<<<<<
== Synopsis

/usr/bin/oo-user-var-add: Adds a set of user environment variables to the app.
  This command must be run as root.

== Usage

/usr/bin/oo-user-var-add  --with-app-uuid APP_UUID \
                --with-container-uuid UUID \
                [--with-gears GEAR_DNS;GEAR_DNS]
                "KEY0=VALUE0" "KEY1=VALUE1"

Options:
-h|--help:
   Prints this message

UUID: Unique identifier for the application
GEAR_DNS: secondary gears to be updated after adding user variables
KEY VALUE pairs

At step 3:
/usr/bin/oo-user-var-list: invalid option -- h  <<<<<<<<<<<<<<<
== Synopsis

/usr/bin/oo-user-var-list: List the user environment variables from the app.
  This command must be run as root.

== Usage

/usr/bin/oo-user-var-list  --with-app-uuid APP_UUID \
                --with-container-uuid UUID

Options:
-h|--help:
   Prints this message

UUID: Unique identifier for the application

== Synopsis

/usr/bin/oo-user-var-list: List the user environment variables from the app.
  This command must be run as root.

== Usage

/usr/bin/oo-user-var-list  --with-app-uuid APP_UUID \
                --with-container-uuid UUID

Options:
-h|--help:
   Prints this message

UUID: Unique identifier for the application

can't find user for    <<<<<<<<<<<<<<<

At step 4:
/usr/bin/oo-user-var-remove: invalid option -- h   <<<<<<<<<<<<<<
== Synopsis

/usr/bin/oo-user-var-remove: Remove a list of user environment variables from the app.
  This command must be run as root.

== Usage

/usr/bin/oo-user-var-remove  --with-app-uuid APP_UUID \
                --with-container-uuid UUID \
                [--with-gears GEAR_DNS;GEAR_DNS]
                KEY0 KEY1

Options:
-h|--help:
   Prints this message

UUID: Unique identifier for the application
GEAR_DNS: secondary gears to be updated after adding user variables
KEY VALUE pairs

== Synopsis

/usr/bin/oo-user-var-remove: Remove a list of user environment variables from the app.
  This command must be run as root.

== Usage

/usr/bin/oo-user-var-remove  --with-app-uuid APP_UUID \
                --with-container-uuid UUID \
                [--with-gears GEAR_DNS;GEAR_DNS]
                KEY0 KEY1

Options:
-h|--help:
   Prints this message

UUID: Unique identifier for the application
GEAR_DNS: secondary gears to be updated after adding user variables
KEY VALUE pairs

can't find user for <<<<<<<<<<<<<<<<<<<<

Expected results:
All commands should prompt friendly help messages.

Additional info:

Comment 1 Michal Fojtik 2013-08-20 13:51:13 UTC
https://github.com/openshift/origin-server/pull/3429

Comment 2 openshift-github-bot 2013-08-21 16:43:37 UTC
Commit pushed to master at https://github.com/openshift/origin-server

https://github.com/openshift/origin-server/commit/dca12faf49c8d0448edf8a43a55648f2fdfc0085
Bug 988662 - Add exit(255) to the usage() methods for oo-user-var scripts

Comment 3 chunchen 2013-08-26 01:40:45 UTC
I have checked this issue on devenv_3706, found it is not completely fixed. 
When these commands are executed with "-h/--help", will still meet "invalid option -- h/unrecognized option `--help'" in help messages.

Comment 4 Michal Fojtik 2013-08-26 14:35:21 UTC
Hmm it seems like the --help/-h was not included in the list of available options... Fixed that: https://github.com/openshift/origin-server/pull/3497

Comment 5 openshift-github-bot 2013-08-26 16:56:58 UTC
Commit pushed to master at https://github.com/openshift/origin-server

https://github.com/openshift/origin-server/commit/56ac6aba01a5da6aba1cac888f253a4e139cfe8c
Bug 988662 - Add --help to list of arguments for oo-env-var-* commands

Comment 6 chunchen 2013-08-27 02:30:26 UTC
The issue in comment 3 is reproduced, checked on devenv_3711.

Comment 7 Michal Fojtik 2013-09-10 12:14:31 UTC
IMHO the commands were removed in the latest devenv:

# oo-user-var-add -h
-bash: oo-user-var-add: command not found

# oo-user-var-list -h
-bash: oo-user-var-list: command not found

I suggest to close this as a NOT_A_BUG? Thoughts?

Comment 8 Jhon Honce 2013-09-10 15:29:05 UTC
These commands were removed and replaced with oo-devel-node