Mobrovac has uploaded a new change for review. https://gerrit.wikimedia.org/r/308957
Change subject: Change-Prop: Fix concurrency calculation for transcludes ...................................................................... Change-Prop: Fix concurrency calculation for transcludes 364a635 doubled the concurrency for transclusion updates. However, it seems that Puppet interprets that as a string, causing Ruby to repeat the number twice instead of doubling it numerically. This patch fixes that by forcing a conversion to a number before the multiplication. Change-Id: I8af871cfd15accaa2337716d4960dec497c607b2 --- M modules/changeprop/templates/config.yaml.erb 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/operations/puppet refs/changes/57/308957/1 diff --git a/modules/changeprop/templates/config.yaml.erb b/modules/changeprop/templates/config.yaml.erb index 8d5476b..6904feb 100644 --- a/modules/changeprop/templates/config.yaml.erb +++ b/modules/changeprop/templates/config.yaml.erb @@ -220,7 +220,7 @@ body: '{{globals.message}}' on_transclusion_update: - concurrency: <%= (@concurrency * 2).to_i %> + concurrency: <%= @concurrency.to_i * 2 %> topic: resource_change match: meta: -- To view, visit https://gerrit.wikimedia.org/r/308957 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I8af871cfd15accaa2337716d4960dec497c607b2 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Mobrovac <mobro...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits