From 2c9312670d3d2363248c93bc43e0fb50c53eef45 Mon Sep 17 00:00:00 2001 From: Takashi Kajinami Date: Mon, 31 Aug 2020 18:00:17 +0900 Subject: [PATCH] Do not include placement::config by default ... because there is no requirement to have this inclusion. Change-Id: I1b7380dfcd6b009328e334dd4b5701dd529fceee --- manifests/db.pp | 1 - ...ement-db-includes-placmenet-config-8015c11cda289a3b.yaml | 6 ++++++ spec/classes/placement_db_spec.rb | 2 -- 3 files changed, 6 insertions(+), 3 deletions(-) create mode 100644 releasenotes/notes/placement-db-includes-placmenet-config-8015c11cda289a3b.yaml diff --git a/manifests/db.pp b/manifests/db.pp index 4930d9e..3a87193 100644 --- a/manifests/db.pp +++ b/manifests/db.pp @@ -70,7 +70,6 @@ class placement::db ( ) { include placement::deps - include placement::config oslo::db { 'placement_config': config_group => 'placement_database', diff --git a/releasenotes/notes/placement-db-includes-placmenet-config-8015c11cda289a3b.yaml b/releasenotes/notes/placement-db-includes-placmenet-config-8015c11cda289a3b.yaml new file mode 100644 index 0000000..962318e --- /dev/null +++ b/releasenotes/notes/placement-db-includes-placmenet-config-8015c11cda289a3b.yaml @@ -0,0 +1,6 @@ +--- +upgrade: + - | + Now the ``placmenet::db`` class doesn't include the ``placement::config`` + class automatically. Include the ``placement::config`` explicitly in + manifests if the class is required. diff --git a/spec/classes/placement_db_spec.rb b/spec/classes/placement_db_spec.rb index ed262d3..8dca783 100644 --- a/spec/classes/placement_db_spec.rb +++ b/spec/classes/placement_db_spec.rb @@ -5,7 +5,6 @@ describe 'placement::db' do context 'with default parameters' do it { should contain_class('placement::deps') - should contain_class('placement::config') } it { should contain_oslo__db('placement_config').with( @@ -45,7 +44,6 @@ describe 'placement::db' do it { should contain_class('placement::deps') - should contain_class('placement::config') } it { should contain_oslo__db('placement_config').with(