diff --git a/manifests/init.pp b/manifests/init.pp index 125fd80f0..23619c42b 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -174,14 +174,6 @@ # (Optional) Where to log # Defaults to undef. # -# [*admin_endpoint*] -# (Optional) The base admin endpoint URL for keystone that are -# advertised to clients (NOTE: this does NOT affect how keystone listens -# for connections) (string value) -# If set to false, no admin_endpoint will be defined in keystone.conf. -# Sample value: 'http://localhost:5000/' -# Defaults to $::os_service_default -# # [*public_endpoint*] # (Optional) The base public endpoint URL for keystone that are # advertised to clients (NOTE: this does NOT affect how @@ -399,6 +391,14 @@ # member_role_id option; see that option for more detail. # Defaults to undef # +# [*admin_endpoint*] +# (Optional) The base admin endpoint URL for keystone that are +# advertised to clients (NOTE: this does NOT affect how keystone listens +# for connections) (string value) +# If set to false, no admin_endpoint will be defined in keystone.conf. +# Sample value: 'http://localhost:5000/' +# Defaults to undef +# # == Authors # # Dan Bode dan@puppetlabs.com @@ -421,7 +421,6 @@ class keystone( $password_hash_rounds = $::os_service_default, $revoke_driver = $::os_service_default, $revoke_by_id = true, - $admin_endpoint = $::os_service_default, $public_endpoint = $::os_service_default, $enable_ssl = false, $ssl_certfile = '/etc/keystone/ssl/certs/keystone.pem', @@ -481,6 +480,7 @@ class keystone( $database_max_overflow = undef, $member_role_id = undef, $member_role_name = undef, + $admin_endpoint = undef, ) inherits keystone::params { include keystone::deps @@ -552,6 +552,10 @@ removed in a future realse. Use keystone::db::database_max_overflow instead') 'DEFAULT/member_role_name': ensure => absent; } + if $admin_endpoint != undef { + warning('The keystone::admin_endpoint parameter is deprecated. This parameter has no effect') + } + package { 'keystone': ensure => $package_ensure, name => $::keystone::params::package_name, diff --git a/releasenotes/notes/deprecate-admin_endpoint-d8ea2cc9c7423bf4.yaml b/releasenotes/notes/deprecate-admin_endpoint-d8ea2cc9c7423bf4.yaml new file mode 100644 index 000000000..04dab980d --- /dev/null +++ b/releasenotes/notes/deprecate-admin_endpoint-d8ea2cc9c7423bf4.yaml @@ -0,0 +1,5 @@ +--- +features: + - | + The ``keystone::admin_endpoint`` parameter has been deprecated because it + has not been used for several releases.