consul-exporter leaks file descriptors

prometheus / consul_exporter

Exporter for Consul metrics

Apache License 2.0

440 stars 167 forks source link

consul-exporter leaks file descriptors #26

Closed discordianfish closed 7 years ago

discordianfish commented 8 years ago

Hi,

just debugged a stale consul-exporter instance which was running for a few weeks and saw it used all 4096 FDs. They were all used for socket connection. Probably the request body doesn't get closed properly.

grobie commented 7 years ago

Pull request #36 added a client timeout, which should fix this issue. Please re-open if you still experience this issue.