BBlack has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/387225 )

Change subject: cache_text: reduce applayer timeouts to reasonable values
......................................................................


cache_text: reduce applayer timeouts to reasonable values

This is a more-moderate replacement for the emergency changes over
the weekend in 40b19a79 .

connect_timeout: 3s should be more than enough for reasonable
conditions, and we can fail a little faster in the bad cases than
at the previous 5s value.

first_byte_timeout: HHVM's current max execution time is 60s.  Set
our ttfb limit to just a bit over that at 63s for all text
services (MW, RB, cxserver).

between_bytes_timeout:  31s here is basically half the above.
Tuning this one is difficult, but we didn't see any real
complaints over the weekend when we had it emergency-set to 10s,
either.

What's missing here (from varnishd) is an overall timeout on
backend transaction completion (receipt of whole response).

Bug: T179156
Change-Id: Ic45360730d7da7cfed61e9df89678b340d40b77e
---
M hieradata/role/common/cache/text.yaml
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Giuseppe Lavagetto: Looks good to me, but someone else must approve
  Ema: Looks good to me, but someone else must approve
  BBlack: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/hieradata/role/common/cache/text.yaml 
b/hieradata/role/common/cache/text.yaml
index ee10a2b..3f1330f 100644
--- a/hieradata/role/common/cache/text.yaml
+++ b/hieradata/role/common/cache/text.yaml
@@ -21,9 +21,9 @@
   esams: 'eqiad'
 cache::app_def_be_opts:
   port: 80
-  connect_timeout: '5s'
-  first_byte_timeout: '180s'
-  between_bytes_timeout: '60s'
+  connect_timeout: '3s'
+  first_byte_timeout: '63s'
+  between_bytes_timeout: '31s'
   max_connections: 1000
 cache::app_directors:
   appservers:

-- 
To view, visit https://gerrit.wikimedia.org/r/387225
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic45360730d7da7cfed61e9df89678b340d40b77e
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: BBlack <[email protected]>
Gerrit-Reviewer: BBlack <[email protected]>
Gerrit-Reviewer: Ema <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to