14528b2640
Change c376d64e5f deprecated
the key_mgr_backend parameter of the cinder::api class, but the logic
to pick up the deprecatd parameter, which was implemented by this
change, was invalid because the cinder class is included before
the cinder::api class is included.
This change removes the invalid implementation and makes that parameter
just ineffective, because we use the cinder::* parameter from t-h-t
and it is unlikely that users would use that puppet-tripleo parameter
directly.
Change-Id: Ie24f82fcede4d2862f9e6a49734959499d9fd845
118 lines
4.1 KiB
Ruby
118 lines
4.1 KiB
Ruby
#
|
|
# Copyright (C) 2016 Red Hat, Inc.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
#
|
|
|
|
require 'spec_helper'
|
|
|
|
describe 'tripleo::profile::base::cinder::api' do
|
|
shared_examples_for 'tripleo::profile::base::cinder::api' do
|
|
before :each do
|
|
facts.merge!({ :step => params[:step] })
|
|
end
|
|
|
|
let(:pre_condition) do
|
|
"class { 'tripleo::profile::base::cinder': step => #{params[:step]}, oslomsg_rpc_hosts => ['127.0.0.1'] }"
|
|
end
|
|
|
|
context 'with step less than 3' do
|
|
let(:params) { { :step => 1 } }
|
|
|
|
it 'should do nothing' do
|
|
is_expected.to contain_class('tripleo::profile::base::cinder::api')
|
|
is_expected.to contain_class('tripleo::profile::base::cinder')
|
|
is_expected.to contain_class('tripleo::profile::base::cinder::authtoken')
|
|
is_expected.to_not contain_class('cinder::api')
|
|
is_expected.to_not contain_class('tripleo::profile::base::apache')
|
|
end
|
|
end
|
|
|
|
context 'with step 3 on bootstrap node' do
|
|
let(:params) { {
|
|
:step => 3,
|
|
:bootstrap_node => 'node.example.com',
|
|
} }
|
|
|
|
it 'should trigger complete configuration' do
|
|
is_expected.to contain_class('tripleo::profile::base::cinder::api')
|
|
is_expected.to contain_class('tripleo::profile::base::cinder')
|
|
is_expected.to contain_class('tripleo::profile::base::cinder::authtoken')
|
|
is_expected.to contain_class('cinder::api').with(
|
|
:sync_db => true,
|
|
)
|
|
is_expected.to contain_class('tripleo::profile::base::apache')
|
|
end
|
|
end
|
|
|
|
context 'with step 3 not on bootstrap node' do
|
|
let(:params) { {
|
|
:step => 3,
|
|
:bootstrap_node => 'other.example.com',
|
|
} }
|
|
|
|
it 'should not trigger any configuration' do
|
|
is_expected.to contain_class('tripleo::profile::base::cinder::api')
|
|
is_expected.to contain_class('tripleo::profile::base::cinder')
|
|
is_expected.to contain_class('tripleo::profile::base::cinder::authtoken')
|
|
is_expected.to_not contain_class('cinder::api')
|
|
is_expected.to_not contain_class('tripleo::profile::base::apache')
|
|
end
|
|
end
|
|
|
|
context 'with step 4 on bootstrap node' do
|
|
let(:params) { {
|
|
:step => 4,
|
|
:bootstrap_node => 'node.example.com',
|
|
} }
|
|
|
|
it 'should trigger complete configuration' do
|
|
is_expected.to contain_class('tripleo::profile::base::cinder::api')
|
|
is_expected.to contain_class('tripleo::profile::base::cinder')
|
|
is_expected.to contain_class('tripleo::profile::base::cinder::authtoken')
|
|
is_expected.to contain_class('cinder::api').with(
|
|
:sync_db => true,
|
|
)
|
|
is_expected.to contain_class('tripleo::profile::base::apache')
|
|
end
|
|
end
|
|
|
|
context 'with step 4 not on bootstrap node' do
|
|
let(:params) { {
|
|
:step => 4,
|
|
:bootstrap_node => 'other.example.com',
|
|
} }
|
|
|
|
it 'should trigger complete configuration but with no db sync' do
|
|
is_expected.to contain_class('tripleo::profile::base::cinder::api')
|
|
is_expected.to contain_class('tripleo::profile::base::cinder')
|
|
is_expected.to contain_class('tripleo::profile::base::cinder::authtoken')
|
|
is_expected.to contain_class('cinder::api').with(
|
|
:sync_db => false,
|
|
)
|
|
is_expected.to contain_class('tripleo::profile::base::apache')
|
|
end
|
|
end
|
|
end
|
|
|
|
on_supported_os.each do |os, facts|
|
|
context "on #{os}" do
|
|
let(:facts) do
|
|
facts.merge({ :hostname => 'node.example.com' })
|
|
end
|
|
|
|
it_behaves_like 'tripleo::profile::base::cinder::api'
|
|
end
|
|
end
|
|
end
|