opendev/puppet-httpd
Code Issues Proposed changes

Fix httpd::dev.

Browse Source 
While writing acceptance tests for puppet-httpd we noticed that the
manifest of httpd::dev was not working on Ubuntu Trusty.

That is because httpd::dev was trying to install multiple package using
the wrong syntax. Also, the one of the modules for Ubuntu was wrong and
we updated to keep applying.

Co-Authored-By: Bruno Tavares <btavare@thoughtworks.com>
Co-Authored-By: Danilo Ramalho <dramalho@thoughtworks.com>

Change-Id: Ibe1e107417b4670902f74b8c2fedc0d9d927ce9b
This commit is contained in:
Bruno Tavares 2015-09-15 10:07:22 -03:00
parent d3261cdc81
commit 84c5d2bf6a
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
manifests/dev.pp
View File

@ -14,8 +14,7 @@
class httpd::dev {
include ::httpd::params
package { 'apache_dev_package':
package { $httpd::params::apache_dev:
ensure => installed,
name => $httpd::params::apache_dev,
}
}

2
manifests/params.pp
View File

@ -47,7 +47,7 @@ class httpd::params {
$mod_python_package = 'libapache2-mod-python'
$mod_wsgi_package = 'libapache2-mod-wsgi'
$ssl_package = 'apache-ssl'
$apache_dev = ['libaprutil1-dev', 'libapr1-dev', 'apache2-prefork-dev']
$apache_dev = ['libaprutil1-dev', 'libapr1-dev', 'apache2-dev']
$vdir = '/etc/apache2/sites-enabled/'
}
default: {