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

[Docs] Add instructions how to remove Che user #17500

Closed
RomanNikitenko opened this issue Jul 28, 2020 · 15 comments
Closed

[Docs] Add instructions how to remove Che user #17500

RomanNikitenko opened this issue Jul 28, 2020 · 15 comments
Labels
area/che-server area/doc Issues related to documentation kind/question Questions that haven't been identified as being feature requests or bugs. new&noteworthy For new and/or noteworthy issues that deserve a blog post, new docs, or emphasis in release notes severity/P1 Has a major impact to usage or development of the system.

Comments

@RomanNikitenko
Copy link
Member

RomanNikitenko commented Jul 28, 2020

Summary

Update: Please see #17500 (comment)

My question is: is it possible to remove an user on Che?

My motivation is: I can not start any workspace due to incorrect namespace, please see relevant info section.
I tried to remove user using swagger. After removing user doesn't exist, but after refreshing a page or at next login the system has the same user with the same ID.

Please see my short demo:

user_delete

Relevant information

I faced with the problem like described here: #17265 (comment)
So:

  • Che's Keycloak shows the form with username, email, etc. when a user is trying to login into Che Dashboard the very first time
  • autofill plugin sets the fields
  • as result username can get incorrect value (with real names, including spaces or email for example)

Then user can not create a workspace due to not a valid namespace:
The workspace would be started in a namespace/project '<incorrect value here>', which is not a valid namespace/project name.

Please see #17265 (comment).

@RomanNikitenko RomanNikitenko added the kind/question Questions that haven't been identified as being feature requests or bugs. label Jul 28, 2020
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Jul 28, 2020
@RomanNikitenko RomanNikitenko removed the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Jul 28, 2020
@sleshchenko sleshchenko added area/che-server area/install Issues related to installation, including offline/air gap and initial setup labels Jul 28, 2020
@sleshchenko
Copy link
Member

cc @skabashnyuk @tolusha Any thoughts if it's possible to fully unregister the user by themselves? Seems keycloak user should be removed as well, but is it possible?

@skabashnyuk
Copy link
Contributor

@sleshchenko I don't know. Probably keycloak admin can do that. Che-server API behaves correctly in this use-case as far as I can see.

@tolusha
Copy link
Contributor

tolusha commented Jul 29, 2020

It is necessary to remove user in keycloak.
Screenshot from 2020-07-29 13-57-19

@tolusha
Copy link
Contributor

tolusha commented Jul 29, 2020

@RomanNikitenko
Did I answer your question?

@RomanNikitenko
Copy link
Member Author

@tolusha
I guess it's possible for admin role only.

So for my case for https://che-dogfooding.apps.che-dev.x6e0.p1.openshiftapps.com/ instance I can not remove myself from keycloak.

correct?

@tolusha
Copy link
Contributor

tolusha commented Jul 29, 2020

yes, only admin can remove user from the keycloak.

@RomanNikitenko
Copy link
Member Author

@skabashnyuk @tolusha @sleshchenko
thank you very much for your help and support!

Now it's clear for me, so I'm closing the issue

@l0rd
Copy link
Contributor

l0rd commented Jul 29, 2020

@RomanNikitenko if it's not in documentation yet we should probably re-open this issue as a documentation issue. We need to add:

@l0rd l0rd added the new&noteworthy For new and/or noteworthy issues that deserve a blog post, new docs, or emphasis in release notes label Jul 29, 2020
@tolusha tolusha reopened this Jul 30, 2020
@tolusha tolusha added the area/doc Issues related to documentation label Jul 30, 2020
@RomanNikitenko RomanNikitenko changed the title How to remove Che user [Docs] Add instructions how to remove Che user Jul 31, 2020
@ibuziuk
Copy link
Member

ibuziuk commented Jul 31, 2020

@l0rd there was a request from the Toolchain team to make the Delete API call GDPR compliant and cleanup resources not only on the db level, but also on keycloak - #17541
I believe we need to treat it as a high priority issue ^

@l0rd
Copy link
Contributor

l0rd commented Jul 31, 2020

@ibuziuk priority now is documentation (let's include this issue into next sprint backlog) and then let's prioritize the API side (the issue you mentioned).

@skabashnyuk
Copy link
Contributor

@ibuziuk is there anything left on this task?

@tolusha tolusha removed the area/install Issues related to installation, including offline/air gap and initial setup label Jan 13, 2021
@yhontyk
Copy link

yhontyk commented Jan 28, 2021

Docs Jira issue https://issues.redhat.com/browse/RHDEVDOCS-2576

@themr0c
Copy link
Contributor

themr0c commented Mar 2, 2021

@yhontyk
Copy link

yhontyk commented Mar 23, 2021

@l0rd could you take a look if the current documentation covers the issue https://www.eclipse.org/che/docs/che-7/administration-guide/removing-user-data/ ? Can we close it?

@l0rd
Copy link
Contributor

l0rd commented Mar 24, 2021

@boczkowska yes. Closing.

@l0rd l0rd closed this as completed Mar 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/che-server area/doc Issues related to documentation kind/question Questions that haven't been identified as being feature requests or bugs. new&noteworthy For new and/or noteworthy issues that deserve a blog post, new docs, or emphasis in release notes severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

9 participants