Skip to content

Commit

Permalink
Add serviceaccount list to network-admin and user-ui (tigera#3687)
Browse files Browse the repository at this point in the history
* Add serviceaccount list to network-admin and user-ui

* Fix dirty felixconfigurations.yaml
  • Loading branch information
dimitri-nicolo authored and ti-afra committed Jan 14, 2025
1 parent 214d606 commit 1011972
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 0 deletions.
12 changes: 12 additions & 0 deletions pkg/render/apiserver.go
Original file line number Diff line number Diff line change
Expand Up @@ -1661,6 +1661,12 @@ func (c *apiServerComponent) tigeraUserClusterRole() *rbacv1.ClusterRole {
Resources: []string{"pods"},
Verbs: []string{"list"},
},
// Additional "list" requests required to view serviceaccount labels.
{
APIGroups: []string{""},
Resources: []string{"serviceaccounts"},
Verbs: []string{"list"},
},
// Access to statistics.
{
APIGroups: []string{""},
Expand Down Expand Up @@ -1845,6 +1851,12 @@ func (c *apiServerComponent) tigeraNetworkAdminClusterRole() *rbacv1.ClusterRole
Resources: []string{"pods"},
Verbs: []string{"list"},
},
// Additional "list" requests required to view serviceaccount labels.
{
APIGroups: []string{""},
Resources: []string{"serviceaccounts"},
Verbs: []string{"list"},
},
// Access to statistics.
{
APIGroups: []string{""},
Expand Down
10 changes: 10 additions & 0 deletions pkg/render/apiserver_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1314,6 +1314,11 @@ var (
Resources: []string{"pods"},
Verbs: []string{"list"},
},
{
APIGroups: []string{""},
Resources: []string{"serviceaccounts"},
Verbs: []string{"list"},
},
{
APIGroups: []string{""},
Resources: []string{"services/proxy"},
Expand Down Expand Up @@ -1458,6 +1463,11 @@ var (
Resources: []string{"pods"},
Verbs: []string{"list"},
},
{
APIGroups: []string{""},
Resources: []string{"serviceaccounts"},
Verbs: []string{"list"},
},
{
APIGroups: []string{""},
Resources: []string{"services/proxy"},
Expand Down

0 comments on commit 1011972

Please sign in to comment.