openstack/puppet-manila
Code Issues Proposed changes

Manila $::os_service_default for db and log

Browse Source 
Switch to $::os_service_default all params in logging and db.
Changes: logging.pp, db.pp and tests.

Related-bug: #1515273

Change-Id: Idf7bc36d321328391a1b361add965c53627124a4
This commit is contained in:
Iury Gregory Melo Ferreira 2015-11-24 19:49:36 +00:00
parent b77f1736bc
commit e6ab8881ee
22 changed files with 146 additions and 250 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

88
manifests/db.pp
View File

@ -10,37 +10,37 @@
#
# [*database_idle_timeout*]
# Timeout when db connections should be reaped.
# (Optional) Defaults to 3600.
# (Optional) Defaults to $::os_service_default
#
# [*database_min_pool_size*]
# Minimum number of SQL connections to keep open in a pool.
# (Optional) Defaults to 1.
# (Optional) Defaults to $::os_service_default
#
# [*database_max_pool_size*]
# Maximum number of SQL connections to keep open in a pool.
# (Optional) Defaults to 10.
# (Optional) Defaults to $::os_service_default
#
# [*database_max_retries*]
# Maximum db connection retries during startup.
# Setting -1 implies an infinite retry count.
# (Optional) Defaults to 10.
# (Optional) Defaults to $::os_service_default
#
# [*database_retry_interval*]
# Interval between retries of opening a sql connection.
# (Optional) Defaults to 10.
# (Optional) Defaults to $::os_service_default
#
# [*database_max_overflow*]
# If set, use this value for max_overflow with sqlalchemy.
# (Optional) Defaults to 20.
# (Optional) Defaults to $::os_service_default
#
class manila::db (
$database_connection = 'sqlite:////var/lib/manila/manila.sqlite',
$database_idle_timeout = 3600,
$database_min_pool_size = 1,
$database_max_pool_size = 10,
$database_max_retries = 10,
$database_retry_interval = 10,
$database_max_overflow = 20,
$database_idle_timeout = $::os_service_default,
$database_min_pool_size = $::os_service_default,
$database_max_pool_size = $::os_service_default,
$database_max_retries = $::os_service_default,
$database_retry_interval = $::os_service_default,
$database_max_overflow = $::os_service_default,
) {
# NOTE(spredzy): In order to keep backward compatibility we rely on the pick function
@ -56,42 +56,40 @@ class manila::db (
validate_re($database_connection_real,
'(sqlite|mysql|postgresql):\/\/(\S+:\S+@\S+\/\S+)?')
if $database_connection_real {
case $database_connection_real {
/^mysql:\/\//: {
$backend_package = false
require 'mysql::bindings'
require 'mysql::bindings::python'
}
/^postgresql:\/\//: {
$backend_package = false
require 'postgresql::lib::python'
}
/^sqlite:\/\//: {
$backend_package = $::manila::params::sqlite_package_name
}
default: {
fail('Unsupported backend configured')
}
case $database_connection_real {
/^mysql:\/\//: {
$backend_package = false
require 'mysql::bindings'
require 'mysql::bindings::python'
}
if $backend_package and !defined(Package[$backend_package]) {
package {'manila-backend-package':
ensure => present,
name => $backend_package,
tag => 'openstack',
}
/^postgresql:\/\//: {
$backend_package = false
require 'postgresql::lib::python'
}
manila_config {
'database/connection': value => $database_connection_real, secret => true;
'database/idle_timeout': value => $database_idle_timeout_real;
'database/min_pool_size': value => $database_min_pool_size_real;
'database/max_retries': value => $database_max_retries_real;
'database/retry_interval': value => $database_retry_interval_real;
'database/max_pool_size': value => $database_max_pool_size_real;
'database/max_overflow': value => $database_max_overflow_real;
/^sqlite:\/\//: {
$backend_package = $::manila::params::sqlite_package_name
}
default: {
fail('Unsupported backend configured')
}
}
if $backend_package and !defined(Package[$backend_package]) {
package {'manila-backend-package':
ensure => present,
name => $backend_package,
tag => 'openstack',
}
}
manila_config {
'database/connection': value => $database_connection_real, secret => true;
'database/idle_timeout': value => $database_idle_timeout_real;
'database/min_pool_size': value => $database_min_pool_size_real;
'database/max_retries': value => $database_max_retries_real;
'database/retry_interval': value => $database_retry_interval_real;
'database/max_pool_size': value => $database_max_pool_size_real;
'database/max_overflow': value => $database_max_overflow_real;
}
}

