Deprecate support for amqp1 driver

The amqp1 driver has been deprecated in oslo.messaging because of
lack of qpid_dispatch module. This deprecates support for the driver.

Because deprecating all parameters for amqp1 driver needs large changes
in multiple modules, this change introduces the logic to detect usage
of amqp1 driver and shows the warning instead.

Depends-on: https://review.opendev.org/861070/
Change-Id: Ia176b8d881eb9bd1bb5e22b437b4e60ecd6963af
This commit is contained in:
Takashi Kajinami 2023-08-14 10:39:21 +09:00
parent 1eedd9bcc7
commit 9676c66dc3
4 changed files with 14 additions and 0 deletions

View File

@ -1,5 +1,6 @@
# == Define: oslo::messaging::amqp
#
# DEPRECATED !!
# Configure oslo_messaging_amqp options
#
# This resource configures Oslo messaging resources for an OpenStack service.

View File

@ -37,6 +37,10 @@ define oslo::messaging::default(
$control_exchange = $facts['os_service_default'],
) {
if $transport_url =~ 'amqp://.+' {
warning('amqp1 driver support has been deprecated, because the driver is deprecated.')
}
$default_options = {
'DEFAULT/executor_thread_pool_size' => { value => $executor_thread_pool_size },
'DEFAULT/rpc_response_timeout' => { value => $rpc_response_timeout },

View File

@ -33,6 +33,10 @@ define oslo::messaging::notifications(
$topics = $facts['os_service_default'],
) {
if $transport_url =~ 'amqp://.+' {
warning('amqp driver support has been deprecated, because the driver is deprecated.')
}
# When we have a string value for driver, we keep passing it as string
# to reduce any chance of breaking things in a backwards incompatible way
$driver_real = $driver ? {

View File

@ -0,0 +1,5 @@
---
deprecations:
- |
Support for the amqp1 driver has been deprecated and will be removed in
a future release.