Faidon Liambotis has submitted this change and it was merged. Change subject: Varnish: add German version of CentralAutoLogin ......................................................................
Varnish: add German version of CentralAutoLogin Spezial:Zentrale_automatische_Anmeldung URLs are getting hashed to cp1065 and contention is causing requests to them occasionally fail. For the period of Nov 26th-29th, those URLs currently account for 387.375 503s, i.e. ~48% of all 503s: 264244 http://de.wikipedia.org/w/index.php?title=Spezial:Zentrale_automatische_Anmeldung/start&type=script 123131 http://de.wikipedia.org/w/index.php?title=Spezial:Zentrale_automatische_Anmeldung/createSession&gu_id=0&type=script&proto=https Set backend_random to those too, similarly to what we've done to the English version of CentralAutoLogin URLs. Change-Id: I24419f378dc0e9aacd5ff7dab9dda891d36084ff --- M templates/varnish/text-backend.inc.vcl.erb M templates/varnish/text-frontend.inc.vcl.erb 2 files changed, 5 insertions(+), 3 deletions(-) Approvals: Faidon Liambotis: Looks good to me, approved jenkins-bot: Verified diff --git a/templates/varnish/text-backend.inc.vcl.erb b/templates/varnish/text-backend.inc.vcl.erb index 9fc421f..9184b3f 100644 --- a/templates/varnish/text-backend.inc.vcl.erb +++ b/templates/varnish/text-backend.inc.vcl.erb @@ -40,7 +40,8 @@ call pass_requests; <% if vcl_config.fetch("cluster_tier", "1") != "1" -%> - if (req.url ~ "^/wiki/Special:CentralAutoLogin/") { + if (req.url ~ "^/wiki/Special:CentralAutoLogin/" || + req.url ~ "^/w/index.php?title=Spezial:Zentrale_automatische_Anmeldung/") { set req.backend = <%= @vcl_config.fetch("default_backend", "backend") %>_random; } <% end -%> @@ -88,4 +89,4 @@ sub vcl_error { call errorpage; return(deliver); -} \ No newline at end of file +} diff --git a/templates/varnish/text-frontend.inc.vcl.erb b/templates/varnish/text-frontend.inc.vcl.erb index c41b87e..39a75ed 100644 --- a/templates/varnish/text-frontend.inc.vcl.erb +++ b/templates/varnish/text-frontend.inc.vcl.erb @@ -68,7 +68,8 @@ call mobile_redirect; call pass_requests; - if (req.url ~ "^/wiki/Special:CentralAutoLogin/") { + if (req.url ~ "^/wiki/Special:CentralAutoLogin/" || + req.url ~ "^/w/index.php?title=Spezial:Zentrale_automatische_Anmeldung/") { set req.backend = backend_random; } -- To view, visit https://gerrit.wikimedia.org/r/98166 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I24419f378dc0e9aacd5ff7dab9dda891d36084ff Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Faidon Liambotis <fai...@wikimedia.org> Gerrit-Reviewer: Faidon Liambotis <fai...@wikimedia.org> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
