Skip to content

Improve CSM Authorization v2.x concepts and configuration docs for v1.15 and v1.16#1759

Merged
prablr79 merged 11 commits intorelease-1.16.2from
usr/mg/pmax-auth-fix
Feb 25, 2026
Merged

Improve CSM Authorization v2.x concepts and configuration docs for v1.15 and v1.16#1759
prablr79 merged 11 commits intorelease-1.16.2from
usr/mg/pmax-auth-fix

Conversation

@mgandharva
Copy link
Copy Markdown
Collaborator

@mgandharva mgandharva commented Feb 19, 2026

Description

Improve CSM Authorization v2.x concepts and configuration documentation: clarify token lifecycle, secrets reference, certificate validation settings, ingress exposure for Kubernetes/OpenShift, and remove redundant sections across all versioned docs.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #

Checklist:

  • Have you run a grammar and spell checks against your submission?
  • Have you tested the changes locally?
  • Have you tested whether the hyperlinks are working properly?
  • Did you add the examples wherever applicable?
  • Have you added high-resolution images?

Updated image

image (22)

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 19, 2026

Test Results

79 tests   79 ✅  4s ⏱️
 3 suites   0 💤
 1 files     0 ❌

Results for commit 5874ee7.

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Contributor

@rajkumar-palani rajkumar-palani left a comment

Choose a reason for hiding this comment

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

One minor comment related to "karavi-config-secret".

@mgandharva mgandharva marked this pull request as ready for review February 20, 2026 14:50
@mgandharva mgandharva requested a review from prablr79 as a code owner February 20, 2026 14:50
@mgandharva mgandharva changed the title Auth installation Fix Improve CSM Authorization v2.x concepts and configuration docs for v1.15 and v1.16 Feb 20, 2026
Copy link
Copy Markdown
Contributor

@prablr79 prablr79 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@alikdell alikdell left a comment

Choose a reason for hiding this comment

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

This also need go to main and notify Ray so that he incorporate the changes for future doc or have this documented in new doc repo, either way this information need to go to Ray so that this changes are part of future doc releases

@prablr79 prablr79 merged commit 098729a into release-1.16.2 Feb 25, 2026
8 checks passed
@prablr79 prablr79 deleted the usr/mg/pmax-auth-fix branch February 25, 2026 05:55
@mgandharva
Copy link
Copy Markdown
Collaborator Author

This also need go to main and notify Ray so that he incorporate the changes for future doc or have this documented in new doc repo, either way this information need to go to Ray so that this changes are part of future doc releases

Sure, I'll put the PR for main

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.

6 participants