Skip to content

Commit 5e4e0a3

Browse files
authored
Merge pull request #9768 from zalando-incubator/read-only-note
Add string docs for RBAC read-only testing
2 parents a644c80 + 60920c1 commit 5e4e0a3

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

test/e2e/authorization.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,14 @@ var _ = g.Describe("Authorization [RBAC] [Zalando]", func() {
130130
})
131131
})
132132

133+
// NOTE: The read-only role is restricted by RBAC to non-mutating operations.
134+
// Such requests bypass the admission controller, which only processes
135+
// mutating requests. Admission controller tests for this role are unnecessary,
136+
// as access control is fully enforced at the RBAC authorization stage.
137+
// Flow example:
138+
// 1. Request Received → RBAC checks role permissions.
139+
// 2. Read-Only Role (`GET`) → Allowed by RBAC, **skips** admission controller.
140+
// 3. Read-Only Role (`DELETE`) → Blocked by RBAC, so never reaches admission controller.
133141
g.Context("For ReadOnly group", func() {
134142
var tc testCase
135143
g.BeforeEach(func() {

0 commit comments

Comments
 (0)