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

Improve wording to avoid implying pinniped is used either way. #5695

Merged
merged 1 commit into from
Nov 30, 2022

Conversation

absoludity
Copy link
Contributor

Signed-off-by: Michael Nelson [email protected]

Description of the change

While chatting about OIDC configuration with @RGPosadas I was looking through the docs and found that it reads currently as if you should use Pinniped either way.

This change just clears that up a bit, I hope, so that people use the cluster's own OIDC support when they can, and pinniped when they can't.

Benefits

Possible drawbacks

Applicable issues

Additional information

@netlify
Copy link

netlify bot commented Nov 22, 2022

Deploy Preview for kubeapps-dev canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit c17552b
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/6387ddf7540aa8000856639f

Copy link
Collaborator

@ppbaena ppbaena left a comment

Choose a reason for hiding this comment

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

Thanks for improving Kubeapps documentation!

@absoludity absoludity merged commit 082cc90 into vmware-tanzu:main Nov 30, 2022
@absoludity absoludity deleted the improve-oidc-docs branch November 30, 2022 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants