Skip to content

OSDOCS#16171: Docs for BYO OIDC GA #100805

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

OSDOCS#16171: Docs for BYO OIDC GA #100805

wants to merge 1 commit into from
+27 −6

Conversation

@bergerhoffer
Copy link
Contributor

@bergerhoffer bergerhoffer commented Oct 20, 2025 •
edited
Loading

Version(s):
4.20+

Issue:
https://issues.redhat.com/browse/OSDOCS-16171

Link to docs preview:
https://100805--ocpdocs-pr.netlify.app/openshift-enterprise/latest/authentication/external-auth.html

QE review:

  • QE has approved this change.

Additional information:
Do not merge until this goes out in a z-stream

Once this feature goes out in a z-stream, these related PRs should all be merged:

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 20, 2025
@bergerhoffer bergerhoffer added this to the Continuous Release milestone Oct 20, 2025
@bergerhoffer
Copy link
Contributor Author

/retest

1 similar comment
@bergerhoffer
Copy link
Contributor Author

/retest

@bergerhoffer bergerhoffer reopened this Oct 20, 2025
@bergerhoffer
Copy link
Contributor Author

/retest

1 similar comment
@bergerhoffer
Copy link
Contributor Author

/retest

@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Oct 21, 2025 •
edited
Loading

🤖 Tue Oct 28 13:16:56 - Prow CI generated the docs preview:

https://100805--ocpdocs-pr.netlify.app/openshift-enterprise/latest/authentication/external-auth.html

liouk
liouk reviewed Oct 28, 2025
View reviewed changes
modules/external-auth-providers.adoc
Comment on lines +11 to +18
* Active Directory Federation Services for Windows Server
* GitLab
* Google
* Keycloak
* Microsoft Entra ID
* Okta
* Ping Identity
* Red Hat Single Sign-On
Copy link
Member

Choose a reason for hiding this comment

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

This looks good to me, but @xingxingxia is probably better suited to review this list.

@liouk
Copy link
Member

liouk commented Oct 28, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 28, 2025
@bergerhoffer
Copy link
Contributor Author

@xingxingxia @wewang58 Can you please QE review?

Preview: https://100805--ocpdocs-pr.netlify.app/openshift-enterprise/latest/authentication/external-auth.html

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 28, 2025
@openshift-ci
Copy link

openshift-ci bot commented Oct 28, 2025

New changes are detected. LGTM label has been removed.

@openshift-ci
Copy link

openshift-ci bot commented Oct 28, 2025

@bergerhoffer: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@liouk liouk liouk left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@liouk liouk

Labels

branch/enterprise-4.20 branch/enterprise-4.21 size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

Continuous Release

Development

Successfully merging this pull request may close these issues.

3 participants

@bergerhoffer @ocpdocs-previewbot @liouk