openstack
/
puppet-designate
Code Issues Proposed changes

Merge "Add support for [DEFAULT] default_ttl"

This commit is contained in:
Zuul 2022-08-29 16:32:54 +00:00 committed by Gerrit Code Review
parent 3d8fa7d60e 3d68d191e2
commit a4c4758780
3 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
manifests/init.pp
View File

@ -110,6 +110,10 @@
# (optional) Whether to use durable queues in AMQP.
# Defaults to $::os_service_default.
#
# [*default_ttl*]
# (Optional) TTL Value.
# Defaults to $::os_service_default.
#
class designate(
$package_ensure = present,
$common_package_name = $::designate::params::common_package_name,
@ -133,6 +137,7 @@ class designate(
$notification_topics = 'notifications',
$purge_config = false,
$amqp_durable_queues = $::os_service_default,
$default_ttl = $::os_service_default,
) inherits designate::params {
include designate::deps
@ -177,6 +182,7 @@ class designate(
'DEFAULT/host': value => $host;
'DEFAULT/root_helper': value => $root_helper;
'DEFAULT/state_path' : value => $state_path;
'DEFAULT/default_ttl': value => $default_ttl;
}
}

4
releasenotes/notes/default_ttl-c8c719139b5d218c.yaml Normal file
View File

@ -0,0 +1,4 @@
---
features:
- |
The ``designate::default_ttl`` parameter has been added.

7
spec/classes/designate_init_spec.rb
View File

@ -37,20 +37,23 @@ describe 'designate' do
it 'configures the default values' do
is_expected.to contain_designate_config('DEFAULT/host').with_value('<SERVICE DEFAULT>')
is_expected.to contain_designate_config('DEFAULT/state_path').with_value('/var/lib/designate')
is_expected.to contain_designate_config('DEFAULT/default_ttl').with_value('<SERVICE DEFAULT>')
end
end
context 'with parameters' do
let :params do
{
:host => 'current_hostname',
:state_path => '/var/tmp/designate'
:host => 'current_hostname',
:state_path => '/var/tmp/designate',
:default_ttl => 3600
}
end
it 'configures the given values' do
is_expected.to contain_designate_config('DEFAULT/host').with_value('current_hostname')
is_expected.to contain_designate_config('DEFAULT/state_path').with_value('/var/tmp/designate')
is_expected.to contain_designate_config('DEFAULT/default_ttl').with_value(3600)
end
end