metadata: Remove unnecessary class inclusions
The nova::metadata class contains only resources to manage options and should be used along with the nova::api class to launch the service. Thus we don't have to include the keystoneauth class or the db class from this class because these are included by the api class. Change-Id: I457f6b31cee4653358343e71b1e1c96ca7fdabe7
This commit is contained in:
parent
1c2f147c2a
commit
900266e84f
@ -36,8 +36,6 @@ class nova::metadata(
|
||||
) inherits nova::params {
|
||||
|
||||
include nova::deps
|
||||
include nova::db
|
||||
include nova::keystone::authtoken
|
||||
|
||||
nova_config {
|
||||
'api/dhcp_domain': value => $dhcp_domain;
|
||||
|
@ -2,13 +2,6 @@ require 'spec_helper'
|
||||
|
||||
describe 'nova::metadata' do
|
||||
|
||||
let :pre_condition do
|
||||
"include nova
|
||||
class { 'nova::keystone::authtoken':
|
||||
password => 'passw0rd',
|
||||
}"
|
||||
end
|
||||
|
||||
let :params do
|
||||
{}
|
||||
end
|
||||
@ -17,8 +10,6 @@ describe 'nova::metadata' do
|
||||
|
||||
context 'with default parameters' do
|
||||
|
||||
it { is_expected.to contain_class('nova::keystone::authtoken') }
|
||||
|
||||
it 'configures various stuff' do
|
||||
is_expected.to contain_nova_config('api/metadata_cache_expiration').with('value' => '<SERVICE DEFAULT>')
|
||||
is_expected.to contain_nova_config('api/local_metadata_per_cell').with('value' => '<SERVICE DEFAULT>')
|
||||
|
Loading…
x
Reference in New Issue
Block a user