From e7d3b24d1c6556839f702850d33aece730dad806 Mon Sep 17 00:00:00 2001
From: Takashi Kajinami <kajinamit@oss.nttdata.com>
Date: Sat, 27 Sep 2025 23:48:21 +0900
Subject: [PATCH] Import base parameter types for ::policy

Import the parameter types from openstacklib::policy so that parameter
types are validated at module level, instead of internal resource
call.

Also remove the tag which is no longer necessary after dependency
refactoring.

Change-Id: Ida348e6f4734ca2d264efe613050aad3f083b844
Signed-off-by: Takashi Kajinami <kajinamit@oss.nttdata.com>
---
 manifests/policy.pp               | 15 +++++++--------
 spec/classes/zaqar_policy_spec.rb |  2 --
 2 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/manifests/policy.pp b/manifests/policy.pp
index 0562494..ef7ca94 100644
--- a/manifests/policy.pp
+++ b/manifests/policy.pp
@@ -46,13 +46,13 @@
 #    Defaults to false.
 #
 class zaqar::policy (
-  $enforce_scope        = $facts['os_service_default'],
-  $enforce_new_defaults = $facts['os_service_default'],
-  Hash $policies        = {},
-  $policy_path          = '/etc/zaqar/policy.yaml',
-  $policy_default_rule  = $facts['os_service_default'],
-  $policy_dirs          = $facts['os_service_default'],
-  Boolean $purge_config = false,
+  $enforce_scope                    = $facts['os_service_default'],
+  $enforce_new_defaults             = $facts['os_service_default'],
+  Openstacklib::Policies $policies  = {},
+  Stdlib::Absolutepath $policy_path = '/etc/zaqar/policy.yaml',
+  $policy_default_rule              = $facts['os_service_default'],
+  $policy_dirs                      = $facts['os_service_default'],
+  Boolean $purge_config             = false,
 ) {
   include zaqar::deps
   include zaqar::params
@@ -64,7 +64,6 @@ class zaqar::policy (
     file_group   => $zaqar::params::group,
     file_format  => 'yaml',
     purge_config => $purge_config,
-    tag          => 'zaqar',
   }
 
   create_resources('openstacklib::policy', { $policy_path => $policy_parameters })
diff --git a/spec/classes/zaqar_policy_spec.rb b/spec/classes/zaqar_policy_spec.rb
index edc2bc0..1b3419a 100644
--- a/spec/classes/zaqar_policy_spec.rb
+++ b/spec/classes/zaqar_policy_spec.rb
@@ -33,7 +33,6 @@ describe 'zaqar::policy' do
           :file_group   => 'zaqar',
           :file_format  => 'yaml',
           :purge_config => false,
-          :tag          => 'zaqar',
         )
         is_expected.to contain_oslo__policy('zaqar_config').with(
           :enforce_scope        => false,
@@ -64,7 +63,6 @@ describe 'zaqar::policy' do
           :file_group   => 'zaqar',
           :file_format  => 'yaml',
           :purge_config => true,
-          :tag          => 'zaqar',
         )
         is_expected.to contain_oslo__policy('zaqar_config').with(
           :enforce_scope        => false,