Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No mention of password during oc cluster up operation #14832

Closed
coolbrg opened this issue Jun 22, 2017 · 6 comments
Closed

No mention of password during oc cluster up operation #14832

coolbrg opened this issue Jun 22, 2017 · 6 comments

Comments

@coolbrg
Copy link
Contributor

coolbrg commented Jun 22, 2017

Version
$ oc version
oc v3.6.0-alpha.1+46942ad
kubernetes v1.5.2+43a9be4
features: Basic-Auth GSSAPI Kerberos SPNEGO

Steps To Reproduce
  1. Run oc cluster up
Current Result
Starting OpenShift using openshift/origin:v3.6.0-alpha.1 ...
...

The server is accessible via web console at:
    https://127.0.0.1:8443

You are logged in as:
    User:     developer

To login as administrator:
    oc login -u system:admin
Expected Result
Starting OpenShift using openshift/origin:v3.6.0-alpha.1 ...
...

The server is accessible via web console at:
    https://127.0.0.1:8443

You are logged in as:
    User:     developer
    Password: developer

To login as administrator:
    oc login -u system:admin

Curious to know why we are dropping Password here. How will the user know how to login to web console?

Is it due to the fact that password here is just a fake one as we can use anything in the password field?

@csrwng
Copy link
Contributor

csrwng commented Jun 23, 2017

Is it due to the fact that password here is just a fake one as we can use anything in the password field?

Yes, you can use anything. Displaying one is misleading.

@coolbrg
Copy link
Contributor Author

coolbrg commented Jun 23, 2017

Displaying one is misleading.

Showing nothing as well also might be confusing to new users (may be my opinion only).
Probably something like:

User:      developer
Password:  <anything> (unauthenticated)

will convey fast and better to user that he/she has to use anything for password which is unauthenticated.

@csrwng
Copy link
Contributor

csrwng commented Jun 23, 2017

@budhrg agree. I'll submit a pr with a fix.

@coolbrg
Copy link
Contributor Author

coolbrg commented Jun 23, 2017

@csrwng or should I try 😄

@csrwng
Copy link
Contributor

csrwng commented Jun 23, 2017

definitely 👍

@coolbrg
Copy link
Contributor Author

coolbrg commented Jun 23, 2017

Added fix @csrwng

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants