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

ccl/jwtauthccl: allow tenants to set jwt auth settings #92406

Conversation

kpatron-cockroachlabs
Copy link
Contributor

Previously, tenants were prevented from modifying the cluster settings that controlled jwt based auth. This prevented these customers from using external signers and made it more complicated for us to support CC
serverless cluster SSO.

This change makes it so that tenants can configure their JWT auth cluster settings. This paves the way for full support of cluster SSO in severless.

Release note (enterprise change): JWT authentication cluster settings can now be modified from within
tenants to better support serverless use cases.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor

@knz knz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great thanks

@kpatron-cockroachlabs
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Nov 28, 2022

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Nov 28, 2022

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Nov 28, 2022

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Nov 29, 2022

Build failed:

@kpatron-cockroachlabs
Copy link
Contributor Author

bors r-

@kpatron-cockroachlabs kpatron-cockroachlabs force-pushed the kpatron/tenant-writable-jwt-settings branch 2 times, most recently from c76f2be to ff83994 Compare November 29, 2022 17:16
Previously, tenants were prevented from modifying the
cluster settings that controlled jwt based auth. This
prevented these customers from using external signers
and made it more complicated for us to support CC
serverless cluster SSO.

This change makes it so that tenants can configure their
JWT auth cluster settings. This paves the way for full
support of cluster SSO in severless.

Fixes CC-8608.

Release note (enterprise change): JWT authentication
cluster settings can now be modified from within
tenants to better support serverless use cases.
@kpatron-cockroachlabs
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Nov 30, 2022

Build succeeded:

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

Successfully merging this pull request may close these issues.

None yet

3 participants