Dzahn has submitted this change and it was merged.
Change subject: mediawiki/jobrunner: fix top-scope var without namespace
......................................................................
mediawiki/jobrunner: fix top-scope var without namespace
./modules/mediawiki/manifests/jobrunner.pp
WARNING: top-scope variable being used without an explicit namespace on line 27
Change-Id: I8b82c2218b0103acdc7358d126864f5159868df5
---
M modules/mediawiki/manifests/jobrunner.pp
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
ArielGlenn: Looks good to me, but someone else must approve
jenkins-bot: Verified
Dzahn: Looks good to me, approved
diff --git a/modules/mediawiki/manifests/jobrunner.pp
b/modules/mediawiki/manifests/jobrunner.pp
index cd65bda..713bf91 100644
--- a/modules/mediawiki/manifests/jobrunner.pp
+++ b/modules/mediawiki/manifests/jobrunner.pp
@@ -24,7 +24,7 @@
notify => Service['jobrunner'],
}
- $dispatcher =
template("mediawiki/jobrunner/dispatchers/${lsbdistcodename}.erb")
+ $dispatcher =
template("mediawiki/jobrunner/dispatchers/${::lsbdistcodename}.erb")
file { '/etc/default/jobrunner':
content => template('mediawiki/jobrunner/jobrunner.default.erb'),
--
To view, visit https://gerrit.wikimedia.org/r/266967
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8b82c2218b0103acdc7358d126864f5159868df5
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <[email protected]>
Gerrit-Reviewer: ArielGlenn <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
