Skip to content

Commit

Permalink
[varnish] bugfix, fix arg in _parse_varnishstat
Browse files Browse the repository at this point in the history
Another change from #1213 introduced a regression by passing the
path of the command instead of the output of the comand.
  • Loading branch information
mms-gianni authored and LeoCavaille committed Feb 19, 2015
1 parent 28a69b4 commit 7d801e5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion checks.d/varnish.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ def check(self, instance):
output, error = proc.communicate()
if error and len(error) > 0:
self.log.error(error)
self._parse_varnishstat(varnishstat_path, use_xml, tags)
self._parse_varnishstat(output, use_xml, tags)

# Parse service checks from varnishadm.
varnishadm_path = instance.get('varnishadm')
Expand Down

0 comments on commit 7d801e5

Please sign in to comment.