2850def15c
Previously even if manage_service was set to false, we were always attempting to define the octavia-api service within puppet. This change updates the manage_service to control skipping the definition of the octavia-api service. Co-Authored-By: Alex Schultz <aschultz@redhat.com> Change-Id: I6d2729b3e22f4c434da098ea180a9d11cf73058b Closes-Bug: 1633001
122 lines
3.7 KiB
Ruby
122 lines
3.7 KiB
Ruby
require 'spec_helper'
|
|
|
|
describe 'octavia::api' do
|
|
|
|
let :pre_condition do
|
|
"class { 'octavia': }
|
|
include ::octavia::db
|
|
class { '::octavia::keystone::authtoken':
|
|
password => 'password';
|
|
}
|
|
"
|
|
end
|
|
|
|
let :params do
|
|
{ :enabled => true,
|
|
:manage_service => true,
|
|
:package_ensure => 'latest',
|
|
:port => '9876',
|
|
:host => '0.0.0.0',
|
|
}
|
|
end
|
|
|
|
shared_examples_for 'octavia-api' do
|
|
|
|
it { is_expected.to contain_class('octavia::deps') }
|
|
it { is_expected.to contain_class('octavia::params') }
|
|
it { is_expected.to contain_class('octavia::policy') }
|
|
it { is_expected.to contain_class('octavia::keystone::authtoken') }
|
|
|
|
it 'installs octavia-api package' do
|
|
is_expected.to contain_package('octavia-api').with(
|
|
:ensure => 'latest',
|
|
:name => platform_params[:api_package_name],
|
|
:tag => ['openstack', 'octavia-package'],
|
|
)
|
|
end
|
|
|
|
context 'when not parameters are defined' do
|
|
before do
|
|
params.clear()
|
|
end
|
|
it 'configures with default values' do
|
|
is_expected.to contain_octavia_config('DEFAULT/host').with_value( '0.0.0.0' )
|
|
is_expected.to contain_octavia_config('DEFAULT/port').with_value( '9876' )
|
|
is_expected.to contain_octavia_config('DEFAULT/auth_strategy').with_value( 'keystone' )
|
|
end
|
|
it 'does not sync the database' do
|
|
is_expected.not_to contain_class('octavia::db::sync')
|
|
end
|
|
end
|
|
|
|
it 'configures bind_host and bind_port' do
|
|
is_expected.to contain_octavia_config('DEFAULT/host').with_value( params[:host] )
|
|
is_expected.to contain_octavia_config('DEFAULT/port').with_value( params[:port] )
|
|
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 octavia-api service' do
|
|
is_expected.to contain_service('octavia-api').with(
|
|
:ensure => (params[:manage_service] && params[:enabled]) ? 'running' : 'stopped',
|
|
:name => platform_params[:api_service_name],
|
|
:enable => params[:enabled],
|
|
:hasstatus => true,
|
|
:hasrestart => true,
|
|
:tag => ['octavia-service', 'octavia-db-sync-service'],
|
|
)
|
|
end
|
|
it { is_expected.to contain_service('octavia-api').that_subscribes_to('Anchor[octavia::service::begin]')}
|
|
it { is_expected.to contain_service('octavia-api').that_notifies('Anchor[octavia::service::end]')}
|
|
end
|
|
end
|
|
|
|
context 'with sync_db set to true' do
|
|
before do
|
|
params.merge!({
|
|
:sync_db => true})
|
|
end
|
|
it { is_expected.to contain_class('octavia::db::sync') }
|
|
end
|
|
|
|
context 'with disabled service managing' do
|
|
before do
|
|
params.merge!({
|
|
:manage_service => false,
|
|
:enabled => false })
|
|
end
|
|
|
|
it 'configures octavia-api service' do
|
|
is_expected.to_not contain_service('octavia-api')
|
|
end
|
|
end
|
|
|
|
end
|
|
|
|
on_supported_os({
|
|
:supported_os => OSDefaults.get_supported_os
|
|
}).each do |os,facts|
|
|
context "on #{os}" do
|
|
let (:facts) do
|
|
facts.merge!(OSDefaults.get_facts())
|
|
end
|
|
let(:platform_params) do
|
|
case facts[:osfamily]
|
|
when 'Debian'
|
|
{ :api_package_name => 'octavia-api',
|
|
:api_service_name => 'octavia-api' }
|
|
when 'RedHat'
|
|
{ :api_package_name => 'openstack-octavia-api',
|
|
:api_service_name => 'octavia-api' }
|
|
end
|
|
end
|
|
it_behaves_like 'octavia-api'
|
|
end
|
|
end
|
|
|
|
end
|