Skip to content

Commit 70a29a0

Browse files
Merge pull request openstack-k8s-operators#346 from dmendiza/enable-secure-rbac
Enable Secure RBAC by default
2 parents 9bbb46c + 13db6de commit 70a29a0

File tree

5 files changed

+20
-0
lines changed

5 files changed

+20
-0
lines changed

api/bases/keystone.openstack.org_keystoneapis.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,11 @@ spec:
8686
files. Those get added to the service config dir in /etc/<service>
8787
. TODO: -> implement'
8888
type: object
89+
enableSecureRBAC:
90+
default: true
91+
description: EnableSecureRBAC - Enable Consistent and Secure RBAC
92+
policies
93+
type: boolean
8994
memcachedInstance:
9095
default: memcached
9196
description: Memcached instance name.

api/v1beta1/keystoneapi_types.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,11 @@ type KeystoneAPISpec struct {
9696
// Secret containing OpenStack password information for keystone KeystoneDatabasePassword, AdminPassword
9797
Secret string `json:"secret"`
9898

99+
// +kubebuilder:validation:Optional
100+
// +kubebuilder:default=true
101+
// EnableSecureRBAC - Enable Consistent and Secure RBAC policies
102+
EnableSecureRBAC bool `json:"enableSecureRBAC"`
103+
99104
// +kubebuilder:validation:Optional
100105
// +kubebuilder:default=""
101106
// TrustFlushArgs - Arguments added to keystone-manage trust_flush command

config/crd/bases/keystone.openstack.org_keystoneapis.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,11 @@ spec:
8686
files. Those get added to the service config dir in /etc/<service>
8787
. TODO: -> implement'
8888
type: object
89+
enableSecureRBAC:
90+
default: true
91+
description: EnableSecureRBAC - Enable Consistent and Secure RBAC
92+
policies
93+
type: boolean
8994
memcachedInstance:
9095
default: memcached
9196
description: Memcached instance name.

controllers/keystoneapi_controller.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1198,6 +1198,7 @@ func (r *KeystoneAPIReconciler) generateServiceConfigMaps(
11981198
instance.Status.DatabaseHostname,
11991199
keystone.DatabaseName,
12001200
),
1201+
"enableSecureRBAC": instance.Spec.EnableSecureRBAC,
12011202
}
12021203

12031204
// create httpd vhost template parameters

templates/keystoneapi/config/keystone.conf

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,10 @@ max_retries=-1
1111
db_max_retries=-1
1212
connection={{ .DatabaseConnection }}
1313

14+
[oslo_policy]
15+
enforce_new_defaults = {{ .enableSecureRBAC }}
16+
enforce_scope = {{ .enableSecureRBAC }}
17+
1418
[fernet_tokens]
1519
key_repository=/etc/keystone/fernet-keys
1620
max_active_keys=2

0 commit comments

Comments
 (0)