BBlack has submitted this change and it was merged.

Change subject: raise fe mem size to 37% on text and upload
......................................................................


raise fe mem size to 37% on text and upload

This gets us halfway further from the current 25% values to the
50% values tested maps and misc (where the cache doesn't often
grow to full size anyways...).  With the new jemalloc tuning, the
virtual overhead shouldn't get unreasonable.

Bug: T135384
Change-Id: I05d0a59a72ea7b05367343e717b7f709fd2903b5
---
M modules/role/manifests/cache/text.pp
M modules/role/manifests/cache/upload.pp
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  BBlack: Verified; Looks good to me, approved



diff --git a/modules/role/manifests/cache/text.pp 
b/modules/role/manifests/cache/text.pp
index 2898553..ac914a5 100644
--- a/modules/role/manifests/cache/text.pp
+++ b/modules/role/manifests/cache/text.pp
@@ -108,7 +108,7 @@
     })
 
     role::cache::instances { 'text':
-        fe_mem_gb        => ceiling(0.25 * $::memorysize_mb / 1024.0),
+        fe_mem_gb        => ceiling(0.37 * $::memorysize_mb / 1024.0),
         fe_jemalloc_conf => 'lg_dirty_mult:8,lg_chunk_size:16',
         runtime_params   => ['default_ttl=2592000'],
         app_directors    => $app_directors,
diff --git a/modules/role/manifests/cache/upload.pp 
b/modules/role/manifests/cache/upload.pp
index 0953f7b..8a25f95 100644
--- a/modules/role/manifests/cache/upload.pp
+++ b/modules/role/manifests/cache/upload.pp
@@ -89,7 +89,7 @@
     ], ' ')
 
     role::cache::instances { 'upload':
-        fe_mem_gb        => ceiling(0.25 * $::memorysize_mb / 1024.0),
+        fe_mem_gb        => ceiling(0.37 * $::memorysize_mb / 1024.0),
         fe_jemalloc_conf => 'lg_dirty_mult:8,lg_chunk_size:17',
         runtime_params   => ['default_ttl=2592000'],
         app_directors    => $app_directors,

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

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

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

Reply via email to