GWicke has uploaded a new change for review.
https://gerrit.wikimedia.org/r/287148
Change subject: Set conservative retry limits & delays
......................................................................
Set conservative retry limits & delays
Changeprop was previously using the default retry limits & delays of:
- 500ms delay
- up to 5 retries.
This patch changes these values to something more reasonable:
- 60s delay, and
- up to 2 retries.
Bug: T134456
Change-Id: Ifaf44d1562681758e8d44ec7cc9fa1faf99374e5
---
M modules/changeprop/templates/config.yaml.erb
1 file changed, 12 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/48/287148/1
diff --git a/modules/changeprop/templates/config.yaml.erb
b/modules/changeprop/templates/config.yaml.erb
index cd7d456..6130ae4 100644
--- a/modules/changeprop/templates/config.yaml.erb
+++ b/modules/changeprop/templates/config.yaml.erb
@@ -26,6 +26,12 @@
templates:
summary_rerender:
topic: resource_change
+ retry_limit: 2
+ # 1 minute
+ retry_delay: 60000
+ retry_on:
+ status:
+ - '5xx'
match:
meta:
uri:
'/^https?:\/\/[^\/]+\/api\/rest_v1\/page\/html\/([^\/]+)$/'
@@ -43,6 +49,12 @@
definition_rerender:
topic: resource_change
+ retry_limit: 2
+ # 1 minute
+ retry_delay: 60000
+ retry_on:
+ status:
+ - '5xx'
match:
meta:
# These URIs are coming from RESTBase, so we know that
article titles will be normalized
--
To view, visit https://gerrit.wikimedia.org/r/287148
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifaf44d1562681758e8d44ec7cc9fa1faf99374e5
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: GWicke <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
