Page MenuHomePhabricator

varnish: implement FetchError logging
Closed, ResolvedPublic

Description

We currently have no way of knowing the cause of Varnish fetch errors other than observing them live with varnishlog -g request -q 'FetchError ne "Pass delivery abandoned"'

Keep track of FetchError causes by sending the logs to logstash.

See https://varnish-cache.org/tips/varnishlog/fetcherror.html and https://varnish-cache.org/docs/5.1/reference/vsl.html

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
ema triaged this task as Medium priority.Jun 4 2019, 2:36 PM
ema renamed this task from varnish: add X-Fetch-Error response header to varnish: implement FetchError logging .Jun 17 2019, 7:44 AM
ema updated the task description. (Show Details)

Change 517375 had a related patch set uploaded (by Ema; owner: Ema):
[operations/puppet@production] varnishlog: implement varnishfetcherr

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

The initial plan of adding a synthetic header to varnish with the FetchError cause seems a little too complicated to implement. Send error logs to logstash instead.

Change 517375 merged by Ema:
[operations/puppet@production] varnishlog: implement varnishfetcherr

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

Change 517649 had a related patch set uploaded (by Ema; owner: Ema):
[operations/puppet@production] varnishfetcherr: do send logs, handle interesting tags

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

Change 517649 merged by Ema:
[operations/puppet@production] varnishfetcherr: do send logs, handle interesting tags

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

Change 517811 had a related patch set uploaded (by Ema; owner: Ema):
[operations/puppet@production] varnishfetcherror: do not overwrite multiple tags

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

Change 517811 merged by Ema:
[operations/puppet@production] varnishfetcherror: do not overwrite multiple tags

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