209
manifests/logging.pp
View File

@ -6,23 +6,23 @@
#
# [*verbose*]
# (Optional) Should the daemons log verbose messages
# Defaults to 'false'
# Defaults to $::os_service_default
#
# [*debug*]
# (Optional) Should the daemons log debug messages
# Defaults to 'false'
# Defaults to $::os_service_default
#
# [*use_syslog*]
# (Optional) Use syslog for logging.
# Defaults to 'false'
# Defaults to $::os_service_default
#
# [*use_stderr*]
# (optional) Use stderr for logging
# Defaults to 'true'
# Defaults to $::os_service_default
#
# [*log_facility*]
# (Optional) Syslog facility to receive log lines.
# Defaults to 'LOG_USER'
# Defaults to $::os_service_default
#
# [*log_dir*]
# (optional) Directory where logs should be stored.
@ -31,34 +31,34 @@
#
# [*logging_context_format_string*]
# (optional) Format string to use for log messages with context.
# Defaults to undef.
# Defaults to $::os_service_default
# Example: '%(asctime)s.%(msecs)03d %(process)d %(levelname)s %(name)s\
# [%(request_id)s %(user_identity)s] %(instance)s%(message)s'
#
# [*logging_default_format_string*]
# (optional) Format string to use for log messages without context.
# Defaults to undef.
# Defaults to $::os_service_default
# Example: '%(asctime)s.%(msecs)03d %(process)d %(levelname)s %(name)s\
# [-] %(instance)s%(message)s'
#
# [*logging_debug_format_suffix*]
# (optional) Formatted data to append to log format when level is DEBUG.
# Defaults to undef.
# Defaults to $::os_service_default
# Example: '%(funcName)s %(pathname)s:%(lineno)d'
#
# [*logging_exception_prefix*]
# (optional) Prefix each line of exception output with this format.
# Defaults to undef.
# Defaults to $::os_service_default
# Example: '%(asctime)s.%(msecs)03d %(process)d TRACE %(name)s %(instance)s'
#
# [*log_config_append*]
# The name of an additional logging configuration file.
# Defaults to undef.
# Defaults to $::os_service_default
# See https://docs.python.org/2/howto/logging.html
#
# [*default_log_levels*]
# (optional) Hash of logger (keys) and level (values) pairs.
# Defaults to undef.
# Defaults to $::os_service_default
# Example:
# { 'amqp' => 'WARN', 'amqplib' => 'WARN', 'boto' => 'WARN',
# 'qpid' => 'WARN', 'sqlalchemy' => 'WARN', 'suds' => 'INFO',
@ -67,47 +67,47 @@
#
# [*publish_errors*]
# (optional) Publish error events (boolean value).
# Defaults to undef (false if unconfigured).
# Defaults to $::os_service_default
#
# [*fatal_deprecations*]
# (optional) Make deprecations fatal (boolean value)
# Defaults to undef (false if unconfigured).
# Defaults to $::os_service_default
#
# [*instance_format*]
# (optional) If an instance is passed with the log message, format it
# like this (string value).
# Defaults to undef.
# Defaults to $::os_service_default
# Example: '[instance: %(uuid)s] '
#
# [*instance_uuid_format*]
# (optional) If an instance UUID is passed with the log message, format
# it like this (string value).
# Defaults to undef.
# Defaults to $::os_service_default
# Example: instance_uuid_format='[instance: %(uuid)s] '
#
# [*log_date_format*]
# (optional) Format string for %%(asctime)s in log records.
# Defaults to undef.
# Defaults to $::os_service_default
# Example: 'Y-%m-%d %H:%M:%S'
class manila::logging(
$use_syslog = false,
$use_stderr = true,
$log_facility = 'LOG_USER',
$use_syslog = $::os_service_default,
$use_stderr = $::os_service_default,
$log_facility = $::os_service_default,
$log_dir = '/var/log/manila',
$verbose = false,
$debug = false,
$logging_context_format_string = undef,
$logging_default_format_string = undef,
$logging_debug_format_suffix = undef,
$logging_exception_prefix = undef,
$log_config_append = undef,
$default_log_levels = undef,
$publish_errors = undef,
$fatal_deprecations = undef,
$instance_format = undef,
$instance_uuid_format = undef,
$log_date_format = undef,
$verbose = $::os_service_default,
$debug = $::os_service_default,
$logging_context_format_string = $::os_service_default,
$logging_default_format_string = $::os_service_default,
$logging_debug_format_suffix = $::os_service_default,
$logging_exception_prefix = $::os_service_default,
$log_config_append = $::os_service_default,
$default_log_levels = $::os_service_default,
$publish_errors = $::os_service_default,
$fatal_deprecations = $::os_service_default,
$instance_format = $::os_service_default,
$instance_uuid_format = $::os_service_default,
$log_date_format = $::os_service_default,
) {
# NOTE(spredzy): In order to keep backward compatibility we rely on the pick function
@ -119,6 +119,12 @@ class manila::logging(
$verbose_real = pick($::manila::verbose,$verbose)
$debug_real = pick($::manila::debug,$debug)
if is_service_default($default_log_levels) {
$default_log_levels_real = $default_log_levels
} else {
$default_log_levels_real = join(sort(join_keys_to_values($default_log_levels, '=')), ',')
}
manila_config {
'DEFAULT/debug' : value => $debug_real;
'DEFAULT/verbose' : value => $verbose_real;
@ -128,130 +134,19 @@ class manila::logging(
'DEFAULT/syslog_log_facility': value => $log_facility_real;
}
if $logging_context_format_string {
manila_config {
'DEFAULT/logging_context_format_string' :
value => $logging_context_format_string;
}
}
else {
manila_config {
'DEFAULT/logging_context_format_string' : ensure => absent;
}
}
if $logging_default_format_string {
manila_config {
'DEFAULT/logging_default_format_string' :
value => $logging_default_format_string;
}
}
else {
manila_config {
'DEFAULT/logging_default_format_string' : ensure => absent;
}
}
if $logging_debug_format_suffix {
manila_config {
'DEFAULT/logging_debug_format_suffix' :
value => $logging_debug_format_suffix;
}
}
else {
manila_config {
'DEFAULT/logging_debug_format_suffix' : ensure => absent;
}
}
if $logging_exception_prefix {
manila_config {
'DEFAULT/logging_exception_prefix' : value => $logging_exception_prefix;
}
}
else {
manila_config {
'DEFAULT/logging_exception_prefix' : ensure => absent;
}
}
if $log_config_append {
manila_config {
'DEFAULT/log_config_append' : value => $log_config_append;
}
}
else {
manila_config {
'DEFAULT/log_config_append' : ensure => absent;
}
}
if $default_log_levels {
manila_config {
'DEFAULT/default_log_levels' :
value => join(sort(join_keys_to_values($default_log_levels, '=')), ',');
}
}
else {
manila_config {
'DEFAULT/default_log_levels' : ensure => absent;
}
}
if $publish_errors {
manila_config {
'DEFAULT/publish_errors' : value => $publish_errors;
}
}
else {
manila_config {
'DEFAULT/publish_errors' : ensure => absent;
}
}
if $fatal_deprecations {
manila_config {
'DEFAULT/fatal_deprecations' : value => $fatal_deprecations;
}
}
else {
manila_config {
'DEFAULT/fatal_deprecations' : ensure => absent;
}
}
if $instance_format {
manila_config {
'DEFAULT/instance_format' : value => $instance_format;
}
}
else {
manila_config {
'DEFAULT/instance_format' : ensure => absent;
}
}
if $instance_uuid_format {
manila_config {
'DEFAULT/instance_uuid_format' : value => $instance_uuid_format;
}
}
else {
manila_config {
'DEFAULT/instance_uuid_format' : ensure => absent;
}
}
if $log_date_format {
manila_config {
'DEFAULT/log_date_format' : value => $log_date_format;
}
}
else {
manila_config {
'DEFAULT/log_date_format' : ensure => absent;
}
}
manila_config {
'DEFAULT/logging_context_format_string': value => $logging_context_format_string;
'DEFAULT/logging_default_format_string': value => $logging_default_format_string;
'DEFAULT/logging_debug_format_suffix': value => $logging_debug_format_suffix;
'DEFAULT/logging_exception_prefix': value => $logging_exception_prefix;
'DEFAULT/log_config_append': value => $log_config_append;
'DEFAULT/default_log_levels': value => $default_log_levels_real;
'DEFAULT/publish_errors': value => $publish_errors;
'DEFAULT/fatal_deprecations': value => $fatal_deprecations;
'DEFAULT/instance_format': value => $instance_format;
'DEFAULT/instance_uuid_format': value => $instance_uuid_format;
'DEFAULT/log_date_format': value => $log_date_format;
}
}

2
spec/classes/manila_api_spec.rb
View File

@ -6,7 +6,7 @@ describe 'manila::api' do
{:keystone_password => 'foo'}
end
let :facts do
{:osfamily => 'Debian'}
@default_facts.merge({:osfamily => 'Debian'})
end
describe 'with only required params' do

4
spec/classes/manila_backends_spec.rb
View File

@ -52,7 +52,7 @@ describe 'manila::backends' do
context 'on Debian platforms' do
let :facts do
{ :osfamily => 'Debian' }
@default_facts.merge({ :osfamily => 'Debian' })
end
it_configures 'manila backends'
@ -60,7 +60,7 @@ describe 'manila::backends' do
context 'on RedHat platforms' do
let :facts do
{ :osfamily => 'RedHat' }
@default_facts.merge({ :osfamily => 'RedHat' })
end
it_configures 'manila backends'

2
spec/classes/manila_client_spec.rb
View File

@ -3,7 +3,7 @@ require 'spec_helper'
describe 'manila::client' do
it { is_expected.to contain_package('python-manilaclient').with_ensure('present') }
let :facts do
{:osfamily => 'Debian'}
@default_facts.merge({:osfamily => 'Debian'})
end
context 'with params' do
let :params do

2
spec/classes/manila_db_mysql_spec.rb
View File

@ -8,7 +8,7 @@ describe 'manila::db::mysql' do
end
let :facts do
{:osfamily => 'Debian'}
@default_facts.merge({:osfamily => 'Debian'})
end
let :pre_condition do

8
spec/classes/manila_db_postgresql_spec.rb
View File

@ -12,11 +12,11 @@ describe 'manila::db::postgresql' do
context 'on a RedHat osfamily' do
let :facts do
{
@default_facts.merge({
:osfamily => 'RedHat',
:operatingsystemrelease => '7.0',
:concat_basedir => '/var/lib/puppet/concat'
}
})
end
context 'with only required parameters' do
@ -34,12 +34,12 @@ describe 'manila::db::postgresql' do
context 'on a Debian osfamily' do
let :facts do
{
@default_facts.merge({
:operatingsystemrelease => '7.8',
:operatingsystem => 'Debian',
:osfamily => 'Debian',
:concat_basedir => '/var/lib/puppet/concat'
}
})
end
context 'with only required parameters' do

20
spec/classes/manila_db_spec.rb
View File

@ -7,12 +7,12 @@ describe 'manila::db' do
context 'with default parameters' do
it { is_expected.to contain_manila_config('database/connection').with_value('sqlite:////var/lib/manila/manila.sqlite').with_secret(true) }
it { is_expected.to contain_manila_config('database/idle_timeout').with_value('3600') }
it { is_expected.to contain_manila_config('database/min_pool_size').with_value('1') }
it { is_expected.to contain_manila_config('database/max_pool_size').with_value('10') }
it { is_expected.to contain_manila_config('database/max_overflow').with_value('20') }
it { is_expected.to contain_manila_config('database/max_retries').with_value('10') }
it { is_expected.to contain_manila_config('database/retry_interval').with_value('10') }
it { is_expected.to contain_manila_config('database/idle_timeout').with_value('<SERVICE DEFAULT>') }
it { is_expected.to contain_manila_config('database/min_pool_size').with_value('<SERVICE DEFAULT>') }
it { is_expected.to contain_manila_config('database/max_pool_size').with_value('<SERVICE DEFAULT>') }
it { is_expected.to contain_manila_config('database/max_overflow').with_value('<SERVICE DEFAULT>') }
it { is_expected.to contain_manila_config('database/max_retries').with_value('<SERVICE DEFAULT>') }
it { is_expected.to contain_manila_config('database/retry_interval').with_value('<SERVICE DEFAULT>') }
end
@ -60,10 +60,10 @@ describe 'manila::db' do
context 'on Debian platforms' do
let :facts do
{ :osfamily => 'Debian',
@default_facts.merge({ :osfamily => 'Debian',
:operatingsystem => 'Debian',
:operatingsystemrelease => 'jessie',
}
})
end
it_configures 'manila::db'
@ -71,9 +71,9 @@ describe 'manila::db' do
context 'on Redhat platforms' do
let :facts do
{ :osfamily => 'RedHat',
@default_facts.merge({ :osfamily => 'RedHat',
:operatingsystemrelease => '7.1',
}
})
end
it_configures 'manila::db'

2
spec/classes/manila_db_sync_spec.rb
View File

@ -3,7 +3,7 @@ require 'spec_helper'
describe 'manila::db::sync' do
let :facts do
{:osfamily => 'Debian'}
@default_facts.merge({:osfamily => 'Debian'})
end
it { is_expected.to contain_exec('manila-manage db_sync').with(
:command => 'manila-manage db sync',

2
spec/classes/manila_ganesha_spec.rb
View File

@ -27,7 +27,7 @@ describe 'manila::ganesha' do
context 'on Red Hat platforms' do
let :facts do
{:osfamily => 'RedHat'}
@default_facts.merge({:osfamily => 'RedHat'})
end
it_configures 'manila NFS Ganesha options for share drivers'
end

14
spec/classes/manila_logging_spec.rb
View File

@ -57,11 +57,11 @@ describe 'manila::logging' do
shared_examples 'basic default logging settings' do
it 'configures manila logging settins with default values' do
is_expected.to contain_manila_config('DEFAULT/use_syslog').with(:value => 'false')
is_expected.to contain_manila_config('DEFAULT/use_stderr').with(:value => 'true')
is_expected.to contain_manila_config('DEFAULT/use_syslog').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_manila_config('DEFAULT/use_stderr').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_manila_config('DEFAULT/log_dir').with(:value => '/var/log/manila')
is_expected.to contain_manila_config('DEFAULT/verbose').with(:value => 'false')
is_expected.to contain_manila_config('DEFAULT/debug').with(:value => 'false')
is_expected.to contain_manila_config('DEFAULT/verbose').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_manila_config('DEFAULT/debug').with(:value => '<SERVICE DEFAULT>')
end
end
@ -120,13 +120,13 @@ describe 'manila::logging' do
:default_log_levels, :fatal_deprecations,
:instance_format, :instance_uuid_format,
:log_date_format, ].each { |param|
it { is_expected.to contain_manila_config("DEFAULT/#{param}").with_ensure('absent') }
it { is_expected.to contain_manila_config("DEFAULT/#{param}").with_value('<SERVICE DEFAULT>') }
}
end
context 'on Debian platforms' do
let :facts do
{ :osfamily => 'Debian' }
@default_facts.merge({ :osfamily => 'Debian' })
end
it_configures 'manila-logging'
@ -134,7 +134,7 @@ describe 'manila::logging' do
context 'on RedHat platforms' do
let :facts do
{ :osfamily => 'RedHat' }
@default_facts.merge({ :osfamily => 'RedHat' })
end
it_configures 'manila-logging'

2
spec/classes/manila_params_spec.rb
View File

@ -3,7 +3,7 @@ require 'spec_helper'
describe 'manila::params' do
let :facts do
{:osfamily => 'Debian'}
@default_facts.merge({:osfamily => 'Debian'})
end
it 'should compile' do
subject

4
spec/classes/manila_qpid_spec.rb
View File

@ -3,8 +3,8 @@ require 'spec_helper'
describe 'manila::qpid' do
let :facts do
{:puppetversion => '2.7',
:osfamily => 'RedHat'}
@default_facts.merge({:puppetversion => '2.7',
:osfamily => 'RedHat'})
end
describe 'with defaults' do

4
spec/classes/manila_rabbitmq_spec.rb
View File

@ -3,9 +3,9 @@ require 'spec_helper'
describe 'manila::rabbitmq' do
let :facts do
{ :puppetversion => '2.7',
@default_facts.merge({ :puppetversion => '2.7',
:osfamily => 'Debian',
}
})
end
describe 'with defaults' do

4
spec/classes/manila_scheduler_spec.rb
View File

@ -5,7 +5,7 @@ describe 'manila::scheduler' do
describe 'on debian platforms' do
let :facts do
{ :osfamily => 'Debian' }
@default_facts.merge({ :osfamily => 'Debian' })
end
describe 'with default parameters' do
@ -56,7 +56,7 @@ describe 'manila::scheduler' do
describe 'on rhel platforms' do
let :facts do
{ :osfamily => 'RedHat' }
@default_facts.merge({ :osfamily => 'RedHat' })
end
describe 'with default parameters' do

4
spec/classes/manila_share_glusterfs_spec.rb
View File

@ -23,7 +23,7 @@ describe 'manila::share::glusterfs' do
context 'on Debian platforms' do
let :facts do
{ :osfamily => 'Debian' }
@default_facts.merge({ :osfamily => 'Debian' })
end
it_configures 'glusterfs share driver'
@ -31,7 +31,7 @@ describe 'manila::share::glusterfs' do
context 'on RedHat platforms' do
let :facts do
{ :osfamily => 'RedHat' }
@default_facts.merge({ :osfamily => 'RedHat' })
end
it_configures 'glusterfs share driver'

4
spec/classes/manila_share_spec.rb
View File

@ -29,7 +29,7 @@ describe 'manila::share' do
context 'on Debian platforms' do
let :facts do
{ :osfamily => 'Debian' }
@default_facts.merge({ :osfamily => 'Debian' })
end
let :platform_params do
@ -41,7 +41,7 @@ describe 'manila::share' do
context 'on RedHat platforms' do
let :facts do
{ :osfamily => 'RedHat' }
@default_facts.merge({ :osfamily => 'RedHat' })
end
let :platform_params do

6
spec/classes/manila_spec.rb
View File

@ -5,7 +5,7 @@ describe 'manila' do
end
let :facts do
{:osfamily => 'Debian'}
@default_facts.merge({:osfamily => 'Debian'})
end
describe 'with only required params' do
@ -49,10 +49,10 @@ describe 'manila' do
:value => 'guest'
)
is_expected.to contain_manila_config('DEFAULT/verbose').with(
:value => false
:value => '<SERVICE DEFAULT>'
)
is_expected.to contain_manila_config('DEFAULT/debug').with(
:value => false
:value => '<SERVICE DEFAULT>'
)
is_expected.to contain_manila_config('DEFAULT/storage_availability_zone').with(
:value => 'nova'

4
spec/defines/manila_backend_glusterfs_spec.rb
View File

@ -26,7 +26,7 @@ describe 'manila::backend::glusterfs' do
context 'on Debian platforms' do
let :facts do
{ :osfamily => 'Debian' }
@default_facts.merge({ :osfamily => 'Debian' })
end
it_configures 'glusterfs share driver'
@ -34,7 +34,7 @@ describe 'manila::backend::glusterfs' do
context 'on RedHat platforms' do
let :facts do
{ :osfamily => 'RedHat' }
@default_facts.merge({ :osfamily => 'RedHat' })
end
it_configures 'glusterfs share driver'

4
spec/defines/manila_backend_glusternative_spec.rb
View File

@ -28,7 +28,7 @@ describe 'manila::backend::glusternative' do
context 'on Debian platforms' do
let :facts do
{ :osfamily => 'Debian' }
@default_facts.merge({ :osfamily => 'Debian' })
end
it_configures 'glusternative volume driver'
@ -36,7 +36,7 @@ describe 'manila::backend::glusternative' do
context 'on RedHat platforms' do
let :facts do
{ :osfamily => 'RedHat' }
@default_facts.merge({ :osfamily => 'RedHat' })
end
it_configures 'glusternative volume driver'

4
spec/defines/manila_backend_glusternfs_spec.rb
View File

@ -30,7 +30,7 @@ describe 'manila::backend::glusternfs' do
context 'on Debian platforms' do
let :facts do
{ :osfamily => 'Debian' }
@default_facts.merge({ :osfamily => 'Debian' })
end
it_configures 'glusternfs volume driver'
@ -38,7 +38,7 @@ describe 'manila::backend::glusternfs' do
context 'on RedHat platforms' do
let :facts do
{ :osfamily => 'RedHat' }
@default_facts.merge({ :osfamily => 'RedHat' })
end
it_configures 'glusternfs volume driver'

3
spec/spec_helper.rb
View File

@ -5,6 +5,9 @@ require 'webmock/rspec'
RSpec.configure do |c|
c.alias_it_should_behave_like_to :it_configures, 'configures'
c.alias_it_should_behave_like_to :it_raises, 'raises'
c.before :each do
@default_facts = { :os_service_default => '<SERVICE DEFAULT>' }
end
end
at_exit { RSpec::Puppet::Coverage.report! }