Skip to content

Commit 2721c44

Browse files
Adding deployment viewer permission to developers and auditors (#958)
Adding deployment viewer permission to developers and auditors
1 parent a00b431 commit 2721c44

File tree

2 files changed

+21
-0
lines changed

2 files changed

+21
-0
lines changed

config/200-user-roles.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,9 @@ rules:
7070
- apiGroups: [""]
7171
resources: ["pods"]
7272
verbs: ["get", "list", "watch"]
73+
- apiGroups: ["apps"]
74+
resources: ["deployments"]
75+
verbs: ["get", "list", "watch"]
7376
- apiGroups: [""]
7477
resources: ["pods/exec"]
7578
verbs: ["create"]
@@ -119,6 +122,9 @@ rules:
119122
- apiGroups: [""]
120123
resources: ["pods"]
121124
verbs: ["get", "list", "watch"]
125+
- apiGroups: ["apps"]
126+
resources: ["deployments"]
127+
verbs: ["get", "list", "watch"]
122128
- apiGroups: [""]
123129
resources: ["pods/exec"]
124130
verbs: ["get", "list", "watch"]

pkg/kf/commands/spaces/integration_test.go

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,13 @@ func verifyDeveloperPermission(ctx context.Context, t *testing.T, namespace stri
150150
{title: "SpaceDeveloper can not patch spaces in cluster", space: "", verb: "patch", group: "kf.dev", resource: "spaces", expectedOutput: false},
151151
{title: "SpaceDeveloper can not delete spaces in cluster", space: "", verb: "delete", group: "kf.dev", resource: "spaces", expectedOutput: false},
152152
{title: "SpaceDeveloper can not list Apps in other space", space: "kf", verb: "list", group: "kf.dev", resource: "apps", expectedOutput: false},
153+
{title: "SpaceDeveloper gets Deployments in space", space: namespace, verb: "get", group: "apps", resource: "deployments", expectedOutput: true},
154+
{title: "SpaceDeveloper lists Deployments in space", space: namespace, verb: "list", group: "apps", resource: "deployments", expectedOutput: true},
155+
{title: "SpaceDeveloper watches Deployments in space", space: namespace, verb: "watch", group: "apps", resource: "deployments", expectedOutput: true},
156+
{title: "SpaceDeveloper can not create Deployments in space", space: namespace, verb: "create", group: "apps", resource: "deployments", expectedOutput: false},
157+
{title: "SpaceDeveloper can not update Deployments in space", space: namespace, verb: "update", group: "apps", resource: "deployments", expectedOutput: false},
158+
{title: "SpaceDeveloper can not patch Deployments in space", space: namespace, verb: "patch", group: "apps", resource: "deployments", expectedOutput: false},
159+
{title: "SpaceDeveloper can not delete Deployments in space", space: namespace, verb: "delete", group: "apps", resource: "deployments", expectedOutput: false},
153160
}
154161

155162
for _, test := range tests {
@@ -204,6 +211,14 @@ func verifyAuditorPermission(ctx context.Context, t *testing.T, namespace string
204211
{title: "SpaceAuditor can not patch Pods in space", space: namespace, verb: "patch", group: "", resource: "pods", expectedOutput: false},
205212
{title: "SpaceAuditor can not delete Pods in space", space: namespace, verb: "delete", group: "", resource: "pods", expectedOutput: false},
206213

214+
{title: "SpaceAuditor gets Deployments in space", space: namespace, verb: "get", group: "apps", resource: "deployments", expectedOutput: true},
215+
{title: "SpaceAuditor lists Deployments in space", space: namespace, verb: "list", group: "apps", resource: "deployments", expectedOutput: true},
216+
{title: "SpaceAuditor watches Deployments in space", space: namespace, verb: "watch", group: "apps", resource: "deployments", expectedOutput: true},
217+
{title: "SpaceAuditor can not create Deployments in space", space: namespace, verb: "create", group: "apps", resource: "deployments", expectedOutput: false},
218+
{title: "SpaceAuditor can not update Deployments in space", space: namespace, verb: "update", group: "apps", resource: "deployments", expectedOutput: false},
219+
{title: "SpaceAuditor can not patch Deployments in space", space: namespace, verb: "patch", group: "apps", resource: "deployments", expectedOutput: false},
220+
{title: "SpaceAuditor can not delete Deployments in space", space: namespace, verb: "delete", group: "apps", resource: "deployments", expectedOutput: false},
221+
207222
{title: "SpaceAuditor gets Pods log in space", space: namespace, verb: "get", group: "", resource: "pods/log", expectedOutput: true},
208223
{title: "SpaceAuditor lists Pods log in space", space: namespace, verb: "list", group: "", resource: "pods/log", expectedOutput: true},
209224
{title: "SpaceAuditor watches Pods log in space", space: namespace, verb: "watch", group: "", resource: "pods/log", expectedOutput: true},

0 commit comments

Comments
 (0)