[MediaWiki-commits] [Gerrit] Start services after mounting /vagrant - change (mediawiki/vagrant)

2015-05-28 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged.

Change subject: Start services after mounting /vagrant
..


Start services after mounting /vagrant

The mediawiki services are set to start on boot. However, as they reside
in the /vagrant directory mounted by Vagrant after the boot, Upstart
fails to start them. This patch modifies the services' Upstart init
scripts to wait for the mediawiki-ready signal.

Bug: T96231
Change-Id: Ib61b6e33791df96b3ec01030282669905cb681f4
---
M puppet/manifests/site.pp
M puppet/modules/hhvm/files/hhvm.conf
M puppet/modules/mediawiki/templates/jobchron.conf.erb
M puppet/modules/mediawiki/templates/parsoid.conf.erb
M puppet/modules/restbase/templates/upstart.conf.erb
M puppet/modules/scholarships/files/debug_smtp.conf
M puppet/modules/sentry/templates/upstart.erb
M puppet/modules/smtplog/templates/smtplog.conf.erb
8 files changed, 8 insertions(+), 7 deletions(-)

Approvals:
  BryanDavis: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/puppet/manifests/site.pp b/puppet/manifests/site.pp
index 80609de..fe6ae67 100644
--- a/puppet/manifests/site.pp
+++ b/puppet/manifests/site.pp
@@ -35,6 +35,7 @@
 
 Service {
 ensure = running,
+enable = true,
 }
 
 Package {
diff --git a/puppet/modules/hhvm/files/hhvm.conf 
b/puppet/modules/hhvm/files/hhvm.conf
index fcb6b80..4a58f0e 100644
--- a/puppet/modules/hhvm/files/hhvm.conf
+++ b/puppet/modules/hhvm/files/hhvm.conf
@@ -3,7 +3,7 @@
 # Configured via /etc/default/hhvm.
 description HipHop Virtual Machine for PHP
 
-start on (local-filesystems and net-device-up IFACE!=lo)
+start on mediawiki-ready
 
 pre-start script
   [ ! -r /etc/default/hhvm ]  { stop; exit 0; }
diff --git a/puppet/modules/mediawiki/templates/jobchron.conf.erb 
b/puppet/modules/mediawiki/templates/jobchron.conf.erb
index 5727a45..0781e4c 100644
--- a/puppet/modules/mediawiki/templates/jobchron.conf.erb
+++ b/puppet/modules/mediawiki/templates/jobchron.conf.erb
@@ -1,7 +1,7 @@
 # jobchron -- Continuously maintain MediaWiki job queue
 description MediaWiki job queue chron loop
 
-start on (runlevel [2345] and net-device-up IFACE!=lo)
+start on mediawiki-ready
 stop on runlevel [!2345]
 
 pre-start script
diff --git a/puppet/modules/mediawiki/templates/parsoid.conf.erb 
b/puppet/modules/mediawiki/templates/parsoid.conf.erb
index 419c086..56fb405 100644
--- a/puppet/modules/mediawiki/templates/parsoid.conf.erb
+++ b/puppet/modules/mediawiki/templates/parsoid.conf.erb
@@ -5,7 +5,7 @@
 
 description Parsoid HTTP service
 
-start on (local-filesystems and net-device-up IFACE!=lo)
+start on mediawiki-ready
 stop on runlevel [!2345]
 
 setuid www-data
diff --git a/puppet/modules/restbase/templates/upstart.conf.erb 
b/puppet/modules/restbase/templates/upstart.conf.erb
index 0e6c365..2fb01d0 100644
--- a/puppet/modules/restbase/templates/upstart.conf.erb
+++ b/puppet/modules/restbase/templates/upstart.conf.erb
@@ -5,7 +5,7 @@
 
 description RESTBase REST API service
 
-start on (local-filesystems and net-device-up IFACE!=lo)
+start on mediawiki-ready
 stop on runlevel [!2345]
 
 setuid www-data
diff --git a/puppet/modules/scholarships/files/debug_smtp.conf 
b/puppet/modules/scholarships/files/debug_smtp.conf
index e6f1f5e..613d048 100644
--- a/puppet/modules/scholarships/files/debug_smtp.conf
+++ b/puppet/modules/scholarships/files/debug_smtp.conf
@@ -1,7 +1,7 @@
 # This file is managed by puppet
 description Debugging smtp server that logs to /vagrant/logs
 
-start on (local-filesystems and net-device-up IFACE=lo)
+start on mediawiki-ready
 stop on shutdown
 
 script
diff --git a/puppet/modules/sentry/templates/upstart.erb 
b/puppet/modules/sentry/templates/upstart.erb
index afab1c1..2ecb426 100644
--- a/puppet/modules/sentry/templates/upstart.erb
+++ b/puppet/modules/sentry/templates/upstart.erb
@@ -4,7 +4,7 @@
 
 description Sentry error log tracker
 
-start on startup
+start on mediawiki-ready
 
 exec /sbin/start-stop-daemon --quiet --start \
   --chuid 'www-data:www-data' \
diff --git a/puppet/modules/smtplog/templates/smtplog.conf.erb 
b/puppet/modules/smtplog/templates/smtplog.conf.erb
index 6ad5461..708e8ff 100644
--- a/puppet/modules/smtplog/templates/smtplog.conf.erb
+++ b/puppet/modules/smtplog/templates/smtplog.conf.erb
@@ -1,7 +1,7 @@
 # This file is managed by puppet
 description Debugging smtp server that logs to %= @log_file %
 
-start on (local-filesystems and net-device-up IFACE=lo)
+start on mediawiki-ready
 stop on shutdown
 
 script

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib61b6e33791df96b3ec01030282669905cb681f4
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/vagrant
Gerrit-Branch: master
Gerrit-Owner: Mobrovac mobro...@wikimedia.org
Gerrit-Reviewer: BryanDavis bda...@wikimedia.org
Gerrit-Reviewer: Mattflaschen mflasc...@wikimedia.org

[MediaWiki-commits] [Gerrit] Start services after mounting /vagrant - change (mediawiki/vagrant)

2015-05-28 Thread Mobrovac (Code Review)
Mobrovac has uploaded a new change for review.

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

Change subject: Start services after mounting /vagrant
..

Start services after mounting /vagrant

The mediawiki services are set to start on boot. However, as they reside
in the /vagrant directory mounted by Vagrant after the boot, Upstart
fails to start them. This patch modifies the services' Upstart init
scripts to wait for the mediawiki-ready signal.

Bug: T96231
Change-Id: Ib61b6e33791df96b3ec01030282669905cb681f4
---
M puppet/manifests/site.pp
M puppet/modules/mediawiki/templates/jobchron.conf.erb
M puppet/modules/mediawiki/templates/parsoid.conf.erb
M puppet/modules/restbase/templates/upstart.conf.erb
4 files changed, 4 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/vagrant 
refs/changes/26/214326/1

diff --git a/puppet/manifests/site.pp b/puppet/manifests/site.pp
index 80609de..2a9f954 100644
--- a/puppet/manifests/site.pp
+++ b/puppet/manifests/site.pp
@@ -34,6 +34,7 @@
 }
 
 Service {
+enable = true,
 ensure = running,
 }
 
