Herron has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/387113 )
Change subject: puppet: change elasticsearch_5 template to parse under puppet 4 ...................................................................... puppet: change elasticsearch_5 template to parse under puppet 4 Update elasticsearch_5.yml.erb to use @variable to address undefined local variable or method `script_max_compilations_per_minute' error occurring under a puppet 4 master. Bug: T179174 Change-Id: I9147b6ba33632591136daeaa95a15b3ec78cbcfe --- M modules/elasticsearch/templates/elasticsearch_5.yml.erb 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Herron: Looks good to me, approved Alexandros Kosiaris: Looks good to me, but someone else must approve jenkins-bot: Verified diff --git a/modules/elasticsearch/templates/elasticsearch_5.yml.erb b/modules/elasticsearch/templates/elasticsearch_5.yml.erb index 196931d..83bfae7 100644 --- a/modules/elasticsearch/templates/elasticsearch_5.yml.erb +++ b/modules/elasticsearch/templates/elasticsearch_5.yml.erb @@ -444,5 +444,5 @@ <% end %> <% if @script_max_compilations_per_minute %> -script.max_compilations_per_minute: <%=script_max_compilations_per_minute%> +script.max_compilations_per_minute: <%=@script_max_compilations_per_minute%> <% end %> -- To view, visit https://gerrit.wikimedia.org/r/387113 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9147b6ba33632591136daeaa95a15b3ec78cbcfe Gerrit-PatchSet: 3 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Herron <[email protected]> Gerrit-Reviewer: Alexandros Kosiaris <[email protected]> Gerrit-Reviewer: Gehel <[email protected]> Gerrit-Reviewer: Herron <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
