openstack/puppet-nova
Code Issues Proposed changes

Merge "Remove deprecated nova::periodic_interval"

Browse Source
This commit is contained in:
Zuul
2024-04-26 03:37:51 +00:00
committed by Gerrit Code Review
parent 1aa4cce270 b23268b083
commit dd59f94f57
2 changed files with 4 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
manifests/init.pp
View File

@@ -381,10 +381,6 @@
# (optional) The strategy to use for auth: noauth or keystone. # (optional) The strategy to use for auth: noauth or keystone.
# Defaults to undef # Defaults to undef
# #
# [*periodic_interval*]
# (optional) Seconds between running periodic tasks.
# Defaults to undef
#
class nova( class nova(
$ensure_package = 'present', $ensure_package = 'present',
$default_transport_url = $facts['os_service_default'], $default_transport_url = $facts['os_service_default'],
@@ -466,7 +462,6 @@ class nova(
$instance_name_template = $facts['os_service_default'], $instance_name_template = $facts['os_service_default'],
# DEPRECATED PARAMETERS # DEPRECATED PARAMETERS
$auth_strategy = undef, $auth_strategy = undef,
$periodic_interval = undef,
) inherits nova::params { ) inherits nova::params {
include nova::deps include nova::deps
@@ -476,10 +471,6 @@ class nova(
warning('enabled_ssl_apis is empty but use_ssl is set to true') warning('enabled_ssl_apis is empty but use_ssl is set to true')
} }
if $periodic_interval != undef {
warning('The periodic_interval parameter is deprecated and has no effect.')
}
if $use_ssl { if $use_ssl {
if !$cert_file { if !$cert_file {
fail('The cert_file parameter is required when use_ssl is set to true') fail('The cert_file parameter is required when use_ssl is set to true')

4
releasenotes/notes/remove-periodic_interval-299ed428b67b45ba.yaml Normal file
View File

@@ -0,0 +1,4 @@
---
upgrade:
- |
The deprecated ``nova::periodic_interval`` parameter has been removed.