openstack/puppet-keystone
Code Issues Proposed changes

Deprecate support for [DEFAULT] catalog_template_file

Browse Source 
The templated catalog driver has been deprecated in keystone, so we
should deprecate support for the driver options.

Change-Id: I5f3482397883e00d447eb08bed4c57821041826c
This commit is contained in:
Takashi Kajinami 2024-03-15 01:48:41 +09:00
parent 50b5260cc6
commit 29a32c74a6
2 changed files with 18 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
manifests/init.pp
View File

@ -12,10 +12,6 @@
# (Optional) Catalog driver used by Keystone to store endpoints and services. # (Optional) Catalog driver used by Keystone to store endpoints and services.
# Defaults to $facts['os_service_default']. # Defaults to $facts['os_service_default'].
# #
# [*catalog_template_file*]
# (Optional) Path to the catalog used if 'templated' catalog driver is used.
# Defaults to '/etc/keystone/default_catalog.templates'
#
# [*token_provider*] # [*token_provider*]
# (Optional) Format keystone uses for tokens. # (Optional) Format keystone uses for tokens.
# Defaults to 'fernet' # Defaults to 'fernet'
@ -342,6 +338,10 @@
# accepts latest or specific versions. # accepts latest or specific versions.
# Defaults to present. # Defaults to present.
# #
# [*catalog_template_file*]
# (Optional) Path to the catalog used if 'templated' catalog driver is used.
# Defaults to '/etc/keystone/default_catalog.templates'
#
# == Authors # == Authors
# #
# Dan Bode dan@puppetlabs.com # Dan Bode dan@puppetlabs.com
@ -353,7 +353,6 @@
class keystone( class keystone(
$package_ensure = 'present', $package_ensure = 'present',
$catalog_driver = $facts['os_service_default'], $catalog_driver = $facts['os_service_default'],
$catalog_template_file = '/etc/keystone/default_catalog.templates',
$token_provider = 'fernet', $token_provider = 'fernet',
$token_expiration = 3600, $token_expiration = 3600,
$password_hash_algorithm = $facts['os_service_default'], $password_hash_algorithm = $facts['os_service_default'],
@ -412,6 +411,7 @@ class keystone(
$amqp_durable_queues = $facts['os_service_default'], $amqp_durable_queues = $facts['os_service_default'],
# DEPRECATED PARAMETERS # DEPRECATED PARAMETERS
$client_package_ensure = undef, $client_package_ensure = undef,
$catalog_template_file = undef,
) inherits keystone::params { ) inherits keystone::params {
include keystone::deps include keystone::deps
@ -422,6 +422,13 @@ class keystone(
warning('The client_package_ensure parameter is deprecated and has no effect.') warning('The client_package_ensure parameter is deprecated and has no effect.')
} }
if $catalog_template_file != undef {
warning('The catalog_template_file parameter is deprecated and will be removed in a future release')
$catalog_template_file_real = $catalog_template_file
} else {
$catalog_template_file_real = '/etc/keystone/default_catalog.templates'
}
if $manage_policyrcd { if $manage_policyrcd {
# openstacklib policy_rcd only affects debian based systems. # openstacklib policy_rcd only affects debian based systems.
if ($facts['os']['name'] == 'Ubuntu') { if ($facts['os']['name'] == 'Ubuntu') {
@ -478,7 +485,7 @@ class keystone(
keystone_config { keystone_config {
'catalog/driver': value => $catalog_driver; 'catalog/driver': value => $catalog_driver;
'catalog/template_file': value => $catalog_template_file; 'catalog/template_file': value => $catalog_template_file_real;
} }
keystone_config { keystone_config {

5
releasenotes/notes/deprecate-catalog_template_file-5f8202e78af74aa3.yaml Normal file
View File

@ -0,0 +1,5 @@
---
deprecations:
- |
The ``keystone::catalog_template_file`` parameter has been deprecated and
will be removed in a futrue release.