Ori.livneh has uploaded a new change for review.
https://gerrit.wikimedia.org/r/142206
Change subject: mediawiki_singlenode: port apache::vhost to apache::site
......................................................................
mediawiki_singlenode: port apache::vhost to apache::site
* Keep the same template and get rid of some indirection (that is, rather than
pass $install_path to apache::vhost as $docroot, just reference $install_path
in the template, which is local to the module anyhow.
* Don't declare File['/etc/apache2/sites-enabled/wiki'] just to ensure =>
absent it, since the directory is managed recursively by Puppet anyhow.
Change-Id: I47f8f62034b306acb010fe1f544c3faffbe8d02d
---
M modules/mediawiki_singlenode/manifests/init.pp
M modules/mediawiki_singlenode/templates/mediawiki_singlenode.erb
2 files changed, 10 insertions(+), 17 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/06/142206/1
diff --git a/modules/mediawiki_singlenode/manifests/init.pp
b/modules/mediawiki_singlenode/manifests/init.pp
index 9bb02b2..b25e79b 100644
--- a/modules/mediawiki_singlenode/manifests/init.pp
+++ b/modules/mediawiki_singlenode/manifests/init.pp
@@ -111,20 +111,13 @@
Mw-extension <| |> -> Exec['mediawiki_update']
- file { '/etc/apache2/sites-enabled/wiki':
- ensure => absent
- }
- include apache::mod::php
- apache::vhost{ 'wikicontroller':
- port => '80',
- priority => '000',
- vhost_name => 'wiki',
- docroot => $install_path,
- template => $apache_site_template
- }
-
+ include ::apache::mod::php
include ::apache::mod::rewrite
+ apache::site { 'wikicontroller':
+ content => template('mediawiki_singlenode/mediawiki_singlenode.erb'),
+ }
+
file { "${install_path}/cache":
require => Exec['mediawiki_setup'],
mode => '0775',
diff --git a/modules/mediawiki_singlenode/templates/mediawiki_singlenode.erb
b/modules/mediawiki_singlenode/templates/mediawiki_singlenode.erb
index 83e6be2..8e352a3 100644
--- a/modules/mediawiki_singlenode/templates/mediawiki_singlenode.erb
+++ b/modules/mediawiki_singlenode/templates/mediawiki_singlenode.erb
@@ -1,29 +1,29 @@
<VirtualHost *:80>
ServerAdmin
"https://wikitech.wikimedia.org/wiki/Help:Single_Node_MediaWiki"
- DocumentRoot <%=@docroot%>
+ DocumentRoot <%= @install_path %>
<Directory />
Options FollowSymLinks
AllowOverride None
</Directory>
- <Directory <%=@docroot%>>
+ <Directory <%= @install_path %>>
Options Indexes FollowSymLinks MultiViews
AllowOverride None
Order allow,deny
allow from all
</Directory>
- <Directory <%= @docroot %>/orig>
+ <Directory <%= @install_path %>/orig>
Deny from all
</Directory>
RewriteEngine On
RewriteRule ^/$ /w/index.php [R=301]
- Alias /wiki "<%= @docroot %>/index.php"
- Alias /w "<%= @docroot %>"
+ Alias /wiki "<%= @install_path %>/index.php"
+ Alias /w "<%= @install_path %>"
ErrorLog /var/log/apache2/error.log
--
To view, visit https://gerrit.wikimedia.org/r/142206
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I47f8f62034b306acb010fe1f544c3faffbe8d02d
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ori.livneh <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits