openstack/puppet-ironic
Code Issues Proposed changes

Deprecate ironic::conductor::api_url

Browse Source 
... because its actual parameter, [conductor]api_url, was deprecated
in ironic while ago[1], and will be replaced by
[service_catalog]endpoint_override.

[1] 308e414a57941675acf6cd825daafc653cf5927e

Change-Id: Ifde129eb4cf1487c41c4f015154dec316057ca8a
This commit is contained in:
Takashi Kajinami 2020-04-13 21:35:30 +09:00
parent 01a4ef49a4
commit 305964417d
3 changed files with 19 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
manifests/conductor.pp
View File

@ -72,10 +72,6 @@
# erasure fails. Only makes sense if full hard disk erasing is enabled.
# Defaults to $::os_service_default
#
# [*api_url*]
# (optional) Ironic API URL.
# Defaults to $::os_service_default
#
# [*provisioning_network*]
# (optional) Neutron network UUID or name for the ramdisk to be booted into
# for provisioning nodes. Required for neutron network interface.
@ -204,6 +200,10 @@
# Should be an interger value
# Defaults to undef.
#
# [*api_url*]
# (optional) Ironic API URL.
# Defaults to undef.
#
class ironic::conductor (
$package_ensure = 'present',
$enabled = true,
@ -216,7 +216,6 @@ class ironic::conductor (
$cleaning_network = $::os_service_default,
$cleaning_disk_erase = undef,
$continue_if_disk_secure_erase_fails = $::os_service_default,
$api_url = $::os_service_default,
$provisioning_network = $::os_service_default,
$rescuing_network = $::os_service_default,
$inspection_network = $::os_service_default,
@ -242,6 +241,7 @@ class ironic::conductor (
$allow_provisioning_in_maintenance = $::os_service_default,
# DEPRECATED PARAMETERS
$max_time_interval = undef,
$api_url = undef,
) {
include ironic::deps
@ -254,6 +254,14 @@ class ironic::conductor (
warning('ironic::conductor::max_time_interval is deprecated and has no effect')
}
if $api_url != undef {
warning('ironic::conductor::api_url is deprecated. \
Use ironic::service_catalog::endpoint_override instead')
ironic_config {
'conductor/api_url': value => $api_url;
}
}
if ($cleaning_network_name and !is_service_default($cleaning_network)) {
fail('cleaning_network_name and cleaning_network can not be specified at the same time.')
}
@ -316,7 +324,6 @@ class ironic::conductor (
'DEFAULT/enabled_hardware_types': value => join($enabled_hardware_types, ',');
'conductor/force_power_state_during_sync': value => $force_power_state_during_sync;
'conductor/automated_clean': value => $automated_clean;
'conductor/api_url': value => $api_url;
'deploy/http_url': value => $http_url;
'deploy/http_root': value => $http_root;
'DEFAULT/force_raw_images': value => $force_raw_images;

6
releasenotes/notes/deprecate-conductor-api_url-4887de8eaac90c73.yaml Normal file
View File

@ -0,0 +1,6 @@
---
deprecations:
- |
The ``ironic::conductor::api_url`` parameter was deprecated because its
actual parameter was deprecated in ironic.
Use ``ironic::service_catalog::endpoint_override`` instead.

1
spec/classes/ironic_conductor_spec.rb
View File

@ -64,7 +64,6 @@ describe 'ironic::conductor' do
is_expected.to contain_ironic_config('DEFAULT/enabled_hardware_types').with_value('ipmi')
is_expected.to contain_ironic_config('conductor/force_power_state_during_sync').with_value(p[:force_power_state_during_sync])
is_expected.to contain_ironic_config('conductor/automated_clean').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_ironic_config('conductor/api_url').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_ironic_config('neutron/cleaning_network').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_ironic_config('neutron/provisioning_network').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_ironic_config('neutron/rescuing_network').with(:value => '<SERVICE DEFAULT>')