Only designate-central and mdns require database access

According to Designate architecture documentation,
only the designate-central and mdns services require database access.

This change removes the designate::db class for generic service and
include the designate::db class for services that need it instead.

Change-Id: I696745e2d57aade6cbb0fe89e334fddf1beaafdb
This commit is contained in:
Mathieu Gagné 2018-10-22 13:41:36 -04:00
parent 61bef03c90
commit 412ca9519c
5 changed files with 10 additions and 1 deletions

View File

@ -69,6 +69,7 @@ class designate::central (
) inherits designate {
include ::designate::deps
include ::designate::db
designate_config {
'service:central/managed_resource_email' : value => $managed_resource_email;

View File

@ -41,7 +41,6 @@ define designate::generic_service(
include ::designate::deps
include ::designate::params
include ::designate::db
$designate_title = "designate-${name}"
Exec['post-designate_config'] ~> Anchor['designate::service::end']

View File

@ -68,6 +68,7 @@ class designate::mdns (
) inherits designate {
include ::designate::deps
include ::designate::db
designate_config {
'service:mdns/workers' : value => $workers;

View File

@ -26,6 +26,10 @@ describe 'designate::central' do
)
end
it 'includes designate::db' do
is_expected.to contain_class('designate::db')
end
it 'configures designate-central with default parameters' do
is_expected.to contain_designate_config('service:central/managed_resource_email').with_value('hostmaster@example.com')
is_expected.to contain_designate_config('service:central/managed_resource_tenant_id').with_value('123456')

View File

@ -27,6 +27,10 @@ describe 'designate::mdns' do
)
end
it 'includes designate::db' do
is_expected.to contain_class('designate::db')
end
it 'configures designate-mdns with default parameters' do
is_expected.to contain_designate_config('service:mdns/workers').with_value(8)
is_expected.to contain_designate_config('service:mdns/threads').with_value('<SERVICE DEFAULT>')