Merge "vpnaas: Do not install wrong backend package for OpenSwan driver"

This commit is contained in:
Zuul
2023-11-08 10:17:36 +00:00
committed by Gerrit Code Review
4 changed files with 6 additions and 18 deletions

View File

@@ -53,12 +53,8 @@ class neutron::agents::vpnaas (
case $vpn_device_driver { case $vpn_device_driver {
/\.OpenSwan/: { /\.OpenSwan/: {
Package['openswan'] -> Package<| title == 'neutron-vpnaas-agent' |> warning("Support for OpenSwan has been deprecated, because of lack of \
package { 'openswan': openswan package in distributions")
ensure => present,
name => $::neutron::params::openswan_package,
tag => ['openstack', 'neutron-support-package'],
}
} }
/\.LibreSwan/: { /\.LibreSwan/: {
Package['libreswan'] -> Package<| title == 'neutron-vpnaas-agent' |> Package['libreswan'] -> Package<| title == 'neutron-vpnaas-agent' |>

View File

@@ -54,7 +54,6 @@ class neutron::params {
$ovn_agent_package = 'openstack-neutron-ovn-agent' $ovn_agent_package = 'openstack-neutron-ovn-agent'
$dynamic_routing_package = 'python3-neutron-dynamic-routing' $dynamic_routing_package = 'python3-neutron-dynamic-routing'
$bgp_dragent_package = 'openstack-neutron-bgp-dragent' $bgp_dragent_package = 'openstack-neutron-bgp-dragent'
$openswan_package = 'libreswan'
$libreswan_package = 'libreswan' $libreswan_package = 'libreswan'
$strongswan_package = 'strongswan' $strongswan_package = 'strongswan'
$metadata_agent_package = false $metadata_agent_package = false
@@ -101,7 +100,6 @@ class neutron::params {
$dhcp_agent_package = 'neutron-dhcp-agent' $dhcp_agent_package = 'neutron-dhcp-agent'
$metering_agent_package = 'neutron-metering-agent' $metering_agent_package = 'neutron-metering-agent'
$vpnaas_agent_package = 'python3-neutron-vpnaas' $vpnaas_agent_package = 'python3-neutron-vpnaas'
$openswan_package = 'strongswan'
$libreswan_package = 'libreswan' $libreswan_package = 'libreswan'
$strongswan_package = 'strongswan' $strongswan_package = 'strongswan'
$metadata_agent_package = 'neutron-metadata-agent' $metadata_agent_package = 'neutron-metadata-agent'

View File

@@ -0,0 +1,4 @@
---
deprecations:
- |
Support for OpenSwan VPNaaS driver has been derecated.

View File

@@ -48,14 +48,6 @@ describe 'neutron::agents::vpnaas' do
:tag => ['openstack', 'neutron-package'], :tag => ['openstack', 'neutron-package'],
) )
end end
it 'installs openswan packages' do
should contain_package('openswan').with(
:ensure => 'present',
:name => platform_params[:openswan_package],
:tag => ['openstack', 'neutron-support-package'],
)
end
end end
context 'with libreswan vpnaas driver' do context 'with libreswan vpnaas driver' do
@@ -113,14 +105,12 @@ describe 'neutron::agents::vpnaas' do
case facts[:os]['family'] case facts[:os]['family']
when 'Debian' when 'Debian'
{ {
:openswan_package => 'strongswan',
:libreswan_package => 'libreswan', :libreswan_package => 'libreswan',
:strongswan_package => 'strongswan', :strongswan_package => 'strongswan',
:vpnaas_agent_package => 'python3-neutron-vpnaas' :vpnaas_agent_package => 'python3-neutron-vpnaas'
} }
when 'RedHat' when 'RedHat'
{ {
:openswan_package => 'libreswan',
:libreswan_package => 'libreswan', :libreswan_package => 'libreswan',
:strongswan_package => 'strongswan', :strongswan_package => 'strongswan',
:vpnaas_agent_package => 'openstack-neutron-vpnaas' :vpnaas_agent_package => 'openstack-neutron-vpnaas'