openstack
/
puppet-designate
Code Issues Proposed changes

Remove deprecated keystone::auth options 

Change-Id: I6737d69bb886d6a27ecb91b8b2ef7e09dddfd49f
This commit is contained in:
Iury Gregory Melo Ferreira 2016-06-03 01:49:26 -03:00
parent 752e69aaae
commit 2dacd96def
3 changed files with 6 additions and 137 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

122
manifests/keystone/auth.pp
View File

@ -44,51 +44,6 @@
# (optional) The endpoint's internal url. (Defaults to 'http://127.0.0.1:9001')
# This url should *not* contain any trailing '/'.
#
# [*version*]
# (optional) DEPRECATED: Use public_url, internal_url and admin_url instead.
# API version endpoint. (Defaults to 'v1')
# Setting this parameter overrides public_url, internal_url and admin_url parameters.
#
# [*port*]
# (optional) DEPRECATED: Use public_url, internal_url and admin_url instead.
# Default port for endpoints. (Defaults to 9001)
# Setting this parameter overrides public_url, internal_url and admin_url parameters.
#
# [*public_protocol*]
# (optional) DEPRECATED: Use public_url instead.
# Protocol for public endpoint. (Defaults to 'http')
# Setting this parameter overrides public_url parameter.
#
# [*public_address*]
# (optional) DEPRECATED: Use public_url instead.
# Public address for endpoint. (Defaults to '127.0.0.1')
# Setting this parameter overrides public_url parameter.
#
# [*internal_protocol*]
# (optional) DEPRECATED: Use internal_url instead.
# Protocol for internal endpoint. (Defaults to 'http')
# Setting this parameter overrides internal_url parameter.
#
# [*internal_address*]
# (optional) DEPRECATED: Use internal_url instead.
# Internal address for endpoint. (Defaults to '127.0.0.1')
# Setting this parameter overrides internal_url parameter.
#
# [*admin_protocol*]
# (optional) DEPRECATED: Use admin_url instead.
# Protocol for admin endpoint. (Defaults to 'http')
# Setting this parameter overrides admin_url parameter.
#
# [*admin_address*]
# (optional) DEPRECATED: Use admin_url instead.
# Admin address for endpoint. (Defaults to '127.0.0.1')
# Setting this parameter overrides admin_url parameter.
#
# === Deprecation notes
#
# If any value is provided for public_protocol, public_address or port parameters,
# public_url will be completely ignored. The same applies for internal and admin parameters.
#
# === Examples
#
# class { 'designate::keystone::auth':
@ -110,79 +65,8 @@ class designate::keystone::auth (
$public_url = 'http://127.0.0.1:9001/v1',
$admin_url = 'http://127.0.0.1:9001/v1',
$internal_url = 'http://127.0.0.1:9001/v1',
# DEPRECATED PARAMETERS
$version = undef,
$port = undef,
$public_protocol = undef,
$public_address = undef,
$internal_protocol = undef,
$internal_address = undef,
$admin_protocol = undef,
$admin_address = undef,
) {
if $version {
warning('The version parameter is deprecated, use public_url, internal_url and admin_url instead.')
}
if $port {
warning('The port parameter is deprecated, use public_url, internal_url and admin_url instead.')
}
if $public_protocol {
warning('The public_protocol parameter is deprecated, use public_url instead.')
}
if $internal_protocol {
warning('The internal_protocol parameter is deprecated, use internal_url instead.')
}
if $admin_protocol {
warning('The admin_protocol parameter is deprecated, use admin_url instead.')
}
if $public_address {
warning('The public_address parameter is deprecated, use public_url instead.')
}
if $internal_address {
warning('The internal_address parameter is deprecated, use internal_url instead.')
}
if $admin_address {
warning('The admin_address parameter is deprecated, use admin_url instead.')
}
if ($public_protocol or $public_address or $port or $version) {
$public_url_real = sprintf('%s://%s:%s/%s',
pick($public_protocol, 'http'),
pick($public_address, '127.0.0.1'),
pick($port, '9001'),
pick($version, 'v1'))
} else {
$public_url_real = $public_url
}
if ($admin_protocol or $admin_address or $port or $version) {
$admin_url_real = sprintf('%s://%s:%s/%s',
pick($admin_protocol, 'http'),
pick($admin_address, '127.0.0.1'),
pick($port, '9001'),
pick($version, 'v1'))
} else {
$admin_url_real = $admin_url
}
if ($internal_protocol or $internal_address or $port or $version) {
$internal_url_real = sprintf('%s://%s:%s/%s',
pick($internal_protocol, 'http'),
pick($internal_address, '127.0.0.1'),
pick($port, '9001'),
pick($version, 'v1'))
} else {
$internal_url_real = $internal_url
}
$real_service_name = pick($service_name, $auth_name)
Keystone_user_role["${auth_name}@${tenant}"] ~>
@ -200,9 +84,9 @@ class designate::keystone::auth (
password => $password,
email => $email,
tenant => $tenant,
public_url => $public_url_real,
internal_url => $internal_url_real,
admin_url => $admin_url_real,
public_url => $public_url,
internal_url => $internal_url,
admin_url => $admin_url,
}
}

3
releasenotes/notes/remove_deprecated_keystone_auth_options-e53e50874281e009.yaml Normal file
View File

@ -0,0 +1,3 @@
---
other:
- Removed deprecated options for designate::keystone::auth class

18
spec/classes/designate_keystone_auth_spec.rb
View File

@ -51,24 +51,6 @@ describe 'designate::keystone::auth' do
) }
end
context 'with deprecated endpoint parameters' do
let :params do
{ :password => 'desigpwd',
:public_protocol => 'https',
:public_address => '10.10.10.10',
:port => '81',
:internal_address => '10.10.10.11',
:admin_address => '10.10.10.12' }
end
it { is_expected.to contain_keystone_endpoint('RegionOne/designate::dns').with(
:ensure => 'present',
:public_url => "https://10.10.10.10:81/v1",
:internal_url => "http://10.10.10.11:81/v1",
:admin_url => "http://10.10.10.12:81/v1"
) }
end
context 'when overriding auth name' do
let :params do
{ :password => 'foo',