opendev
/
puppet-httpd
Code Issues Proposed changes

Use httpd::mod when adding Apache modules. 

Replace usage of httpd_mod with the httpd::mod wrapper to allow classes
to be applied as well on RedHat systems and have the httpd service
ordering fixed.

Change-Id: If790efd7b465e77c2eca7710f79e563d69ee3783
Co-Authored-By: Danilo Ramalho <dramalho@thoughtworks.com>
This commit is contained in:
Bruno Tavares 2015-09-15 10:08:02 -03:00
parent a6f2016567
commit c180eeac3f
4 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/mod/python.pp
View File

@ -20,7 +20,7 @@ class httpd::mod::python {
require => Package['httpd']; require => Package['httpd'];
} }
httpd_mod { 'python': ensure => present; } httpd::mod { 'python': ensure => present; }
} }

2
manifests/mod/wsgi.pp
View File

@ -20,7 +20,7 @@ class httpd::mod::wsgi {
require => Package['httpd']; require => Package['httpd'];
} }
httpd_mod { 'wsgi': ensure => present; } httpd::mod { 'wsgi': ensure => present; }
} }

2
manifests/python.pp
View File

@ -20,6 +20,6 @@ class httpd::python {
ensure => present, ensure => present,
name => $httpd::params::mod_python_package, name => $httpd::params::mod_python_package,
} }
httpd_mod { 'python': ensure => present, } httpd::mod { 'python': ensure => present, }
} }

2
manifests/ssl.pp
View File

@ -25,7 +25,7 @@ class httpd::ssl {
} }
} }
'ubuntu', 'debian': { 'ubuntu', 'debian': {
httpd_mod { 'ssl': ensure => present, } httpd::mod { 'ssl': ensure => present, }
} }
default: { default: {
fail( "${::operatingsystem} not defined in httpd::ssl.") fail( "${::operatingsystem} not defined in httpd::ssl.")