Bug 816763

Summary:	Better default config file
Product:	OKD	Reporter:	Michael S. <misc>
Component:	oc	Assignee:	Fotios Lindiakos <fotios>
Status:	CLOSED UPSTREAM	QA Contact:	libra bugs <libra-bugs>
Severity:	low	Docs Contact:
Priority:	medium
Version:	2.x	CC:	bzhao, ccoleman, fotios, jkeck, johnp, mpatel
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-02-05 18:44:01 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 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