BBlack has submitted this change and it was merged.
Change subject: re-disable compact_memory cron for jessie caches
......................................................................
re-disable compact_memory cron for jessie caches
Doesn't seem to be causing the cp1064 issues
See also: b7c51400 f8f989df
Change-Id: I2b56966a42980e3c6c6ef5e78aa82a6af74c85ef
---
M manifests/role/cache.pp
1 file changed, 8 insertions(+), 6 deletions(-)
Approvals:
BBlack: Verified; Looks good to me, approved
diff --git a/manifests/role/cache.pp b/manifests/role/cache.pp
index b50bac4..a2267c9 100644
--- a/manifests/role/cache.pp
+++ b/manifests/role/cache.pp
@@ -710,12 +710,14 @@
'labs' => ['vdb'],
}
- # This seems to prevent long term memory fragmentation issues that
- # result in XFS deadlock log spam + bad problems for varnish health
- cron { 'varnish_vm_compact_cron':
- command => 'echo 1 >/proc/sys/vm/compact_memory',
- user => 'root',
- minute => '*',
+ unless os_version('debian >= jessie') { # testing whether we need this
for jessie's 3.16 kernel
+ # This seems to prevent long term memory fragmentation issues that
+ # result in XFS deadlock log spam + bad problems for varnish
health
+ cron { 'varnish_vm_compact_cron':
+ command => 'echo 1 >/proc/sys/vm/compact_memory',
+ user => 'root',
+ minute => '*',
+ }
}
#class { "varnish::packages": version => "3.0.3plus~rc1-wm5" }
--
To view, visit https://gerrit.wikimedia.org/r/188973
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2b56966a42980e3c6c6ef5e78aa82a6af74c85ef
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: BBlack <[email protected]>
Gerrit-Reviewer: BBlack <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
