Faidon Liambotis has submitted this change and it was merged.
Change subject: varnish: lint fixups
......................................................................
varnish: lint fixups
Change-Id: Iae136d73c8fc912e64b10f8e4796c81eb32dcc4a
---
M modules/varnish/manifests/instance.pp
M modules/varnish/manifests/packages.pp
2 files changed, 7 insertions(+), 4 deletions(-)
Approvals:
Faidon Liambotis: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/varnish/manifests/instance.pp
b/modules/varnish/manifests/instance.pp
index f944160..ef50ec1 100644
--- a/modules/varnish/manifests/instance.pp
+++ b/modules/varnish/manifests/instance.pp
@@ -112,7 +112,8 @@
# Restart gmond if this varnish instance has been (re)started later
# than gmond was started
exec { "restart gmond for varnish${instancesuffix}":
- command => '/bin/true',
+ path => '/bin:/usr/bin',
+ command => 'true',
onlyif => "test /var/run/varnishd${instancesuffix}.pid -nt
/var/run/gmond.pid",
notify => Service['gmond'],
}
diff --git a/modules/varnish/manifests/packages.pp
b/modules/varnish/manifests/packages.pp
index 3062479..1dcda2a 100644
--- a/modules/varnish/manifests/packages.pp
+++ b/modules/varnish/manifests/packages.pp
@@ -1,7 +1,9 @@
class varnish::packages($version='installed') {
- package { [ 'varnish',
- 'libvarnishapi1',
- 'varnish-dbg' ]:
+ package { [
+ 'varnish',
+ 'varnish-dbg',
+ 'libvarnishapi1',
+ ]:
ensure => $version
}
}
--
To view, visit https://gerrit.wikimedia.org/r/98493
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iae136d73c8fc912e64b10f8e4796c81eb32dcc4a
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon Liambotis <fai...@wikimedia.org>
Gerrit-Reviewer: Faidon Liambotis <fai...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
