3e50ac09ee
The code matching the existing endpoints did not take the region into account. This was giving random results and messing up the catalog badly. This code fix it this and add associated rspec checks. Closes-Bug: #1535939 Change-Id: If1cdf30c37194b3a7b08bf85860cf7fb7266f6e1
189 lines
5.8 KiB
Ruby
189 lines
5.8 KiB
Ruby
#
|
|
# Copyright (C) 2014 eNovance SAS <licensing@enovance.com>
|
|
#
|
|
# Author: Emilien Macchi <emilien.macchi@enovance.com>
|
|
#
|
|
# 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 'keystone::resource::service_identity' do
|
|
|
|
let (:title) { 'neutron' }
|
|
|
|
let :required_params do
|
|
{ :password => 'secrete',
|
|
:service_type => 'network',
|
|
:admin_url => 'http://192.168.0.1:9696',
|
|
:internal_url => 'http://10.0.0.1:9696',
|
|
:public_url => 'http://7.7.7.7:9696' }
|
|
end
|
|
|
|
shared_examples 'keystone::resource::service_identity examples' do
|
|
|
|
context 'with only required parameters' do
|
|
let :params do
|
|
required_params
|
|
end
|
|
|
|
it { is_expected.to contain_keystone_user(title).with(
|
|
:ensure => 'present',
|
|
:password => 'secrete',
|
|
:email => 'neutron@localhost',
|
|
)}
|
|
|
|
it { is_expected.to contain_keystone_user_role("#{title}@services").with(
|
|
:ensure => 'present',
|
|
:roles => ['admin'],
|
|
)}
|
|
|
|
it { is_expected.to contain_keystone_service("#{title}::network").with(
|
|
:ensure => 'present',
|
|
:description => 'neutron service',
|
|
)}
|
|
|
|
it { is_expected.to contain_keystone_endpoint("RegionOne/#{title}::network").with(
|
|
:ensure => 'present',
|
|
:public_url => 'http://7.7.7.7:9696',
|
|
:internal_url => 'http://10.0.0.1:9696',
|
|
:admin_url => 'http://192.168.0.1:9696',
|
|
:region => 'RegionOne',
|
|
)}
|
|
end
|
|
|
|
context 'when explicitly setting an region' do
|
|
let :params do
|
|
required_params.merge(
|
|
:region => 'East',
|
|
)
|
|
end
|
|
it { is_expected.to contain_keystone_endpoint("East/#{title}::network").with(
|
|
:ensure => 'present',
|
|
:public_url => 'http://7.7.7.7:9696',
|
|
:internal_url => 'http://10.0.0.1:9696',
|
|
:admin_url => 'http://192.168.0.1:9696',
|
|
:region => 'East',
|
|
)}
|
|
end
|
|
|
|
context 'when trying to create an endpoint without service_type (will be dropped in Mitaka)' do
|
|
let :params do
|
|
required_params.merge(
|
|
:configure_service => false,
|
|
:service_type => false,
|
|
)
|
|
end
|
|
it { is_expected.to contain_keystone_endpoint("RegionOne/#{title}").with(
|
|
:ensure => 'present',
|
|
:public_url => 'http://7.7.7.7:9696',
|
|
:internal_url => 'http://10.0.0.1:9696',
|
|
:admin_url => 'http://192.168.0.1:9696',
|
|
)}
|
|
end
|
|
|
|
context 'when trying to create a service without service_type' do
|
|
let :params do
|
|
required_params.delete(:service_type)
|
|
required_params
|
|
end
|
|
it_raises 'a Puppet::Error', /When configuring a service, you need to set the service_type parameter/
|
|
end
|
|
|
|
context 'when trying to create an endpoint without url' do
|
|
let :params do
|
|
required_params.delete(:public_url)
|
|
required_params
|
|
end
|
|
it_raises 'a Puppet::Error', /When configuring an endpoint, you need to set the _url parameters/
|
|
end
|
|
|
|
context 'with user domain' do
|
|
let :params do
|
|
required_params.merge({:user_domain => 'userdomain'})
|
|
end
|
|
it { is_expected.to contain_keystone_domain('userdomain').with(
|
|
:ensure => 'present',
|
|
)}
|
|
it { is_expected.to contain_keystone_user(title).with(
|
|
:ensure => 'present',
|
|
:password => 'secrete',
|
|
:email => 'neutron@localhost',
|
|
:domain => 'userdomain',
|
|
)}
|
|
it { is_expected.to contain_keystone_user_role("#{title}@services").with(
|
|
:ensure => 'present',
|
|
:roles => ['admin'],
|
|
)}
|
|
end
|
|
|
|
context 'with user and project domain' do
|
|
let :params do
|
|
required_params.merge({
|
|
:user_domain => 'userdomain',
|
|
:project_domain => 'projdomain',
|
|
})
|
|
end
|
|
it { is_expected.to contain_keystone_user(title).with(
|
|
:ensure => 'present',
|
|
:password => 'secrete',
|
|
:email => 'neutron@localhost',
|
|
:domain => 'userdomain',
|
|
)}
|
|
it { is_expected.to contain_keystone_domain('userdomain').with(
|
|
:ensure => 'present',
|
|
)}
|
|
it { is_expected.to contain_keystone_user_role("#{title}@services").with(
|
|
:ensure => 'present',
|
|
:roles => ['admin'],
|
|
)}
|
|
end
|
|
context 'with default domain only' do
|
|
let :params do
|
|
required_params.merge({
|
|
:default_domain => 'defaultdomain',
|
|
})
|
|
end
|
|
it { is_expected.to contain_keystone_user(title).with(
|
|
:ensure => 'present',
|
|
:password => 'secrete',
|
|
:email => 'neutron@localhost',
|
|
:domain => 'defaultdomain',
|
|
)}
|
|
it { is_expected.to contain_keystone_domain('defaultdomain').with(
|
|
:ensure => 'present',
|
|
)}
|
|
it { is_expected.to contain_keystone_user_role("#{title}@services").with(
|
|
:ensure => 'present',
|
|
:roles => ['admin'],
|
|
)}
|
|
end
|
|
|
|
end
|
|
|
|
context 'on a Debian osfamily' do
|
|
let :facts do
|
|
@default_facts.merge({ :osfamily => "Debian" })
|
|
end
|
|
|
|
include_examples 'keystone::resource::service_identity examples'
|
|
end
|
|
|
|
context 'on a RedHat osfamily' do
|
|
let :facts do
|
|
@default_facts.merge({ :osfamily => 'RedHat' })
|
|
end
|
|
|
|
include_examples 'keystone::resource::service_identity examples'
|
|
end
|
|
end
|