Ema has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/319609

Change subject: cache_text: switch to file storage backend on Varnish 4
......................................................................

cache_text: switch to file storage backend on Varnish 4

On Varnish 4, the persistent storage backend is buggy (T142810) and
deprecated. Switch cache_text backends to -sfile on Varnish 4.

See T142848 for the full reasoning behind the choice of dropping
persistent storage.

Bug: T142810
Bug: T131503
Ref: https://www.varnish-cache.org/docs/trunk/phk/persistent.html
Change-Id: I3e2b0616d325d6cef4e7259d6aee2a6914f7f3dc
---
M modules/role/manifests/cache/text.pp
1 file changed, 8 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/09/319609/1

diff --git a/modules/role/manifests/cache/text.pp 
b/modules/role/manifests/cache/text.pp
index ce6f535..00a6146 100644
--- a/modules/role/manifests/cache/text.pp
+++ b/modules/role/manifests/cache/text.pp
@@ -113,6 +113,13 @@
 
     $common_runtime_params = ['default_ttl=2592000']
 
+    if ($::role::cache::2layer::varnish_version4) {
+        $text_storage_args = $::role::cache::base::file_storage_args
+    }
+    else {
+        $text_storage_args = $::role::cache::base::persistent_storage_args
+    }
+
     role::cache::instances { 'text':
         fe_mem_gb         => ceiling(0.4 * $::memorysize_mb / 1024.0),
         fe_jemalloc_conf  => 'lg_dirty_mult:8,lg_chunk:19',
@@ -123,7 +130,7 @@
         be_vcl_config     => $be_vcl_config,
         fe_extra_vcl      => ['text-common', 'zero', 'normalize_path', 
'geoip'],
         be_extra_vcl      => ['text-common', 'normalize_path'],
-        be_storage        => $::role::cache::base::persistent_storage_args,
+        be_storage        => $text_storage_args,
         fe_cache_be_opts  => $fe_cache_be_opts,
         be_cache_be_opts  => $be_cache_be_opts,
         cluster_nodes     => hiera('cache::text::nodes'),

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3e2b0616d325d6cef4e7259d6aee2a6914f7f3dc
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ema <e...@wikimedia.org>

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

Reply via email to