From 261f4b1cf6afcd03f829304066494ab04d206221 Mon Sep 17 00:00:00 2001
From: Christopher Brown <chrisbro@redhat.com>
Date: Fri, 16 Oct 2020 22:35:45 +1000
Subject: [PATCH] Add support for the keystone_authtoken/service_type parameter

Change-Id: I2c3a5d3c005963fe490310f6b81619cc85b28cc9
---
 manifests/keystone/authtoken.pp                            | 7 +++++++
 .../keystone-authtoken-service_type-d652a7ff4a1ff96c.yaml  | 5 +++++
 spec/classes/heat_keystone_authtoken_spec.rb               | 3 +++
 3 files changed, 15 insertions(+)
 create mode 100644 releasenotes/notes/keystone-authtoken-service_type-d652a7ff4a1ff96c.yaml

diff --git a/manifests/keystone/authtoken.pp b/manifests/keystone/authtoken.pp
index f5bdc49e..5472f74e 100644
--- a/manifests/keystone/authtoken.pp
+++ b/manifests/keystone/authtoken.pp
@@ -177,6 +177,11 @@
 #   true/false
 #   Defaults to $::os_service_default.
 #
+# [*service_type*]
+#  (Optional) The name or type of the service as it appears in the service
+#  catalog. This is used to validate tokens that have restricted access rules.
+#  Defaults to $::os_service_default.
+#
 # [*interface*]
 #  (Optional) Interface to use for the Identity API endpoint. Valid values are
 #  "public", "internal" or "admin".
@@ -217,6 +222,7 @@ class heat::keystone::authtoken(
   $token_cache_time               = $::os_service_default,
   $service_token_roles            = $::os_service_default,
   $service_token_roles_required   = $::os_service_default,
+  $service_type                   = $::os_service_default,
   $interface                      = $::os_service_default,
 ) {
 
@@ -261,6 +267,7 @@ class heat::keystone::authtoken(
     token_cache_time               => $token_cache_time,
     service_token_roles            => $service_token_roles,
     service_token_roles_required   => $service_token_roles_required,
+    service_type                   => $service_type,
     interface                      => $interface,
   }
 }
diff --git a/releasenotes/notes/keystone-authtoken-service_type-d652a7ff4a1ff96c.yaml b/releasenotes/notes/keystone-authtoken-service_type-d652a7ff4a1ff96c.yaml
new file mode 100644
index 00000000..bc395ff8
--- /dev/null
+++ b/releasenotes/notes/keystone-authtoken-service_type-d652a7ff4a1ff96c.yaml
@@ -0,0 +1,5 @@
+---
+features:
+  - |
+    The new ``heat::keystone::authtoken::service_type`` parameter has been
+    added to configure the service_type parameter in authtoken middleware.
diff --git a/spec/classes/heat_keystone_authtoken_spec.rb b/spec/classes/heat_keystone_authtoken_spec.rb
index db547da5..500c3b05 100644
--- a/spec/classes/heat_keystone_authtoken_spec.rb
+++ b/spec/classes/heat_keystone_authtoken_spec.rb
@@ -44,6 +44,7 @@ describe 'heat::keystone::authtoken' do
         is_expected.to contain_heat_config('keystone_authtoken/token_cache_time').with_value('<SERVICE DEFAULT>')
         is_expected.to contain_heat_config('keystone_authtoken/service_token_roles').with_value('<SERVICE DEFAULT>')
         is_expected.to contain_heat_config('keystone_authtoken/service_token_roles_required').with_value('<SERVICE DEFAULT>')
+        is_expected.to contain_heat_config('keystone_authtoken/service_type').with_value('<SERVICE DEFAULT>')
         is_expected.to contain_heat_config('keystone_authtoken/interface').with_value('<SERVICE DEFAULT>')
       end
     end
@@ -85,6 +86,7 @@ describe 'heat::keystone::authtoken' do
           :token_cache_time                     => '301',
           :service_token_roles                  => ['service'],
           :service_token_roles_required         => false,
+          :service_type                         => 'identity',
           :interface                            => 'internal',
         })
       end
@@ -123,6 +125,7 @@ describe 'heat::keystone::authtoken' do
         is_expected.to contain_heat_config('keystone_authtoken/token_cache_time').with_value(params[:token_cache_time])
         is_expected.to contain_heat_config('keystone_authtoken/service_token_roles').with_value(params[:service_token_roles])
         is_expected.to contain_heat_config('keystone_authtoken/service_token_roles_required').with_value(params[:service_token_roles_required])
+        is_expected.to contain_heat_config('keystone_authtoken/service_type').with_value(params[:service_type])
         is_expected.to contain_heat_config('keystone_authtoken/interface').with_value(params[:interface])
       end