Skip to content

Commit

Permalink
Merge pull request #3465 from LorcanMcVeigh/lorcan/add-deployments-fo…
Browse files Browse the repository at this point in the history
…r-waf

[RS-2023] Change services permissions to deployments permissions
  • Loading branch information
tmjd authored Aug 20, 2024
2 parents 4f6a594 + f9710ab commit a0208eb
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
10 changes: 5 additions & 5 deletions pkg/render/apiserver.go
Original file line number Diff line number Diff line change
Expand Up @@ -1537,8 +1537,8 @@ func (c *apiServerComponent) tigeraUserClusterRole() *rbacv1.ClusterRole {
},
// Allow the user to read services to view WAF configuration.
{
APIGroups: []string{""},
Resources: []string{"services"},
APIGroups: []string{"apps"},
Resources: []string{"deployments"},
Verbs: []string{"get", "list", "watch"},
},
// Allow the user to read felixconfigurations to detect if wireguard and/or other features are enabled.
Expand Down Expand Up @@ -1703,10 +1703,10 @@ func (c *apiServerComponent) tigeraNetworkAdminClusterRole() *rbacv1.ClusterRole
Resources: []string{"applicationlayers", "packetcaptureapis", "compliances", "intrusiondetections"},
Verbs: []string{"get", "update", "patch", "create", "delete"},
},
// Allow the user to read services to view WAF configuration.
// Allow the user to read deployments to view WAF configuration.
{
APIGroups: []string{""},
Resources: []string{"services"},
APIGroups: []string{"apps"},
Resources: []string{"deployments"},
Verbs: []string{"get", "list", "watch", "patch"},
},
// Allow the user to read felixconfigurations to detect if wireguard and/or other features are enabled.
Expand Down
8 changes: 4 additions & 4 deletions pkg/render/apiserver_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1339,8 +1339,8 @@ var (
Verbs: []string{"get"},
},
{
APIGroups: []string{""},
Resources: []string{"services"},
APIGroups: []string{"apps"},
Resources: []string{"deployments"},
Verbs: []string{"get", "list", "watch"},
},
{
Expand Down Expand Up @@ -1467,8 +1467,8 @@ var (
Verbs: []string{"get", "update", "patch", "create", "delete"},
},
{
APIGroups: []string{""},
Resources: []string{"services"},
APIGroups: []string{"apps"},
Resources: []string{"deployments"},
Verbs: []string{"get", "list", "watch", "patch"},
},
{
Expand Down

0 comments on commit a0208eb

Please sign in to comment.