Rush has submitted this change and it was merged.
Change subject: phab local.json should trigger a phd restart
......................................................................
phab local.json should trigger a phd restart
* convert phd to a init style service
(hopefully should manage fine with /etc/init.d link)
* tell template to refresh phd on change
Change-Id: Ie393e690c644467d66163e652257f19d170ac84f
---
M modules/phabricator/manifests/init.pp
1 file changed, 2 insertions(+), 8 deletions(-)
Approvals:
Rush: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/phabricator/manifests/init.pp
b/modules/phabricator/manifests/init.pp
index f0ee310..28ec347 100644
--- a/modules/phabricator/manifests/init.pp
+++ b/modules/phabricator/manifests/init.pp
@@ -237,7 +237,7 @@
file { "${phabdir}/phabricator/conf/local/local.json":
content => template('phabricator/local.json.erb'),
require => Git::Install['phabricator/phabricator'],
- notify => Service[apache2],
+ notify => Service[phd],
}
#default location for phabricator tracked repositories
@@ -278,15 +278,9 @@
# This needs to become Upstart managed
# https://secure.phabricator.com/book/phabricator/article/managing_daemons/
- $phd = "${phabdir}/phabricator/bin/phd"
service { 'phd':
ensure => running,
- provider => base,
- binary => $phd,
- start => "${phd} start",
- stop => "${phd} stop",
- status => "${phd} status",
+ provider => init,
require => Git::Install['phabricator/phabricator'],
}
}
-
--
To view, visit https://gerrit.wikimedia.org/r/189953
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie393e690c644467d66163e652257f19d170ac84f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Rush <[email protected]>
Gerrit-Reviewer: Rush <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
