Browse Source

Service_token_roles_required missing in the server config file

Service_token_roles_required missing in the server config file which
allows backwards compatibility to ensure that the service tokens are
compared against a list of possible roles for validity.

Change-Id: I95089f9f1d5039d7d14e3850535536ec1804fdb1
Closes-Bug: 1778198
ZhongShengping 3 months ago
parent
commit
2b5fdbab82

+ 8
- 0
manifests/keystone/authtoken.pp View File

@@ -160,6 +160,12 @@
160 160
 #   (in seconds). Set to -1 to disable caching completely. Integer value
161 161
 #   Defaults to $::os_service_default.
162 162
 #
163
+# [*service_token_roles_required*]
164
+#   (optional) backwards compatibility to ensure that the service tokens are
165
+#   compared against a list of possible roles for validity
166
+#   true/false
167
+#   Defaults to $::os_service_default.
168
+#
163 169
 # DEPRECATED PARAMETERS
164 170
 #
165 171
 # [*check_revocations_for_cached*]
@@ -212,6 +218,7 @@ class freezer::keystone::authtoken(
212 218
   $manage_memcache_package        = false,
213 219
   $region_name                    = $::os_service_default,
214 220
   $token_cache_time               = $::os_service_default,
221
+  $service_token_roles_required   = $::os_service_default,
215 222
   # DEPRECATED PARAMETERS
216 223
   $check_revocations_for_cached   = undef,
217 224
   $hash_algorithms                = undef,
@@ -260,6 +267,7 @@ class freezer::keystone::authtoken(
260 267
     manage_memcache_package        => $manage_memcache_package,
261 268
     region_name                    => $region_name,
262 269
     token_cache_time               => $token_cache_time,
270
+    service_token_roles_required   => $service_token_roles_required,
263 271
   }
264 272
 }
265 273
 

+ 5
- 0
releasenotes/notes/service_token_roles_required-afb2c84dbd3faaff.yaml View File

@@ -0,0 +1,5 @@
1
+---
2
+features:
3
+  - Service_token_roles_required missing in the server config file which
4
+    allows backwards compatibility to ensure that the service tokens are
5
+    compared against a list of possible roles for validity.

+ 3
- 0
spec/classes/freezer_keystone_authtoken_spec.rb View File

@@ -42,6 +42,7 @@ describe 'freezer::keystone::authtoken' do
42 42
         is_expected.to contain_freezer_config('keystone_authtoken/memcached_servers').with_value('<SERVICE DEFAULT>')
43 43
         is_expected.to contain_freezer_config('keystone_authtoken/region_name').with_value('<SERVICE DEFAULT>')
44 44
         is_expected.to contain_freezer_config('keystone_authtoken/token_cache_time').with_value('<SERVICE DEFAULT>')
45
+        is_expected.to contain_freezer_config('keystone_authtoken/service_token_roles_required').with_value('<SERVICE DEFAULT>')
45 46
       end
46 47
     end
47 48
 
@@ -80,6 +81,7 @@ describe 'freezer::keystone::authtoken' do
80 81
           :manage_memcache_package              => true,
81 82
           :region_name                          => 'region2',
82 83
           :token_cache_time                     => '301',
84
+          :service_token_roles_required         => false,
83 85
         })
84 86
       end
85 87
 
@@ -115,6 +117,7 @@ describe 'freezer::keystone::authtoken' do
115 117
         is_expected.to contain_freezer_config('keystone_authtoken/memcached_servers').with_value('memcached01:11211,memcached02:11211')
116 118
         is_expected.to contain_freezer_config('keystone_authtoken/region_name').with_value(params[:region_name])
117 119
         is_expected.to contain_freezer_config('keystone_authtoken/token_cache_time').with_value(params[:token_cache_time])
120
+        is_expected.to contain_freezer_config('keystone_authtoken/service_token_roles_required').with_value(params[:service_token_roles_required])
118 121
       end
119 122
 
120 123
       it 'installs python memcache package' do

Loading…
Cancel
Save