-
Notifications
You must be signed in to change notification settings - Fork 160
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
Get back to basics with the Admin User #1787
Comments
This will likely need updates to the admin-user.yaml helm chart (this may also be impacted by #1789 as namespace and name of the resources created in k8s should probably be configurable NB this doesn't mean the username has to be, just that the resources being injected into potentially congested namepaces should be controllable) |
@AlinaGoaga Does pesto want this one or shall we take it? |
Hey @Callisto13! I am not sure actually last time we spoke with David we decided it was going to stay as is I believe @davidstauffer. Let's catch up tomorrow, I think we want to keep the style changes and those related to checking for what's been set up in the cluster to update the interface options from 1624. Thanks :) (tagging the rest of the team also @foot @yiannistri) |
Hi, I think we need to separate the UX in the GUI from the mechanism that provides those identities. My takeaway from the mentioned conversation is that we concluded: |
I would note that it is likely that we will end up changing this UX as it is simply not good. Literally just now I had this conversation: person: "I set up my admin user, how do I create another user?" BUT I do see the value in making the admin username configurable, so hopefully it will work out |
Short description
As a user whose OIDC is f’d or cnba to set up OIDC right this second, I can log in as a hardcoded Admin user and get access to my stuff.
Context
This is basically unwinding some of the extensions that have been made to the AdminUser feature. We do not want it so that we end up supporting our own auth provider. We can handle a single Admin user for emergency purposes only. Otherwise we encourage users to have OIDC #1786.
The feature was introduced here #1490 and extended #1624. There is a further PR here #1755.
We basically want to get back to where we were after #1490.
See conversation here.
Acceptance criteria
As a user whose OIDC is f’d or cnba to set up OIDC right this second:
gitops.weave.works/entity: admin-user
admin
and is uneditable)The text was updated successfully, but these errors were encountered: