Remove deprecated logging
Change-Id: Ie331f058148f79a95bb8343f920654ef7dd5798b
This commit is contained in:
parent
abb9ba6f35
commit
22892835e9
|
@ -16,27 +16,6 @@
|
||||||
# (optional) Whether the designate-common package will be present..
|
# (optional) Whether the designate-common package will be present..
|
||||||
# Defaults to 'present'
|
# Defaults to 'present'
|
||||||
#
|
#
|
||||||
# [*debug*]
|
|
||||||
# (optional) should the daemons log debug messages.
|
|
||||||
# Defaults to undef
|
|
||||||
#
|
|
||||||
# [*use_syslog*]
|
|
||||||
# (Optional) Use syslog for logging.
|
|
||||||
# Defaults to undef
|
|
||||||
#
|
|
||||||
# [*use_stderr*]
|
|
||||||
# (optional) Use stderr for logging
|
|
||||||
# Defaults to undef
|
|
||||||
#
|
|
||||||
# [*log_facility*]
|
|
||||||
# (Optional) Syslog facility to receive log lines.
|
|
||||||
# Defaults to undef
|
|
||||||
#
|
|
||||||
# [*log_dir*]
|
|
||||||
# (optional) Directory where logs should be stored.
|
|
||||||
# If set to $::os_service_default, it will not log to any directory.
|
|
||||||
# Defaults to undef
|
|
||||||
#
|
|
||||||
# [*root_helper*]
|
# [*root_helper*]
|
||||||
# (optional) Command for designate rootwrap helper.
|
# (optional) Command for designate rootwrap helper.
|
||||||
# Defaults to 'sudo designate-rootwrap /etc/designate/rootwrap.conf'.
|
# Defaults to 'sudo designate-rootwrap /etc/designate/rootwrap.conf'.
|
||||||
|
@ -124,11 +103,6 @@
|
||||||
class designate(
|
class designate(
|
||||||
$package_ensure = present,
|
$package_ensure = present,
|
||||||
$common_package_name = $::designate::params::common_package_name,
|
$common_package_name = $::designate::params::common_package_name,
|
||||||
$debug = undef,
|
|
||||||
$log_dir = undef,
|
|
||||||
$use_syslog = undef,
|
|
||||||
$use_stderr = undef,
|
|
||||||
$log_facility = undef,
|
|
||||||
$root_helper = 'sudo designate-rootwrap /etc/designate/rootwrap.conf',
|
$root_helper = 'sudo designate-rootwrap /etc/designate/rootwrap.conf',
|
||||||
$notification_transport_url = $::os_service_default,
|
$notification_transport_url = $::os_service_default,
|
||||||
$rabbit_use_ssl = false,
|
$rabbit_use_ssl = false,
|
||||||
|
@ -164,7 +138,6 @@ class designate(
|
||||||
}
|
}
|
||||||
|
|
||||||
include ::designate::deps
|
include ::designate::deps
|
||||||
include ::designate::logging
|
|
||||||
|
|
||||||
exec { 'post-designate_config':
|
exec { 'post-designate_config':
|
||||||
command => '/bin/echo "designate config has changed"',
|
command => '/bin/echo "designate config has changed"',
|
||||||
|
|
|
@ -116,26 +116,14 @@ class designate::logging(
|
||||||
|
|
||||||
include ::designate::deps
|
include ::designate::deps
|
||||||
|
|
||||||
# NOTE(spredzy): In order to keep backward compatibility we rely on the pick function
|
|
||||||
# to use designate::<myparam> first then designate::logging::<myparam>.
|
|
||||||
$use_syslog_real = pick($::designate::use_syslog,$use_syslog)
|
|
||||||
$use_stderr_real = pick($::designate::use_stderr,$use_stderr)
|
|
||||||
$log_facility_real = pick($::designate::log_facility,$log_facility)
|
|
||||||
if $log_dir != '' {
|
|
||||||
$log_dir_real = pick($::designate::log_dir,$log_dir)
|
|
||||||
} else {
|
|
||||||
$log_dir_real = $log_dir
|
|
||||||
}
|
|
||||||
$debug_real = pick($::designate::debug,$debug)
|
|
||||||
|
|
||||||
oslo::log { 'designate_config':
|
oslo::log { 'designate_config':
|
||||||
debug => $debug_real,
|
debug => $debug,
|
||||||
use_syslog => $use_syslog_real,
|
use_syslog => $use_syslog,
|
||||||
use_json => $use_json,
|
use_json => $use_json,
|
||||||
use_journal => $use_journal,
|
use_journal => $use_journal,
|
||||||
use_stderr => $use_stderr_real,
|
use_stderr => $use_stderr,
|
||||||
log_dir => $log_dir_real,
|
log_dir => $log_dir,
|
||||||
syslog_log_facility => $log_facility_real,
|
syslog_log_facility => $log_facility,
|
||||||
logging_context_format_string => $logging_context_format_string,
|
logging_context_format_string => $logging_context_format_string,
|
||||||
logging_default_format_string => $logging_default_format_string,
|
logging_default_format_string => $logging_default_format_string,
|
||||||
logging_debug_format_suffix => $logging_debug_format_suffix,
|
logging_debug_format_suffix => $logging_debug_format_suffix,
|
||||||
|
|
|
@ -8,7 +8,6 @@ describe 'designate' do
|
||||||
let :params do
|
let :params do
|
||||||
{
|
{
|
||||||
:package_ensure => 'installed',
|
:package_ensure => 'installed',
|
||||||
:debug => 'False',
|
|
||||||
:purge_config => false,
|
:purge_config => false,
|
||||||
:neutron_endpoint_type => 'internalURL',
|
:neutron_endpoint_type => 'internalURL',
|
||||||
:root_helper => 'sudo designate-rootwrap /etc/designate/rootwrap.conf'
|
:root_helper => 'sudo designate-rootwrap /etc/designate/rootwrap.conf'
|
||||||
|
@ -76,7 +75,6 @@ describe 'designate' do
|
||||||
shared_examples_for 'a designate base installation' do
|
shared_examples_for 'a designate base installation' do
|
||||||
|
|
||||||
it { is_expected.to contain_class('designate::deps') }
|
it { is_expected.to contain_class('designate::deps') }
|
||||||
it { is_expected.to contain_class('designate::logging') }
|
|
||||||
it { is_expected.to contain_class('designate::params') }
|
it { is_expected.to contain_class('designate::params') }
|
||||||
|
|
||||||
it 'installs designate common package' do
|
it 'installs designate common package' do
|
||||||
|
@ -93,7 +91,7 @@ describe 'designate' do
|
||||||
})
|
})
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'configures debug and verbosity' do
|
it 'configures root_helper' do
|
||||||
is_expected.to contain_designate_config('DEFAULT/root_helper').with_value( params[:root_helper] )
|
is_expected.to contain_designate_config('DEFAULT/root_helper').with_value( params[:root_helper] )
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue