Ori.livneh has submitted this change and it was merged.
Change subject: varnishstatsd: don't report stats for bogus HTTP status codes
......................................................................
varnishstatsd: don't report stats for bogus HTTP status codes
Change-Id: Icbdf0d499e352cbacb16d1894769cf4c5670cd4d
---
M modules/varnish/files/varnishstatsd
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Ori.livneh: Verified; Looks good to me, approved
Filippo Giunchedi: Looks good to me, but someone else must approve
diff --git a/modules/varnish/files/varnishstatsd
b/modules/varnish/files/varnishstatsd
index da2a512..68156f3 100755
--- a/modules/varnish/files/varnishstatsd
+++ b/modules/varnish/files/varnishstatsd
@@ -113,7 +113,8 @@
elif tag == 'RxStatus' and remote_party == 'backend':
# Associate the transaction ID with the response status code.
- statuses[transaction_id] = record[0] + 'xx'
+ if record[0] in '12345':
+ statuses[transaction_id] = record[0] + 'xx'
if len(statuses) > MAX_BACKLOG:
statuses.popitem(last=False)
--
To view, visit https://gerrit.wikimedia.org/r/215370
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icbdf0d499e352cbacb16d1894769cf4c5670cd4d
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ori.livneh <o...@wikimedia.org>
Gerrit-Reviewer: Filippo Giunchedi <fgiunch...@wikimedia.org>
Gerrit-Reviewer: Ori.livneh <o...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
