Skip to content

Commit db8339e

Browse files
committed
Don't create _member_ role from puppet
The _member_ role is provided by db migration or is created automatically when user is created. Creating role _member_ from puppet causes it to have different id than the one specified in member_role_id keystone config option. Change-Id: I3f0102b184a484f5e5a85cadf26ed651829fbf8c Closes-Bug: rhbz#1129760
1 parent d080da7 commit db8339e

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

manifests/roles/admin.pp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
# * "admin" tenant (defaults to "openstack")
77
# * admin user (that defaults to the "admin" tenant)
88
# * admin role
9-
# * _member_ role
109
# * adds admin role to admin user on the "admin" tenant
1110
#
1211
# [*Parameters*]
@@ -58,7 +57,7 @@
5857
password => $password,
5958
ignore_default_tenant => $ignore_default_tenant,
6059
}
61-
keystone_role { ['admin', '_member_']:
60+
keystone_role { 'admin':
6261
ensure => present,
6362
}
6463
keystone_user_role { "${admin}@${admin_tenant}":

spec/classes/keystone_roles_admin_spec.rb

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,7 @@
2929
:password => 'ChangeMe',
3030
:ignore_default_tenant => 'false'
3131
)}
32-
['admin', '_member_'].each do |role_name|
33-
it { should contain_keystone_role(role_name).with_ensure('present') }
34-
end
32+
it { should contain_keystone_role('admin').with_ensure('present') }
3533
it { should contain_keystone_user_role('admin@openstack').with(
3634
:roles => 'admin',
3735
:ensure => 'present'

0 commit comments

Comments
 (0)