-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[Feature Request] API for dynamic connector registration #1472
Comments
It seems that, dex already have defined CRD for k8s storage, and the storage interface also has defined the CreateConnector method. |
There is a PR open for this : #1489 |
@sks looking forward your pr being merged 👍 |
@sks any update on your pr ?;-) |
@piotrmsc : waiting for green signal from any of the maintainers. Ping @srenatus / @JoelSpeed / @bonifaido : #1489 |
Description
It would be really nice to have an option to dynamically register the connector(s) via dex API.
Dex API currently allows dynamic client registration as documented here and perhaps could be extended.
My initial thought was to use CRD for that which was mentioned in a bit general topic in #1381 . Taking into account the results of the conversation there, maybe gRPC API for connector could be the first approach?
Reasons
To configure a connector you have to set it up upfront during installation, but later on, if you like to add/replace a new one, you have to edit the configmap and restart dex pod (k8s scenario).
This process could be improved with having an API so dex restart would not be required.
Restarting dex is not the best way as well. We have a use case where admins would like to configure new connectors easily (kyma-project/kyma#4537)
Additional note, maybe file watch for config file could be also nice small improvement?
The text was updated successfully, but these errors were encountered: