Add support for the oslo_policy/enforce_scope parameter
Depends-on: https://review.opendev.org/#/c/759008/ Change-Id: I7d7776d81f61ebc133185e1d1bc55f84fc58b078
This commit is contained in:
parent
82b53aed5c
commit
77fb34e43e
@ -4,6 +4,10 @@
|
|||||||
#
|
#
|
||||||
# === Parameters
|
# === Parameters
|
||||||
#
|
#
|
||||||
|
# [*enforce_scope*]
|
||||||
|
# (Optional) Whether or not to enforce scope when evaluating policies.
|
||||||
|
# Defaults to $::os_service_default.
|
||||||
|
#
|
||||||
# [*policies*]
|
# [*policies*]
|
||||||
# (Optional) Set of policies to configure for trove
|
# (Optional) Set of policies to configure for trove
|
||||||
# Example :
|
# Example :
|
||||||
@ -24,8 +28,9 @@
|
|||||||
# Defaults to /etc/trove/policy.yaml
|
# Defaults to /etc/trove/policy.yaml
|
||||||
#
|
#
|
||||||
class trove::policy (
|
class trove::policy (
|
||||||
$policies = {},
|
$enforce_scope = $::os_service_default,
|
||||||
$policy_path = '/etc/trove/policy.yaml',
|
$policies = {},
|
||||||
|
$policy_path = '/etc/trove/policy.yaml',
|
||||||
) {
|
) {
|
||||||
|
|
||||||
include trove::deps
|
include trove::deps
|
||||||
@ -42,6 +47,9 @@ class trove::policy (
|
|||||||
|
|
||||||
create_resources('openstacklib::policy::base', $policies)
|
create_resources('openstacklib::policy::base', $policies)
|
||||||
|
|
||||||
oslo::policy { 'trove_config': policy_file => $policy_path }
|
oslo::policy { 'trove_config':
|
||||||
|
enforce_scope => $enforce_scope,
|
||||||
|
policy_file => $policy_path
|
||||||
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
@ -0,0 +1,5 @@
|
|||||||
|
---
|
||||||
|
features:
|
||||||
|
- |
|
||||||
|
The new ``trove::policy::enforce_scope`` parameter has been added to
|
||||||
|
support the corresponding parameter in oslo.policy library.
|
@ -1,12 +1,12 @@
|
|||||||
require 'spec_helper'
|
require 'spec_helper'
|
||||||
|
|
||||||
describe 'trove::policy' do
|
describe 'trove::policy' do
|
||||||
|
shared_examples 'trove::policy' do
|
||||||
shared_examples_for 'trove policies' do
|
|
||||||
let :params do
|
let :params do
|
||||||
{
|
{
|
||||||
:policy_path => '/etc/trove/policy.yaml',
|
:enforce_scope => false,
|
||||||
:policies => {
|
:policy_path => '/etc/trove/policy.yaml',
|
||||||
|
:policies => {
|
||||||
'context_is_admin' => {
|
'context_is_admin' => {
|
||||||
'key' => 'context_is_admin',
|
'key' => 'context_is_admin',
|
||||||
'value' => 'foo:bar'
|
'value' => 'foo:bar'
|
||||||
@ -24,20 +24,21 @@ describe 'trove::policy' do
|
|||||||
:file_format => 'yaml',
|
:file_format => 'yaml',
|
||||||
})
|
})
|
||||||
is_expected.to contain_oslo__policy('trove_config').with(
|
is_expected.to contain_oslo__policy('trove_config').with(
|
||||||
:policy_file => '/etc/trove/policy.yaml',
|
:enforce_scope => false,
|
||||||
|
:policy_file => '/etc/trove/policy.yaml',
|
||||||
)
|
)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
on_supported_os({
|
on_supported_os({
|
||||||
:supported_os => OSDefaults.get_supported_os
|
:supported_os => OSDefaults.get_supported_os
|
||||||
}).each do |os,facts|
|
}).each do |os,facts|
|
||||||
context "on #{os}" do
|
context "on #{os}" do
|
||||||
let (:facts) do
|
let (:facts) do
|
||||||
facts.merge!(OSDefaults.get_facts())
|
facts.merge!(OSDefaults.get_facts())
|
||||||
end
|
end
|
||||||
|
|
||||||
it_configures 'trove policies'
|
it_behaves_like 'trove::policy'
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
Loading…
Reference in New Issue
Block a user