Giuseppe Lavagetto has submitted this change and it was merged.

Change subject: varnish: remove redirection to the hhvm pool
......................................................................


varnish: remove redirection to the hhvm pool

Change-Id: I39064328aabd1e7e5147e464e879a22e39c839fb
Signed-off-by: Giuseppe Lavagetto <glavage...@wikimedia.org>
---
M templates/varnish/bits.inc.vcl.erb
M templates/varnish/mobile-backend.inc.vcl.erb
M templates/varnish/text-backend.inc.vcl.erb
3 files changed, 0 insertions(+), 26 deletions(-)

Approvals:
  Giuseppe Lavagetto: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/templates/varnish/bits.inc.vcl.erb 
b/templates/varnish/bits.inc.vcl.erb
index 86ebb26..6f4e20e 100644
--- a/templates/varnish/bits.inc.vcl.erb
+++ b/templates/varnish/bits.inc.vcl.erb
@@ -39,12 +39,6 @@
                error 403 "HTTP method not allowed.";
        }
 
-<% if vcl_config.fetch("cluster_tier", "1") == "1" -%>
-       if ( req.http.Cookie ~ "hhvm=true" ) {
-                       set req.backend = hhvm_appservers;
-       }
-<% end -%>
-
        if (req.http.host == "<%= cluster_options.fetch( "bits_domain", 
"bits.wikimedia.org" )%>") {
        /* For https-only wikis, the redirect from http to https for bits 
assets should occur
                in varnish instead of apache, since the apache redirect and 
mediawiki doesn't
diff --git a/templates/varnish/mobile-backend.inc.vcl.erb 
b/templates/varnish/mobile-backend.inc.vcl.erb
index 39dd29c..36e7ca4 100644
--- a/templates/varnish/mobile-backend.inc.vcl.erb
+++ b/templates/varnish/mobile-backend.inc.vcl.erb
@@ -19,17 +19,6 @@
        }
 <% end -%>
 
-<% if vcl_config.fetch("cluster_tier", "1") == "1" -%>
-       /* Use the hhvm backend for any request to the standard appservers */
-       if ( req.http.X-Orig-Cookie ~ "hhvm=true" ) {
-               if (req.backend == api) {
-                       set req.backend = hhvm_api;
-               } else {
-                       set req.backend = hhvm_appservers;
-               }
-       }
-<% end -%>
-
        /* Default (now modified) Varnish vcl_recv function */
        if (req.request != "GET" && req.request != "HEAD") {
                /* We only deal with GET and HEAD by default */
diff --git a/templates/varnish/text-backend.inc.vcl.erb 
b/templates/varnish/text-backend.inc.vcl.erb
index 9e27f3c..30f23e3 100644
--- a/templates/varnish/text-backend.inc.vcl.erb
+++ b/templates/varnish/text-backend.inc.vcl.erb
@@ -36,15 +36,6 @@
        if (req.http.host ~ "^test\.") {
                set req.backend = test_wikipedia;
        }
-
-       /* Use the hhvm backend for any request to the standard appservers. */
-       if ( req.http.Cookie ~ "hhvm=true" ) {
-               if (req.backend == api) {
-                       set req.backend = hhvm_api;
-               } else {
-                       set req.backend = hhvm_appservers;
-               }
-       }
 <% end -%>
 
        call pass_requests;

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I39064328aabd1e7e5147e464e879a22e39c839fb
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Giuseppe Lavagetto <glavage...@wikimedia.org>
Gerrit-Reviewer: BBlack <bbl...@wikimedia.org>
Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org>
Gerrit-Reviewer: Mark Bergsma <m...@wikimedia.org>
Gerrit-Reviewer: Ori.livneh <o...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to