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:
parent
61bef03c90
commit
412ca9519c
|
@ -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;
|
||||
|
|
|
@ -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']
|
||||
|
|
|
@ -68,6 +68,7 @@ class designate::mdns (
|
|||
) inherits designate {
|
||||
|
||||
include ::designate::deps
|
||||
include ::designate::db
|
||||
|
||||
designate_config {
|
||||
'service:mdns/workers' : value => $workers;
|
||||
|
|
|
@ -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')
|
||||
|
|
|
@ -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>')
|
||||
|
|
Loading…
Reference in New Issue