Skip to content
This repository has been archived by the owner on Jan 31, 2023. It is now read-only.

update readme to include kube-api oidc config #29

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tweeks024
Copy link

found this info in the video and thought I'd put it in the readme as well

@mlbiam
Copy link
Contributor

mlbiam commented Mar 29, 2019

I didn't put this in there because these steps are custom created for each deployment. For instance if your OU_HOST was for k8s.myhost.dev the first line would be - --oidc-issuer-url=https://k8s.myhost.dev/auth/idp/k8sIdp . I wanted to keep folks from copying and pasting directly from the README. wdyt about changing the "Complete SSO Integration with Kubernetes" to make it clearer?

@tweeks024
Copy link
Author

@mlbiam that makes much more sense, I updated the text under "Complete SSO Integration with Kubernetes" trying to keep it generic, let me know what you think

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

Successfully merging this pull request may close these issues.

2 participants