Ori.livneh has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/72680


Change subject: $settings_dir mode to 0755; comment tweak
......................................................................

$settings_dir mode to 0755; comment tweak

$settings_dir is situated in the shared folder, so Puppet's attempt to pound
mode 0775 are futile and generate noise on Puppet runs. This is probably a good
sign that the creation of the settings_dir should be the job of the role class.

This patch also tweaks a comment in Vagrantfile regarding Vagrantfile-extra.rb.

Change-Id: I5c3dac1eafa26f9d720b78ce07b90ed9730ee638
---
M Vagrantfile
M puppet/modules/mediawiki/manifests/init.pp
2 files changed, 9 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/vagrant 
refs/changes/80/72680/1

diff --git a/Vagrantfile b/Vagrantfile
index dabb889..114ae3f 100644
--- a/Vagrantfile
+++ b/Vagrantfile
@@ -100,7 +100,7 @@
 end
 
 begin
+    # Load custom Vagrantfile overrides from 'Vagrantfile-extra.rb'
     require_relative 'Vagrantfile-extra'
 rescue LoadError
-    # No local Vagrantfile overrides.
 end
diff --git a/puppet/modules/mediawiki/manifests/init.pp 
b/puppet/modules/mediawiki/manifests/init.pp
index 3b35e28..b190972 100644
--- a/puppet/modules/mediawiki/manifests/init.pp
+++ b/puppet/modules/mediawiki/manifests/init.pp
@@ -73,7 +73,14 @@
         unless  => "php ${dir}/maintenance/sql.php </dev/null",
     }
 
-    file { [ $upload_dir, $settings_dir ]:
+    file { $settings_dir:
+        ensure => directory,
+        owner  => 'vagrant',
+        group  => 'www-data',
+        mode   => '0755',
+    }
+
+    file { $upload_dir:
         ensure => directory,
         owner  => 'vagrant',
         group  => 'www-data',

-- 
To view, visit https://gerrit.wikimedia.org/r/72680
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5c3dac1eafa26f9d720b78ce07b90ed9730ee638
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/vagrant
Gerrit-Branch: master
Gerrit-Owner: Ori.livneh <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to