openstack/puppet-nova
Code Issues Proposed changes

Merge "Deprecate support for [upgrade_levels] cert"

Browse Source
This commit is contained in:
Zuul
2022-05-17 18:51:28 +00:00
committed by Gerrit Code Review
parent 622156aa23 5ee0f09605
commit 983374fbee
2 changed files with 16 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
manifests/init.pp
View File

@@ -291,10 +291,6 @@
# (optional) Sets a version cap for messages sent to local cells services
# Defaults to $::os_service_default
#
# [*upgrade_level_cert*]
# (optional) Sets a version cap for messages sent to cert services
# Defaults to $::os_service_default
#
# [*upgrade_level_compute*]
# (optional) Sets a version cap for messages sent to compute services
# Defaults to $::os_service_default
@@ -384,6 +380,10 @@
# (optional) Allow attach between instance and volume in different availability zones.
# Defaults to undef
#
# [*upgrade_level_cert*]
# (optional) Sets a version cap for messages sent to cert services
# Defaults to undef
#
class nova(
$ensure_package = 'present',
$default_transport_url = $::os_service_default,
@@ -445,7 +445,6 @@ class nova(
$notify_on_state_change = undef,
$ovsdb_connection = $::os_service_default,
$upgrade_level_cells = $::os_service_default,
$upgrade_level_cert = $::os_service_default,
$upgrade_level_compute = $::os_service_default,
$upgrade_level_conductor = $::os_service_default,
$upgrade_level_intercell = $::os_service_default,
@@ -465,6 +464,7 @@ class nova(
$os_region_name = undef,
$cinder_catalog_info = undef,
$cross_az_attach = undef,
$upgrade_level_cert = undef,
) inherits nova::params {
include nova::deps
@@ -490,6 +490,11 @@ in a future release. Use nova::cinder::catalog_info instead')
Use nova::cinder::cross_az_attach instead.')
}
if $upgrade_level_cert != undef {
warning('The upgrade_level_cert parameter is deprecated and will be removed \
in a future release.')
}
if $use_ssl {
if !$cert_file {
fail('The cert_file parameter is required when use_ssl is set to true')
@@ -703,7 +708,7 @@ but should be one of: ssh-rsa, ssh-dsa, ssh-ecdsa.")
nova_config {
'upgrade_levels/cells': value => $upgrade_level_cells;
'upgrade_levels/cert': value => $upgrade_level_cert;
'upgrade_levels/cert': value => pick($upgrade_level_cert, $::os_service_default);
'upgrade_levels/compute': value => $upgrade_level_compute;
'upgrade_levels/conductor': value => $upgrade_level_conductor;
'upgrade_levels/intercell': value => $upgrade_level_intercell;

5
releasenotes/notes/deprecate-upgrade_level_cert-6f7dc745cd652300.yaml Normal file
View File

@@ -0,0 +1,5 @@
---
deprecations:
- |
The ``nova::upgrade_level_cert`` parameter has been deprecated and will be
removed in a future release.