Merge "Remove keymgr_api_class option"

This commit is contained in:
Zuul 2018-10-22 05:41:22 +00:00 committed by Gerrit Code Review
commit b697f1f54e
3 changed files with 5 additions and 30 deletions

View File

@ -181,13 +181,6 @@
# tunneled networks. # tunneled networks.
# Defaults to [] # Defaults to []
# #
# DEPRECATED PARAMETERS
#
# [*keymgr_api_class*]
# (optional) Key Manager service.
# Example of valid value: castellan.key_manager.barbican_key_manager.BarbicanKeyManager
# Defaults to $::os_service_default
#
class nova::compute ( class nova::compute (
$enabled = true, $enabled = true,
$manage_service = true, $manage_service = true,
@ -227,8 +220,6 @@ class nova::compute (
$reserved_huge_pages = $::os_service_default, $reserved_huge_pages = $::os_service_default,
$neutron_physnets_numa_nodes_mapping = {}, $neutron_physnets_numa_nodes_mapping = {},
$neutron_tunnel_numa_nodes = [], $neutron_tunnel_numa_nodes = [],
# DEPRECATED PARAMETERS
$keymgr_api_class = undef,
) { ) {
include ::nova::deps include ::nova::deps
@ -240,19 +231,12 @@ class nova::compute (
include ::nova::pci include ::nova::pci
include ::nova::compute::vgpu include ::nova::compute::vgpu
if $keymgr_api_class {
warning('The keymgr_api_class parameter is deprecated, use keymgr_backend')
$keymgr_backend_real = $keymgr_api_class
} else {
$keymgr_backend_real = $keymgr_backend
}
if ($vnc_enabled and $spice_enabled) { if ($vnc_enabled and $spice_enabled) {
fail('vnc_enabled and spice_enabled is mutually exclusive') fail('vnc_enabled and spice_enabled is mutually exclusive')
} }
# cryptsetup is required when Barbican is encrypting volumes # cryptsetup is required when Barbican is encrypting volumes
if $keymgr_backend_real =~ /barbican/ { if $keymgr_backend =~ /barbican/ {
ensure_packages('cryptsetup', { ensure_packages('cryptsetup', {
ensure => present, ensure => present,
tag => 'openstack', tag => 'openstack',
@ -307,7 +291,7 @@ class nova::compute (
'DEFAULT/vcpu_pin_set': value => $vcpu_pin_set_real; 'DEFAULT/vcpu_pin_set': value => $vcpu_pin_set_real;
'DEFAULT/resume_guests_state_on_host_boot': value => $resume_guests_state_on_host_boot; 'DEFAULT/resume_guests_state_on_host_boot': value => $resume_guests_state_on_host_boot;
'compute/cpu_shared_set': value => $cpu_shared_set_real; 'compute/cpu_shared_set': value => $cpu_shared_set_real;
'key_manager/backend': value => $keymgr_backend_real; 'key_manager/backend': value => $keymgr_backend;
'barbican/auth_endpoint': value => $barbican_auth_endpoint; 'barbican/auth_endpoint': value => $barbican_auth_endpoint;
'barbican/barbican_endpoint': value => $barbican_endpoint; 'barbican/barbican_endpoint': value => $barbican_endpoint;
'barbican/barbican_api_version': value => $barbican_api_version; 'barbican/barbican_api_version': value => $barbican_api_version;

View File

@ -0,0 +1,3 @@
---
upgrade:
- Deprecated nova::compute::keymgr_api_class option has been removed.

View File

@ -187,18 +187,6 @@ describe 'nova::compute' do
end end
end end
context 'with barbican deprecated parameters' do
let :params do
{
:keymgr_api_class => 'castellan.key_manager.barbican_key_manager.BarbicanKeyManager',
}
end
it 'should set keymgr parameter' do
is_expected.to contain_nova_config('key_manager/backend').with_value('castellan.key_manager.barbican_key_manager.BarbicanKeyManager')
is_expected.to contain_package('cryptsetup').with( :ensure => 'present' )
end
end
context 'when vcpu_pin_set and pci params are empty' do context 'when vcpu_pin_set and pci params are empty' do
let :params do let :params do
{ :vcpu_pin_set => ""} { :vcpu_pin_set => ""}