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

Define roles to let users use application CRDs dynamically #3

Open
ca-scribner opened this issue Mar 4, 2022 · 0 comments
Open

Define roles to let users use application CRDs dynamically #3

ca-scribner opened this issue Mar 4, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@ca-scribner
Copy link
Contributor

We currently define the roles for different applications statically, where if we add an application we create new roles in this operator. This is irrespective of whether someone has actually deployed this application. We should change this to instead be dynamic and tailored to what has been deployed.

A concept for this could be through this charm providing a user-role relation (or something with a better name :) ). Applications could relate to this relation and provide ClusterRoles like these that define everything that should be included. This operator could then be in charge of maintaining those roles. This way if a Seldon is deployed and kfp is not, roles only exist for Seldon. And if a new Seldon release adds a new CRD, it is managed in what the Seldon charm provides to the user-role relation, rather than here.

@ca-scribner ca-scribner added the enhancement New feature or request label Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant