Skip to content

Commit

Permalink
enable srbac by default
Browse files Browse the repository at this point in the history
This change enables nova's srbac feature
including scope enforcement

Closes: OSPRH-1505
  • Loading branch information
SeanMooney authored and gibizer committed Dec 13, 2023
1 parent 455707a commit a33ea4d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 2 additions & 1 deletion templates/nova.conf
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,8 @@ heartbeat_in_pthread=false

{{ if eq .service_name "nova-api"}}
[oslo_policy]
enforce_new_defaults=false
enforce_new_defaults=true
enforce_scope=true
{{end}}

{{ if eq .service_name "nova-conductor"}}
Expand Down
3 changes: 3 additions & 0 deletions test/functional/novaapi_controller_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -183,6 +183,9 @@ var _ = Describe("NovaAPI controller", func() {
Expect(configData).Should(ContainSubstring("www_authenticate_uri = keystone-public-auth-url"))
Expect(configData).Should(
ContainSubstring("[upgrade_levels]\ncompute = auto"))
Expect(configData).Should(ContainSubstring("enforce_new_defaults=true"))
Expect(configData).Should(ContainSubstring("enforce_scope=true"))
// test config override
Expect(configDataMap.Data).Should(HaveKey("02-nova-override.conf"))
extraData := string(configDataMap.Data["02-nova-override.conf"])
Expect(extraData).To(Equal("foo=bar"))
Expand Down

0 comments on commit a33ea4d

Please sign in to comment.