Subramanya Sastry has uploaded a new change for review.
https://gerrit.wikimedia.org/r/265628
Change subject: Migrate parsoid::role::testing service from upstart to systemd
......................................................................
Migrate parsoid::role::testing service from upstart to systemd
Change-Id: I093ef1f73b61d86753538836e361707c6b58e008
---
A files/misc/parsoid.systemd.service
M manifests/role/parsoid.pp
2 files changed, 23 insertions(+), 26 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/28/265628/1
diff --git a/files/misc/parsoid.systemd.service
b/files/misc/parsoid.systemd.service
new file mode 100644
index 0000000..012bcb7
--- /dev/null
+++ b/files/misc/parsoid.systemd.service
@@ -0,0 +1,16 @@
+[Unit]
+Description=parsoid-vd: Testreduce HTTP service for visual-diff results
+After=network.target
+
+[Service]
+User=testreduce
+Group=testreduce
+LimitNOFILE=10K
+
+Environment=NODE_PATH=/usr/lib/parsoid/deploy/node_modules VCAP_APP_PORT=8000
+WorkingDirectory=/usr/lib/parsoid/deploy/src
+ExecStart=node bin/server.js -c /usr/lib/parsoid/src/localsettings.js
+StandardInput=null
+StandardOutput=/var/log/parsoid/parsoid.log
+StandardError=/var/log/parsoid/parsoid.log
+RestartSec=2
diff --git a/manifests/role/parsoid.pp b/manifests/role/parsoid.pp
index a38e3a8..49b971f 100644
--- a/manifests/role/parsoid.pp
+++ b/manifests/role/parsoid.pp
@@ -285,46 +285,27 @@
before => Service['parsoid'],
}
- file { '/etc/init/parsoid.conf':
- ensure => present,
- owner => root,
- group => root,
+ file { '/lib/systemd/system/parsoid.service':
+ source => 'puppet:///files/misc/parsoid.systemd.service',
+ owner => 'root',
+ group => 'root',
mode => '0444',
- source => 'puppet:///files/misc/parsoid.upstart',
before => Service['parsoid'],
}
file { '/var/log/parsoid':
ensure => directory,
- owner => parsoid,
- group => parsoid,
+ owner => 'parsoid',
+ group => 'parsoid'
mode => '0775',
before => Service['parsoid'],
- }
-
- $parsoid_log_file = '/var/log/parsoid/parsoid.log'
- $parsoid_node_path = '/usr/lib/parsoid/deploy/node_modules'
- # default local settings -- test setups provide their own settings
- $parsoid_settings_file = '/usr/lib/parsoid/src/localsettings.js'
- $parsoid_base_path = '/usr/lib/parsoid/deploy/src'
-
- #TODO: Duplication of code, deduplicate somehow
- file { '/etc/default/parsoid':
- ensure => present,
- owner => root,
- group => root,
- mode => '0444',
- content => template('misc/parsoid.default.erb'),
- before => Service['parsoid'],
}
service { 'parsoid':
hasstatus => true,
hasrestart => true,
- provider => 'upstart',
subscribe => [
- File['/etc/default/parsoid'],
- File['/etc/init/parsoid.conf'],
+ File['/lib/systemd/system/parsoid.service'],
],
}
}
--
To view, visit https://gerrit.wikimedia.org/r/265628
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I093ef1f73b61d86753538836e361707c6b58e008
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Subramanya Sastry <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
