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

Chectl should print dex users credentials #20779

Closed
Tracked by #20839
sparkoo opened this issue Nov 16, 2021 · 2 comments
Closed
Tracked by #20839

Chectl should print dex users credentials #20779

sparkoo opened this issue Nov 16, 2021 · 2 comments
Labels
area/chectl Issues related to chectl, the CLI of Che kind/enhancement A feature request - must adhere to the feature request template. severity/P2 Has a minor but important impact to the usage or development of the system.
Milestone

Comments

@sparkoo
Copy link
Member

sparkoo commented Nov 16, 2021

Is your enhancement related to a problem? Please describe

Chectl can now deploy on minikube with Dex. There are predefined static users https://github.com/che-incubator/chectl/blob/main/resources/dex/configmap.yaml#L28 that can be used to login into Che. Also chectl still prints credentials to keycloak, which does not make sense in this setup.

Describe the solution you'd like

At the end of Che deployment, chectl should print the static users that user can use.
Those are: che@eclipse.org:admin and userN@che:password where N is 1-5.

Remove any other printed credentials to keycloak/idm.

Describe alternatives you've considered

No response

Additional context

No response

@sparkoo sparkoo added kind/enhancement A feature request - must adhere to the feature request template. area/chectl Issues related to chectl, the CLI of Che severity/P2 Has a minor but important impact to the usage or development of the system. labels Nov 16, 2021
@tolusha
Copy link
Contributor

tolusha commented Nov 17, 2021

@sparkoo
To be honest chectl doesn't know users' password in general.
Is it enough to print predefined credentials for admin user?

@sparkoo
Copy link
Member Author

sparkoo commented Nov 17, 2021

@tolusha chectl deploys dex configmap with predefined users https://github.com/che-incubator/chectl/blob/main/resources/dex/configmap.yaml#L28 so it kind-off knows the users credentials. admin user here is same as any other user, it does not have any special privileges. These static users are really meant only for testing. In real world, you would connect dex to github/google/... I will prepare doc for that as well and we should reference it in chectl as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/chectl Issues related to chectl, the CLI of Che kind/enhancement A feature request - must adhere to the feature request template. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Development

No branches or pull requests

2 participants