Commitaff4398

authored and

committed

test: fix rbac benchmark to test performance instead of cache (#20097)

The benchmark should be testing the performance of `authorize`, not acache lookup

1 parent0578b1a commitaff4398Copy full SHA for aff4398

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,7 @@ func BenchmarkRBACAuthorizeGroups(b *testing.B) {`
`187`	`187`	`uuid.MustParse("0632b012-49e0-4d70-a5b3-f4398f1dcd52"),`
`188`	`188`	`uuid.MustParse("70dbaa7a-ea9c-4f68-a781-97b08af8461d"),`
`189`	`189`	`)`
`190`		`-authorizer:=rbac.NewStrictCachingAuthorizer(prometheus.NewRegistry())`
	`190`	`+authorizer:=rbac.NewAuthorizer(prometheus.NewRegistry())`
`191`	`191`
`192`	`192`	`// Same benchmark cases, but this time groups will be used to match.`
`193`	`193`	`// Some '*' permissions will still match, but using a fake action reduces`

Comments

(0)