This patch aim to update our specs test in order to work with the rspec-puppet release 2.0.0, in the mean time, we update rspec syntax in order to be prepared for rspec 3.x move. In details: * Use shared_examples "a Puppet::Error" for puppet::error tests * Convert 'should' keyword to 'is_expected.to' (prepare rspec 3.x) * Fix spec tests for rspec-puppet 2.0.0 * Upgrade and pin rspec-puppet from 1.0.1 to 2.0.0 * Clean Gemfile (remove over-specificication of runtime deps of puppetlabs_spec_helper) Change-Id: Id5b428fb518f40cf92cd27078d36f19b6d60226b Card: https://trello.com/c/eHXc1Ryd/4-investigate-the-necessary-change-to-be-rspec-puppet-2-0-0-compliant
336 lines
13 KiB
Ruby
336 lines
13 KiB
Ruby
require 'spec_helper'
|
|
describe 'cinder' do
|
|
let :req_params do
|
|
{:rabbit_password => 'guest', :database_connection => 'mysql://user:password@host/database'}
|
|
end
|
|
|
|
let :facts do
|
|
{:osfamily => 'Debian'}
|
|
end
|
|
|
|
describe 'with only required params' do
|
|
let :params do
|
|
req_params
|
|
end
|
|
|
|
it { is_expected.to contain_class('cinder::params') }
|
|
it { is_expected.to contain_class('mysql::bindings::python') }
|
|
|
|
it 'should contain default config' do
|
|
is_expected.to contain_cinder_config('DEFAULT/rpc_backend').with(:value => 'cinder.openstack.common.rpc.impl_kombu')
|
|
is_expected.to contain_cinder_config('DEFAULT/control_exchange').with(:value => 'openstack')
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_password').with(:value => 'guest', :secret => true)
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_host').with(:value => '127.0.0.1')
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_port').with(:value => '5672')
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_hosts').with(:value => '127.0.0.1:5672')
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_ha_queues').with(:value => false)
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_virtual_host').with(:value => '/')
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_userid').with(:value => 'guest')
|
|
is_expected.to contain_cinder_config('database/connection').with(:value => 'mysql://user:password@host/database', :secret => true)
|
|
is_expected.to contain_cinder_config('database/idle_timeout').with(:value => '3600')
|
|
is_expected.to contain_cinder_config('database/min_pool_size').with(:value => '1')
|
|
is_expected.to contain_cinder_config('database/max_pool_size').with_ensure('absent')
|
|
is_expected.to contain_cinder_config('database/max_retries').with(:value => '10')
|
|
is_expected.to contain_cinder_config('database/retry_interval').with(:value => '10')
|
|
is_expected.to contain_cinder_config('database/max_overflow').with_ensure('absent')
|
|
is_expected.to contain_cinder_config('DEFAULT/verbose').with(:value => false)
|
|
is_expected.to contain_cinder_config('DEFAULT/debug').with(:value => false)
|
|
is_expected.to contain_cinder_config('DEFAULT/storage_availability_zone').with(:value => 'nova')
|
|
is_expected.to contain_cinder_config('DEFAULT/default_availability_zone').with(:value => 'nova')
|
|
is_expected.to contain_cinder_config('DEFAULT/api_paste_config').with(:value => '/etc/cinder/api-paste.ini')
|
|
is_expected.to contain_cinder_config('DEFAULT/log_dir').with(:value => '/var/log/cinder')
|
|
end
|
|
|
|
it { is_expected.to contain_file('/etc/cinder/cinder.conf').with(
|
|
:owner => 'cinder',
|
|
:group => 'cinder',
|
|
:mode => '0600',
|
|
:require => 'Package[cinder]'
|
|
) }
|
|
|
|
it { is_expected.to contain_file('/etc/cinder/api-paste.ini').with(
|
|
:owner => 'cinder',
|
|
:group => 'cinder',
|
|
:mode => '0600',
|
|
:require => 'Package[cinder]'
|
|
) }
|
|
|
|
end
|
|
describe 'with modified rabbit_hosts' do
|
|
let :params do
|
|
req_params.merge({'rabbit_hosts' => ['rabbit1:5672', 'rabbit2:5672']})
|
|
end
|
|
|
|
it 'should contain many' do
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_host').with(:value => nil)
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_port').with(:value => nil)
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_hosts').with(:value => 'rabbit1:5672,rabbit2:5672')
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_ha_queues').with(:value => true)
|
|
end
|
|
end
|
|
|
|
describe 'with a single rabbit_hosts entry' do
|
|
let :params do
|
|
req_params.merge({'rabbit_hosts' => ['rabbit1:5672']})
|
|
end
|
|
|
|
it 'should contain many' do
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_host').with(:value => nil)
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_port').with(:value => nil)
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_hosts').with(:value => 'rabbit1:5672')
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_ha_queues').with(:value => true)
|
|
end
|
|
end
|
|
|
|
describe 'with qpid rpc supplied' do
|
|
|
|
let :params do
|
|
{
|
|
:database_connection => 'mysql://user:password@host/database',
|
|
:qpid_password => 'guest',
|
|
:rpc_backend => 'cinder.openstack.common.rpc.impl_qpid'
|
|
}
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('database/connection').with_value('mysql://user:password@host/database') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/rpc_backend').with_value('cinder.openstack.common.rpc.impl_qpid') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_hostname').with_value('localhost') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_port').with_value('5672') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_username').with_value('guest') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_password').with_value('guest').with_secret(true) }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_reconnect').with_value(true) }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_reconnect_timeout').with_value('0') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_reconnect_limit').with_value('0') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_reconnect_interval_min').with_value('0') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_reconnect_interval_max').with_value('0') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_reconnect_interval').with_value('0') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_heartbeat').with_value('60') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_protocol').with_value('tcp') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_tcp_nodelay').with_value(true) }
|
|
end
|
|
|
|
describe 'with qpid rpc and no qpid_sasl_mechanisms' do
|
|
let :params do
|
|
{
|
|
:database_connection => 'mysql://user:password@host/database',
|
|
:qpid_password => 'guest',
|
|
:rpc_backend => 'cinder.openstack.common.rpc.impl_qpid'
|
|
}
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_sasl_mechanisms').with_ensure('absent') }
|
|
end
|
|
|
|
describe 'with qpid rpc and qpid_sasl_mechanisms string' do
|
|
let :params do
|
|
{
|
|
:database_connection => 'mysql://user:password@host/database',
|
|
:qpid_password => 'guest',
|
|
:qpid_sasl_mechanisms => 'PLAIN',
|
|
:rpc_backend => 'cinder.openstack.common.rpc.impl_qpid'
|
|
}
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_sasl_mechanisms').with_value('PLAIN') }
|
|
end
|
|
|
|
describe 'with qpid rpc and qpid_sasl_mechanisms array' do
|
|
let :params do
|
|
{
|
|
:database_connection => 'mysql://user:password@host/database',
|
|
:qpid_password => 'guest',
|
|
:qpid_sasl_mechanisms => [ 'DIGEST-MD5', 'GSSAPI', 'PLAIN' ],
|
|
:rpc_backend => 'cinder.openstack.common.rpc.impl_qpid'
|
|
}
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('DEFAULT/qpid_sasl_mechanisms').with_value('DIGEST-MD5 GSSAPI PLAIN') }
|
|
end
|
|
|
|
describe 'with SSL enabled with kombu' do
|
|
let :params do
|
|
req_params.merge!({
|
|
:rabbit_use_ssl => true,
|
|
:kombu_ssl_ca_certs => '/path/to/ssl/ca/certs',
|
|
:kombu_ssl_certfile => '/path/to/ssl/cert/file',
|
|
:kombu_ssl_keyfile => '/path/to/ssl/keyfile',
|
|
:kombu_ssl_version => 'TLSv1'
|
|
})
|
|
end
|
|
|
|
it do
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_use_ssl').with_value('true')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_ca_certs').with_value('/path/to/ssl/ca/certs')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_certfile').with_value('/path/to/ssl/cert/file')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_keyfile').with_value('/path/to/ssl/keyfile')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_version').with_value('TLSv1')
|
|
end
|
|
end
|
|
|
|
describe 'with SSL enabled without kombu' do
|
|
let :params do
|
|
req_params.merge!({
|
|
:rabbit_use_ssl => true,
|
|
})
|
|
end
|
|
|
|
it do
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_use_ssl').with_value('true')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_ca_certs').with_ensure('absent')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_certfile').with_ensure('absent')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_keyfile').with_ensure('absent')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_version').with_value('TLSv1')
|
|
end
|
|
end
|
|
|
|
describe 'with SSL disabled' do
|
|
let :params do
|
|
req_params.merge!({
|
|
:rabbit_use_ssl => false,
|
|
:kombu_ssl_ca_certs => 'undef',
|
|
:kombu_ssl_certfile => 'undef',
|
|
:kombu_ssl_keyfile => 'undef',
|
|
:kombu_ssl_version => 'TLSv1'
|
|
})
|
|
end
|
|
|
|
it do
|
|
is_expected.to contain_cinder_config('DEFAULT/rabbit_use_ssl').with_value('false')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_ca_certs').with_ensure('absent')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_certfile').with_ensure('absent')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_keyfile').with_ensure('absent')
|
|
is_expected.to contain_cinder_config('DEFAULT/kombu_ssl_version').with_ensure('absent')
|
|
end
|
|
end
|
|
|
|
describe 'with syslog disabled' do
|
|
let :params do
|
|
req_params
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('DEFAULT/use_syslog').with_value(false) }
|
|
end
|
|
|
|
describe 'with syslog enabled' do
|
|
let :params do
|
|
req_params.merge({
|
|
:use_syslog => 'true',
|
|
})
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('DEFAULT/use_syslog').with_value(true) }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/syslog_log_facility').with_value('LOG_USER') }
|
|
end
|
|
|
|
describe 'with syslog enabled and custom settings' do
|
|
let :params do
|
|
req_params.merge({
|
|
:use_syslog => 'true',
|
|
:log_facility => 'LOG_LOCAL0'
|
|
})
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('DEFAULT/use_syslog').with_value(true) }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/syslog_log_facility').with_value('LOG_LOCAL0') }
|
|
end
|
|
|
|
describe 'with log_dir disabled' do
|
|
let(:params) { req_params.merge!({:log_dir => false}) }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/log_dir').with_ensure('absent') }
|
|
end
|
|
|
|
describe 'with amqp_durable_queues disabled' do
|
|
let :params do
|
|
req_params
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('DEFAULT/amqp_durable_queues').with_value(false) }
|
|
end
|
|
|
|
describe 'with amqp_durable_queues enabled' do
|
|
let :params do
|
|
req_params.merge({
|
|
:amqp_durable_queues => true,
|
|
})
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('DEFAULT/amqp_durable_queues').with_value(true) }
|
|
end
|
|
|
|
describe 'with postgresql' do
|
|
let :params do
|
|
{
|
|
:database_connection => 'postgresql://user:drowssap@host/database',
|
|
:rabbit_password => 'guest',
|
|
}
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('database/connection').with(
|
|
:value => 'postgresql://user:drowssap@host/database',
|
|
:secret => true
|
|
) }
|
|
it { is_expected.to_not contain_class('mysql::python') }
|
|
it { is_expected.to_not contain_class('mysql::bindings') }
|
|
it { is_expected.to_not contain_class('mysql::bindings::python') }
|
|
end
|
|
|
|
describe 'with SSL socket options set' do
|
|
let :params do
|
|
{
|
|
:use_ssl => true,
|
|
:cert_file => '/path/to/cert',
|
|
:ca_file => '/path/to/ca',
|
|
:key_file => '/path/to/key',
|
|
:rabbit_password => 'guest',
|
|
}
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('DEFAULT/ssl_ca_file').with_value('/path/to/ca') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/ssl_cert_file').with_value('/path/to/cert') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/ssl_key_file').with_value('/path/to/key') }
|
|
end
|
|
|
|
describe 'with SSL socket options set to false' do
|
|
let :params do
|
|
{
|
|
:use_ssl => false,
|
|
:cert_file => false,
|
|
:ca_file => false,
|
|
:key_file => false,
|
|
:rabbit_password => 'guest',
|
|
}
|
|
end
|
|
|
|
it { is_expected.to contain_cinder_config('DEFAULT/ssl_ca_file').with_ensure('absent') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/ssl_cert_file').with_ensure('absent') }
|
|
it { is_expected.to contain_cinder_config('DEFAULT/ssl_key_file').with_ensure('absent') }
|
|
end
|
|
|
|
describe 'with SSL socket options set wrongly configured' do
|
|
let :params do
|
|
{
|
|
:use_ssl => true,
|
|
:ca_file => '/path/to/ca',
|
|
:key_file => '/path/to/key',
|
|
:rabbit_password => 'guest',
|
|
}
|
|
end
|
|
|
|
it_raises 'a Puppet::Error', /The cert_file parameter is required when use_ssl is set to true/
|
|
end
|
|
|
|
describe 'with APIs set for Kilo (proposed)' do
|
|
let :params do
|
|
{
|
|
:enable_v1_api => false,
|
|
:enable_v2_api => true,
|
|
:rabbit_password => 'guest',
|
|
}
|
|
end
|
|
|
|
it { should contain_cinder_config('DEFAULT/enable_v1_api').with_value(false) }
|
|
it { should contain_cinder_config('DEFAULT/enable_v2_api').with_value(true) }
|
|
|
|
end
|
|
end
|