diff --git a/puppet/modules/mediawiki/templates/jobchron.conf.erb 
b/puppet/modules/mediawiki/templates/jobchron.conf.erb
index 5727a45..0781e4c 100644
--- a/puppet/modules/mediawiki/templates/jobchron.conf.erb
+++ b/puppet/modules/mediawiki/templates/jobchron.conf.erb
@@ -1,7 +1,7 @@
 # jobchron -- Continuously maintain MediaWiki job queue
 description MediaWiki job queue chron loop
 
-start on (runlevel [2345] and net-device-up IFACE!=lo)
+start on mediawiki-ready
 stop on runlevel [!2345]
 
 pre-start script
diff --git a/puppet/modules/mediawiki/templates/parsoid.conf.erb 
b/puppet/modules/mediawiki/templates/parsoid.conf.erb
index 419c086..56fb405 100644
--- a/puppet/modules/mediawiki/templates/parsoid.conf.erb
+++ b/puppet/modules/mediawiki/templates/parsoid.conf.erb
@@ -5,7 +5,7 @@
 
 description Parsoid HTTP service
 
-start on (local-filesystems and net-device-up IFACE!=lo)
+start on mediawiki-ready
 stop on runlevel [!2345]
 
 setuid www-data
diff --git a/puppet/modules/restbase/templates/upstart.conf.erb 
b/puppet/modules/restbase/templates/upstart.conf.erb
index 0e6c365..2fb01d0 100644
--- a/puppet/modules/restbase/templates/upstart.conf.erb
+++ b/puppet/modules/restbase/templates/upstart.conf.erb
@@ -5,7 +5,7 @@
 
 description RESTBase REST API service
 
-start on (local-filesystems and net-device-up IFACE!=lo)
+start on mediawiki-ready
 stop on runlevel [!2345]
 
 setuid www-data

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib61b6e33791df96b3ec01030282669905cb681f4
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/vagrant
Gerrit-Branch: master
Gerrit-Owner: Mobrovac mobro...@wikimedia.org

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits