Skip to content
This repository has been archived by the owner on Jul 23, 2020. It is now read-only.

tenant service should abort if service privilege is unavailble #1059

Open
kbsingh opened this issue Oct 5, 2017 · 7 comments
Open

tenant service should abort if service privilege is unavailble #1059

kbsingh opened this issue Oct 5, 2017 · 7 comments

Comments

@kbsingh
Copy link
Collaborator

kbsingh commented Oct 5, 2017

In order to build up the tenant infrastructure on a target openshift cluster, we use a privileged account able to create and then edit role bindings for the and on behalf of the user. In testing, we've noticed that if the privilged account is not running with the required privilges, the tenant does not abort or report-failure.

This results in the false impression that the tenant infra is setup, but it isnt and checks against it fail externally, and internally neither jenkins nor che are able to run.

@jfchevrette
Copy link
Contributor

we should also consider situations where the privileged account token is valid, but some actions can't be executed for all sorts of reasons. ther should/could be appropriate feedback to indicate failure to initialize and to retry or contact support

@aslakknutsen
Copy link
Collaborator

If Token is missing or Invalid the WhoAmI call will fail and the service will fail to start. But yes, it doesn't verify the account auth beyond that.

@kbsingh
Copy link
Collaborator Author

kbsingh commented Oct 5, 2017

in this case, the account exists, and the token is valid. its just unable to do the edits needed to change limits / perms.

@joshuawilson
Copy link
Member

Is anyone working on this? Do we need to make someone else aware of it?

@aslakknutsen
Copy link
Collaborator

@joshuawilson It's brewing in the back of my head. Just not quite sure what to actually check for. Specially when we're starting to talk about Profiles as the templates in a large extent decide what auth the user needs. But we can certainly add a few basic know ones for now, e.g. create/edit RoleBindings/RoleBindingRestrictions/DeploymentConfigs/Secrets/ConfigMaps

@kbsingh
Copy link
Collaborator Author

kbsingh commented Nov 24, 2017

ping @xcoulon - as you are working through, thinking through some parts of this - maybe an easy win to get done, for just validating those few roles are working ?

@xyntrix
Copy link

xyntrix commented Dec 12, 2017

@xcoulon is there anything we can update wrt this item?

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

No branches or pull requests

5 participants