e7ed2b8f9c
keystone_auth_ui and keystone_identity_uri parameters are inconsistent with other modules, change keystone_auth_ui,keystone_identity_uri to auth_uri,identity_uri Change-Id: I0c606b62678c3c293dc0e15c47825909f0c64ee0 Closes-Bug: #1532920
197 lines
6.8 KiB
Ruby
197 lines
6.8 KiB
Ruby
require 'spec_helper'
|
|
|
|
describe 'ceilometer::api' do
|
|
|
|
let :pre_condition do
|
|
"class { 'ceilometer': metering_secret => 's3cr3t' }
|
|
include ::ceilometer::db"
|
|
end
|
|
|
|
let :params do
|
|
{ :enabled => true,
|
|
:manage_service => true,
|
|
:keystone_user => 'ceilometer',
|
|
:keystone_password => 'ceilometer-passw0rd',
|
|
:keystone_tenant => 'services',
|
|
:host => '0.0.0.0',
|
|
:port => '8777',
|
|
:package_ensure => 'latest',
|
|
}
|
|
end
|
|
|
|
shared_examples_for 'ceilometer-api' do
|
|
|
|
context 'without required parameter keystone_password' do
|
|
before { params.delete(:keystone_password) }
|
|
it { expect { is_expected.to raise_error(Puppet::Error) } }
|
|
end
|
|
|
|
it { is_expected.to contain_class('ceilometer::params') }
|
|
it { is_expected.to contain_class('ceilometer::policy') }
|
|
|
|
it 'installs ceilometer-api package' do
|
|
is_expected.to contain_package('ceilometer-api').with(
|
|
:ensure => 'latest',
|
|
:name => platform_params[:api_package_name],
|
|
:tag => ['openstack', 'ceilometer-package'],
|
|
)
|
|
end
|
|
|
|
it 'configures keystone authentication middleware' do
|
|
is_expected.to contain_ceilometer_config('keystone_authtoken/admin_tenant_name').with_value( params[:keystone_tenant] )
|
|
is_expected.to contain_ceilometer_config('keystone_authtoken/admin_user').with_value( params[:keystone_user] )
|
|
is_expected.to contain_ceilometer_config('keystone_authtoken/admin_password').with_value( params[:keystone_password] )
|
|
is_expected.to contain_ceilometer_config('keystone_authtoken/admin_password').with_value( params[:keystone_password] ).with_secret(true)
|
|
is_expected.to contain_ceilometer_config('keystone_authtoken/auth_uri').with_value("http://127.0.0.1:5000/")
|
|
is_expected.to contain_ceilometer_config('keystone_authtoken/identity_uri').with_value("http://127.0.0.1:35357/")
|
|
is_expected.to contain_ceilometer_config('api/host').with_value( params[:host] )
|
|
is_expected.to contain_ceilometer_config('api/port').with_value( params[:port] )
|
|
is_expected.to contain_ceilometer_config('api/workers').with_value('<SERVICE DEFAULT>')
|
|
end
|
|
|
|
[{:enabled => true}, {:enabled => false}].each do |param_hash|
|
|
context "when service should be #{param_hash[:enabled] ? 'enabled' : 'disabled'}" do
|
|
before do
|
|
params.merge!(param_hash)
|
|
end
|
|
|
|
it 'configures ceilometer-api service' do
|
|
is_expected.to contain_service('ceilometer-api').with(
|
|
:ensure => (params[:manage_service] && params[:enabled]) ? 'running' : 'stopped',
|
|
:name => platform_params[:api_service_name],
|
|
:enable => params[:enabled],
|
|
:hasstatus => true,
|
|
:hasrestart => true,
|
|
:require => 'Class[Ceilometer::Db]',
|
|
:tag => 'ceilometer-service',
|
|
)
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'with disabled service managing' do
|
|
before do
|
|
params.merge!({
|
|
:manage_service => false,
|
|
:enabled => false })
|
|
end
|
|
|
|
it 'configures ceilometer-api service' do
|
|
is_expected.to contain_service('ceilometer-api').with(
|
|
:ensure => nil,
|
|
:name => platform_params[:api_service_name],
|
|
:enable => false,
|
|
:hasstatus => true,
|
|
:hasrestart => true,
|
|
:tag => 'ceilometer-service',
|
|
)
|
|
end
|
|
end
|
|
|
|
context 'when running ceilometer-api in wsgi' do
|
|
before do
|
|
params.merge!({ :service_name => 'httpd' })
|
|
end
|
|
|
|
let :pre_condition do
|
|
"include ::apache
|
|
include ::ceilometer::db
|
|
class { 'ceilometer': metering_secret => 's3cr3t' }"
|
|
end
|
|
|
|
it 'configures ceilometer-api service with Apache' do
|
|
is_expected.to contain_service('ceilometer-api').with(
|
|
:ensure => 'stopped',
|
|
:name => platform_params[:api_service_name],
|
|
:enable => false,
|
|
:tag => 'ceilometer-service',
|
|
)
|
|
end
|
|
end
|
|
|
|
context 'when service_name is not valid' do
|
|
before do
|
|
params.merge!({ :service_name => 'foobar' })
|
|
end
|
|
|
|
let :pre_condition do
|
|
"include ::apache
|
|
include ::ceilometer::db
|
|
class { 'ceilometer': metering_secret => 's3cr3t' }"
|
|
end
|
|
|
|
it_raises 'a Puppet::Error', /Invalid service_name/
|
|
end
|
|
end
|
|
|
|
context 'on Debian platforms' do
|
|
let :facts do
|
|
@default_facts.merge({ :osfamily => 'Debian',
|
|
:operatingsystem => 'Debian',
|
|
:operatingsystemrelease => '8.0',
|
|
:concat_basedir => '/var/lib/puppet/concat',
|
|
:fqdn => 'some.host.tld',
|
|
:processorcount => 2 })
|
|
end
|
|
|
|
let :platform_params do
|
|
{ :api_package_name => 'ceilometer-api',
|
|
:api_service_name => 'ceilometer-api' }
|
|
end
|
|
|
|
it_configures 'ceilometer-api'
|
|
end
|
|
|
|
context 'on RedHat platforms' do
|
|
let :facts do
|
|
@default_facts.merge({ :osfamily => 'RedHat',
|
|
:operatingsystem => 'RedHat',
|
|
:operatingsystemrelease => '7.1',
|
|
:operatingsystemmajrelease => '7',
|
|
:fqdn => 'some.host.tld',
|
|
:concat_basedir => '/var/lib/puppet/concat',
|
|
:processorcount => 2 })
|
|
end
|
|
|
|
let :platform_params do
|
|
{ :api_package_name => 'openstack-ceilometer-api',
|
|
:api_service_name => 'openstack-ceilometer-api' }
|
|
end
|
|
|
|
it_configures 'ceilometer-api'
|
|
end
|
|
|
|
describe "with deprecated custom keystone_identity_uri and keystone_auth_uri" do
|
|
let :facts do
|
|
@default_facts.merge({ :osfamily => 'RedHat' })
|
|
end
|
|
before do
|
|
params.merge!({
|
|
:keystone_identity_uri => 'https://foo.bar:35357/',
|
|
:keystone_auth_uri => 'https://foo.bar:5000/',
|
|
})
|
|
end
|
|
it 'configures identity_uri and auth_uri but deprecates old auth settings' do
|
|
is_expected.to contain_ceilometer_config('keystone_authtoken/identity_uri').with_value("https://foo.bar:35357/");
|
|
is_expected.to contain_ceilometer_config('keystone_authtoken/auth_uri').with_value("https://foo.bar:5000/");
|
|
end
|
|
end
|
|
|
|
describe "with custom keystone identity_uri and auth_uri" do
|
|
let :facts do
|
|
@default_facts.merge({ :osfamily => 'RedHat' })
|
|
end
|
|
before do
|
|
params.merge!({
|
|
:identity_uri => 'https://foo.bar:35357/',
|
|
:auth_uri => 'https://foo.bar:5000/',
|
|
})
|
|
end
|
|
it 'configures identity_uri and auth_uri but deprecates old auth settings' do
|
|
is_expected.to contain_ceilometer_config('keystone_authtoken/identity_uri').with_value("https://foo.bar:35357/");
|
|
is_expected.to contain_ceilometer_config('keystone_authtoken/auth_uri').with_value("https://foo.bar:5000/");
|
|
end
|
|
end
|
|
|
|
end
|