Move horizon to step 3

We configure apache in step 3 so horizon should be configured at the
same time or else updates will cause horizon to be unvailable during the
update process.

Change-Id: I4032f7c24edc0ff9ed637e213870cdd3beb9a54e
Closes-Bug: #1678338
(cherry picked from commit e292871741)
changes/77/452977/1
Alex Schultz 6 years ago
parent ddc867f134
commit f206c2c431
  1. 2
      manifests/profile/base/horizon.pp
  2. 57
      spec/classes/tripleo_profile_base_horizon_spec.rb
  3. 1
      spec/fixtures/hieradata/default.yaml

@ -26,7 +26,7 @@
class tripleo::profile::base::horizon (
$step = hiera('step'),
) {
if $step >= 4 {
if $step >= 3 {
# Horizon
include ::apache::mod::remoteip
include ::apache::mod::status

@ -0,0 +1,57 @@
#
# Copyright (C) 2017 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::horizon' do
shared_examples_for 'tripleo::profile::base::horizon' do
let(:pre_condition) do
"class { '::tripleo::profile::base::aodh': step => #{params[:step]} }"
end
context 'with step less than 3' do
let(:params) { { :step => 2 } }
it 'should do nothing' do
is_expected.to contain_class('tripleo::profile::base::horizon')
is_expected.to_not contain_class('horizon')
end
end
context 'with step 3' do
let(:params) { {
:step => 3,
} }
it 'should trigger complete configuration' do
is_expected.to contain_class('horizon')
is_expected.to contain_class('apache::mod::remoteip')
is_expected.to contain_class('apache::mod::status')
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::horizon'
end
end
end

@ -1,3 +1,4 @@
my_hash:
network: '127.0.0.1'
not_hash: string
horizon::secret_key: 'secrete'

Loading…
Cancel
Save