Bug 868435 - error message for invalid rhc app create --timeout should be more specific.
Summary: error message for invalid rhc app create --timeout should be more specific.
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: OKD
Classification: Red Hat
Component: oc
Version: 2.x
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: ---
Assignee: Hiro Asari
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-10-19 20:43 UTC by Peter Ruan
Modified: 2015-05-15 02:07 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-11-16 17:08:00 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Peter Ruan 2012-10-19 20:43:51 UTC
Description of problem:
error message for invalid rhc app create --timeout should be more specific.  In the previous version, if the user typed in the incorrect timeout value, the error message reads 'Timeout must be specified as a number greater than 0'.  Currently, the error message is very geneic. invalid argument: --timeout test


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


How reproducible:

always

Steps to Reproduce:
1. rhc app create testapp php-5.3 --timeout test
2.
3.
  
Actual results:
invalid argument: --timeout test


Expected results:
should give an error message that's more appropriate like 'time out value must be integer greater than 0'

Additional info:

Comment 1 Hiro Asari 2012-11-16 17:08:00 UTC
Right now, the output includes this bit.

===
Global Options
  -l, --rhlogin login       OpenShift login
  -p, --password password   OpenShift password
  -d, --debug               Turn on debugging
  --noprompt                Suppress the interactive setup wizard from running before a command
  --config FILE             Path of a different config file
  -h, --help                Display help documentation
  -v, --version             Display version information
  --timeout seconds         Set the timeout in seconds for network commands
===

Going beyond this will probably be too problematic, since the message "invalid argument: --timeout test" is ultimately coming from Ruby's standard library, and there is no easy way to customize the message. We might be able to *guess* what arguments might have been passed, but I suspect it will lead to some unmaintainable error checking.

I'm marking this as WONTFIX.


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