Index: modules/web/manifests/server.pp |
=================================================================== |
--- a/modules/web/manifests/server.pp |
+++ b/modules/web/manifests/server.pp |
@@ -10,8 +10,6 @@ |
$geoip = false, |
) { |
- include sitescripts |
- |
# Ensure there is at least one character in the respective strings; |
# see https://codereview.adblockplus.org/29329028/#msg3 |
validate_re($vhost, '.+') |
@@ -41,25 +39,7 @@ |
ensure => $geoip ? {false => 'absent', default => 'present'}, |
} |
- ensure_packages(['python-pip']) |
- |
- # Make sure that apt packages corresponding to the pip-installed modules below |
- # won't be installed unintentionally, these will take precedence otherwise. |
- package {['python-jinja2', 'python-markdown']: |
- ensure => 'held', |
- } |
- |
- package {'Jinja2': |
- ensure => '2.8', |
- provider => 'pip', |
- require => [Package['python-pip'], Package['python-jinja2']], |
- } |
- |
- package {'markdown': |
- ensure => '2.6.6', |
- provider => 'pip', |
- require => [Package['python-pip'], Package['python-markdown']], |
- } |
+ package {['python-jinja2', 'python-markdown']:} |
nginx::hostconfig {$vhost: |
content => template('web/site.conf.erb'), |
@@ -70,14 +50,14 @@ |
log => "access_log_$vhost" |
} |
- sitescripts::configfragment {$title: |
- source => 'puppet:///modules/web/sitescripts', |
- } |
- |
if $multiplexer_locations != undef { |
include spawn-fcgi |
package {'python-flup':} |
+ class {'sitescripts': |
+ sitescriptsini_source => 'puppet:///modules/web/sitescripts', |
+ } |
+ |
spawn-fcgi::pool {"multiplexer": |
ensure => present, |
fcgi_app => '/opt/sitescripts/multiplexer.fcgi', |
@@ -86,7 +66,7 @@ |
user => 'nginx', |
children => 1, |
require => [ |
- Class["sitescripts"], |
+ Exec["fetch_sitescripts"], |
Package["python-flup"], |
], |
} |
@@ -172,8 +152,7 @@ |
cron {'generate_docs': |
ensure => present, |
require => [ |
- Class['sitescripts'], |
- Exec['install_jsdoc'], |
+ Exec['fetch_sitescripts', 'install_jsdoc'], |
Package['make', 'doxygen'], |
File['/var/www/docs'], |
], |