Skip to content

extend docs on cross workspace auth#3780

Merged
kcp-ci-bot merged 1 commit into
kcp-dev:mainfrom
SimonTheLeg:extend-cross-workspace-auth-docs
Feb 23, 2026
Merged

extend docs on cross workspace auth#3780
kcp-ci-bot merged 1 commit into
kcp-dev:mainfrom
SimonTheLeg:extend-cross-workspace-auth-docs

Conversation

@SimonTheLeg

@SimonTheLeg SimonTheLeg commented Jan 7, 2026

Copy link
Copy Markdown
Member

Summary

What Type of PR Is This?

/kind documentation

Related Issue(s)

Fixes #3623

Release Notes

NONE

@kcp-ci-bot kcp-ci-bot added release-note-none Denotes a PR that doesn't merit a release note. dco-signoff: yes Indicates the PR's author has signed the DCO. do-not-merge/needs-kind Indicates a PR lacks a `kind/foo` label and requires one. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jan 7, 2026
@SimonTheLeg

Copy link
Copy Markdown
Member Author

This is a first version to extend the docs, however I am unsure if this is everything you meant @ntnn
ptal

@SimonTheLeg SimonTheLeg requested a review from ntnn January 7, 2026 15:48
Comment thread docs/content/concepts/authorization/authorizers.md Outdated
Comment thread docs/content/concepts/authorization/authorizers.md Outdated
@SimonTheLeg SimonTheLeg force-pushed the extend-cross-workspace-auth-docs branch from 07004d9 to 91dfc00 Compare February 5, 2026 09:44
@SimonTheLeg

Copy link
Copy Markdown
Member Author

/kind documentation

@kcp-ci-bot kcp-ci-bot added kind/documentation Categorizes issue or PR as related to documentation. and removed do-not-merge/needs-kind Indicates a PR lacks a `kind/foo` label and requires one. labels Feb 5, 2026
Comment thread docs/content/concepts/authorization/authorizers.md Outdated
Comment thread docs/content/concepts/authorization/authorizers.md Outdated
Comment thread docs/content/concepts/authorization/authorizers.md Outdated
Comment thread docs/content/concepts/authorization/authorizers.md Outdated
@SimonTheLeg SimonTheLeg force-pushed the extend-cross-workspace-auth-docs branch from 91dfc00 to 3db1d4d Compare February 13, 2026 19:07
with groups `system:authenticated` and `system:cluster:<logical-cluster>`, where `<logical-cluster>` is the name of the logical cluster backing
the workspace they originated from.
!!! note
As the example above targets Groups, it applies to all ServiceAccounts within the original workspace,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
As the example above targets Groups, it applies to all ServiceAccounts within the original workspace,
As the example above targets Groups, it applies to all identities authenticated with the original workspace,

Small but important distinction.

On-behalf-of: SAP <simon.bein@sap.com>
Signed-off-by: Simon Bein <simontheleg@gmail.com>
@SimonTheLeg SimonTheLeg force-pushed the extend-cross-workspace-auth-docs branch from 3db1d4d to 5c52ad0 Compare February 23, 2026 09:11
@SimonTheLeg

Copy link
Copy Markdown
Member Author

all changes should now be accounted for

@ntnn ntnn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm
/approve

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Feb 23, 2026
@kcp-ci-bot

Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: 232cbc646cb018bb94f7a7a497908faf5b8ba698

@kcp-ci-bot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ntnn

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kcp-ci-bot kcp-ci-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 23, 2026
@ntnn

ntnn commented Feb 23, 2026

Copy link
Copy Markdown
Member

/retest

@xrstf xrstf closed this Feb 23, 2026
@xrstf xrstf reopened this Feb 23, 2026
@kcp-ci-bot kcp-ci-bot merged commit 2bc000f into kcp-dev:main Feb 23, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has signed the DCO. kind/documentation Categorizes issue or PR as related to documentation. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: Update authorizers docs

5 participants