https://github.com/wikimedia/operations-puppet
Revision 5be836ab0b59314c5e4fe8dd43f4c583135a7c5a authored by Ori Livneh on 15 September 2015, 20:11:32 UTC, committed by BBlack on 21 September 2015, 21:51:45 UTC
The 1024 byte limit is a tad too short for some EventLogging URLs.
Internet Explorer up to and including version 10 limits URLs to 2083
characters, so the 2048 limit would not only be sufficient for now, but
for the foreseeable future as well.

This also increases shm_workspace from the default of 8192 to
16384.  varnishstat on various clusters indicates we were already
having to flush this fairly regularly (in some cases, up to ~4x
per second), so it probably deserves an increase anyways.  Because
not all URLs are of (either) maximum size, doubling the workspace
should more than make up for any shm flush impact from the
shm_reclen increase, and flush rate should still drop after this
change.

Note that we should apply these parameters via varnish daemon
restarts slowly across the clusters, rather than at runtime, as
the timing and sanity of increasing these at runtime via
varnishadm is questionable.

Bug: T112002
Bug: T91347

Change-Id: I8f398936350d0e240a1b55d6a12076a2fd341935
1 parent 0b7f024
History
Tip revision: 5be836ab0b59314c5e4fe8dd43f4c583135a7c5a authored by Ori Livneh on 15 September 2015, 20:11:32 UTC
Increase Varnish's `shm_reclen` from 1024 to 2048
Tip revision: 5be836a
File Mode Size
conftool-data
doc
files
hieradata
manifests
modules
nodes
templates
utils
.ctags -rw-r--r-- 323 bytes
.gitignore -rw-r--r-- 431 bytes
.gitmodules -rw-r--r-- 1.1 KB
.gitreview -rw-r--r-- 101 bytes
.puppet-lint.rc -rw-r--r-- 942 bytes
.rubocop.yml -rw-r--r-- 639 bytes
.rubocop_todo.yml -rw-r--r-- 14.3 KB
Gemfile -rw-r--r-- 74 bytes
Gemfile.lock -rw-r--r-- 446 bytes
README -rw-r--r-- 783 bytes
rakefile -rw-r--r-- 5.1 KB
tox.ini -rw-r--r-- 165 bytes
typos -rw-r--r-- 69 bytes

README

back to top