openstack
/
puppet-congress
Code Issues Proposed changes

Merge "Dont include logging in init by default"

This commit is contained in:
Zuul 2018-12-12 01:20:17 +00:00 committed by Gerrit Code Review
parent 0c232a9d95 c7ea1a1d5f
commit 7e7cb02fa5
3 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
manifests/init.pp
View File

@ -226,7 +226,6 @@ class congress(
) inherits congress::params { ) inherits congress::params {
include ::congress::deps include ::congress::deps
include ::congress::logging
package { 'congress-common': package { 'congress-common':
ensure => $package_ensure, ensure => $package_ensure,

4
releasenotes/notes/dont-include-logging-init-a2ebac9e1ad21682.yaml Normal file
View File

@ -0,0 +1,4 @@
---
upgrade:
- |
congress::logging is now not included in init class by default.

3
spec/classes/congress_init_spec.rb
View File

@ -9,9 +9,8 @@ describe 'congress' do
{} {}
end end
it 'contains the logging class' do it 'contains deps class' do
is_expected.to contain_class('congress::deps') is_expected.to contain_class('congress::deps')
is_expected.to contain_class('congress::logging')
end end
it { is_expected.to contain_package('congress-common').with( it { is_expected.to contain_package('congress-common').with(