Drop ineffective neutron_*_parameter
Although we didn't deprecate these formally during the previous cycle, we added a warning which may effectively encourage users to drop these parameters from their manifests. Depends-on: https://review.opendev.org/957309 Change-Id: Ibd7e6dad11e08f4c2a65ae824c4e037bd101a34f Signed-off-by: Takashi Kajinami <kajinamit@oss.nttdata.com>
This commit is contained in:
@@ -150,14 +150,8 @@
|
|||||||
# Defaults to true
|
# Defaults to true
|
||||||
# [*neutron_available*]
|
# [*neutron_available*]
|
||||||
# Defaults to true
|
# Defaults to true
|
||||||
# [*neutron_bgpvpn_available*]
|
|
||||||
# Defaults to false
|
|
||||||
# [*neutron_l2gw_available*]
|
# [*neutron_l2gw_available*]
|
||||||
# Defaults to true
|
# Defaults to true
|
||||||
# [*neutron_vpnaas_available*]
|
|
||||||
# Defaults to false
|
|
||||||
# [*neutron_dr_available*]
|
|
||||||
# Defaults to false
|
|
||||||
# [*nova_available*]
|
# [*nova_available*]
|
||||||
# Defaults to true
|
# Defaults to true
|
||||||
# [*watcher_available*]
|
# [*watcher_available*]
|
||||||
@@ -464,10 +458,7 @@ class tempest(
|
|||||||
Boolean $designate_available = false,
|
Boolean $designate_available = false,
|
||||||
Boolean $horizon_available = true,
|
Boolean $horizon_available = true,
|
||||||
Boolean $neutron_available = true,
|
Boolean $neutron_available = true,
|
||||||
Boolean $neutron_bgpvpn_available = false,
|
|
||||||
Boolean $neutron_l2gw_available = false,
|
Boolean $neutron_l2gw_available = false,
|
||||||
Boolean $neutron_vpnaas_available = false,
|
|
||||||
Boolean $neutron_dr_available = false,
|
|
||||||
Boolean $nova_available = true,
|
Boolean $nova_available = true,
|
||||||
Boolean $swift_available = false,
|
Boolean $swift_available = false,
|
||||||
Boolean $trove_available = false,
|
Boolean $trove_available = false,
|
||||||
@@ -573,12 +564,6 @@ class tempest(
|
|||||||
}
|
}
|
||||||
$auth_version_real = pick($auth_version, $facts['os_service_default'])
|
$auth_version_real = pick($auth_version, $facts['os_service_default'])
|
||||||
|
|
||||||
[ 'neutron_bgpvpn_available', 'neutron_vpnaas_available', 'neutron_dr_available' ].each |$opt| {
|
|
||||||
if getvar($opt) {
|
|
||||||
warning("The ${opt} parameter has no effect now. Use the neutron_api_extensions parameter instead.")
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if $run_service_broker_tests != undef {
|
if $run_service_broker_tests != undef {
|
||||||
warning('The run_service_broker_tests parameter is deprecated and has no effect.')
|
warning('The run_service_broker_tests parameter is deprecated and has no effect.')
|
||||||
}
|
}
|
||||||
|
@@ -0,0 +1,8 @@
|
|||||||
|
---
|
||||||
|
upgrade:
|
||||||
|
- |
|
||||||
|
The following parameters of the ``temepst`` class have been removed.
|
||||||
|
|
||||||
|
- ``neutron_bgpvpn_available``
|
||||||
|
- ``neutron_dr_available``
|
||||||
|
- ``neutron_vpnass_available``
|
Reference in New Issue
Block a user