diff --git a/manifests/proxy/s3token.pp b/manifests/proxy/s3token.pp index 86edcc6c..dbe32290 100644 --- a/manifests/proxy/s3token.pp +++ b/manifests/proxy/s3token.pp @@ -4,18 +4,6 @@ # # === Parameters # -# [*auth_host*] -# (optional) The keystone host -# Defaults to undef. -# -# [*auth_port*] -# (optional) The Keystone client API port -# Defaults to undef. -# -# [*auth_protocol*] -# (optional) http or https -# Defaults to undef. -# # [*auth_uri*] # (optional) The Keystone server uri # Defaults to http://127.0.0.1:5000 @@ -90,9 +78,6 @@ # Copyright 2012 eNovance licensing@enovance.com # class swift::proxy::s3token( - $auth_host = undef, - $auth_port = undef, - $auth_protocol = undef, $auth_uri = 'http://127.0.0.1:5000', $reseller_prefix = 'AUTH_', $delay_auth_decision = false, @@ -109,13 +94,6 @@ class swift::proxy::s3token( include swift::deps - if $auth_host and $auth_port and $auth_protocol { - warning('Use of the auth_host, auth_port, and auth_protocol options have been deprecated in favor of auth_uri.') - $auth_uri_real = "${auth_protocol}://${auth_host}:${auth_port}" - } else { - $auth_uri_real = $auth_uri - } - if $password == undef { warning('Usage of the default password is deprecated and will be removed in a future release. \ Please set password parameter') @@ -126,7 +104,7 @@ Please set password parameter') swift_proxy_config { 'filter:s3token/use': value => 'egg:swift#s3token'; - 'filter:s3token/auth_uri': value => $auth_uri_real; + 'filter:s3token/auth_uri': value => $auth_uri; 'filter:s3token/reseller_prefix': value => $reseller_prefix; 'filter:s3token/delay_auth_decision': value => $delay_auth_decision; 'filter:s3token/http_timeout': value => $http_timeout; diff --git a/releasenotes/notes/remove-s3token-old-auth-params-1e7a68fa41e7a222.yaml b/releasenotes/notes/remove-s3token-old-auth-params-1e7a68fa41e7a222.yaml new file mode 100644 index 00000000..b16f4bab --- /dev/null +++ b/releasenotes/notes/remove-s3token-old-auth-params-1e7a68fa41e7a222.yaml @@ -0,0 +1,9 @@ +--- +upgrade: + - | + The following three deprecated parameters have been removed from + the ``swift::proxy::s3token`` class. + + - ``auth_host`` + - ``auth_port`` + - ``auth_protocol`` diff --git a/spec/classes/swift_proxy_s3token_spec.rb b/spec/classes/swift_proxy_s3token_spec.rb index 26ab29c1..6622c928 100644 --- a/spec/classes/swift_proxy_s3token_spec.rb +++ b/spec/classes/swift_proxy_s3token_spec.rb @@ -19,18 +19,6 @@ describe 'swift::proxy::s3token' do end - describe "when overriding default parameters" do - let :params do - { - :auth_protocol => 'https', - :auth_host => '192.168.4.2', - :auth_port => '3452' - } - end - - it { is_expected.to contain_swift_proxy_config('filter:s3token/auth_uri').with_value('https://192.168.4.2:3452') } - end - describe "when overriding default parameters" do let :params do {