Bug 816763 - Better default config file
Summary: Better default config file
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: OKD
Classification: Red Hat
Component: oc
Version: 2.x
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
: ---
Assignee: Fotios Lindiakos
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-04-26 22:09 UTC by Michael S.
Modified: 2015-05-15 01:52 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-02-05 18:44:01 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Michael S. 2012-04-26 22:09:12 UTC
While trying openshift today for the first time ( now that I figured that I can use it without dwelling into EC2 ), I first logged on the web console, created a domain, and then ran the command line.

However, the default config file didn't specify any option , so I had to dig in the source code to just found how to set the default user.

Could either :
- a commented config file be shipped as documentation somehwere ?
or 
- a commented config file be used as default ( with stuff like : 
  # default_rhlogin = username
so we can guess the parameter name ( cause neither "rhlogin", "login", or "username" where working 

I think some command line wizard to create the config file could also be a option  ( like git config set, or something similar ).

Comment 1 Fotios Lindiakos 2012-04-27 19:21:48 UTC
Fixed in https://github.com/openshift/os-client-tools/pull/9

Might wait to pull the change until next sprint so it can get adequate testing

Comment 2 Fotios Lindiakos 2012-04-27 21:33:47 UTC
Scratch that, I made some better changes in this commit: https://github.com/openshift/os-client-tools/pull/10

Comment 3 bzhao 2012-05-03 06:30:30 UTC
As Michael said a commented config file be used as default ( with stuff like : 
  # default_rhlogin = username

But it hasn't still including  "# default_rhlogin = username" in express.conf,

See the contents of express.conf
  1 # Remote API server
  2 libra_server = 'openshift.redhat.com'
  3 
  4 # Logging
  5 debug = 'false'
  6 
  7 # Timeout
  8 #timeout = '10

Comment 4 Fotios Lindiakos 2012-05-31 00:53:54 UTC
Still working on this bug, I am making a US in our sprint tracker to make sure it get's adequate testing in an upcoming sprint. Sorry this has taken so long.

Comment 5 Clayton Coleman 2012-06-21 18:18:45 UTC
Did J5 merge bits of this, or is it still outstanding?

Comment 6 John (J5) Palmieri 2012-10-09 14:06:03 UTC
Can we close this?

Comment 7 Clayton Coleman 2013-02-05 18:44:01 UTC
Fixed as part of an earlier story


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