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

Reply via email to