openstack
/
puppet-nova
Code Issues Proposed changes

Merge "Drop too strict validation of instance_usage_audit_period"

This commit is contained in:
Zuul 2023-03-22 05:45:17 +00:00 committed by Gerrit Code Review
parent d12c547f5b b7dc7fbbfe
commit 54aad4e30b
2 changed files with 4 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
manifests/compute.pp
View File

@ -64,8 +64,8 @@
#
# [*instance_usage_audit_period*]
# (optional) Time period to generate instance usages for.
# Time period must be hour, day, month or year
# Defaults to 'month'
# Time period must be hour, day, month or year with/without offset.
# Defaults to $facts['os_service_default']
#
# [*use_cow_images*]
# (optional) Enable use of copy-on-write (cow) images.
@ -266,7 +266,7 @@ class nova::compute (
$vncproxy_path = '/vnc_auto.html',
$force_config_drive = false,
$instance_usage_audit = false,
$instance_usage_audit_period = 'month',
$instance_usage_audit_period = $facts['os_service_default'],
$mkisofs_cmd = $facts['os_service_default'],
$use_cow_images = $facts['os_service_default'],
$force_raw_images = $facts['os_service_default'],
@ -447,7 +447,7 @@ class nova::compute (
nova_config { 'DEFAULT/force_config_drive': ensure => absent }
}
if $instance_usage_audit and $instance_usage_audit_period in ['hour', 'day', 'month', 'year'] {
if $instance_usage_audit {
nova_config {
'DEFAULT/instance_usage_audit': value => $instance_usage_audit;
'DEFAULT/instance_usage_audit_period': value => $instance_usage_audit_period;

10
spec/classes/nova_compute_spec.rb
View File

@ -326,16 +326,6 @@ describe 'nova::compute' do
it { is_expected.to contain_nova_config('DEFAULT/instance_usage_audit_period').with_ensure('absent') }
end
context 'with instance_usage_audit parameter and wrong period' do
let :params do
{ :instance_usage_audit => true,
:instance_usage_audit_period => 'fake', }
end
it { is_expected.to contain_nova_config('DEFAULT/instance_usage_audit').with_ensure('absent') }
it { is_expected.to contain_nova_config('DEFAULT/instance_usage_audit_period').with_ensure('absent') }
end
context 'with instance_usage_audit parameter and period' do
let :params do
{ :instance_usage_audit => true,