Storj-Exporter icon indicating copy to clipboard operation
Storj-Exporter copied to clipboard

Added timeout as env variable to avoid timeout on API Storj payout

Open aldelsa opened this issue 1 year ago • 4 comments

Added timeout as env variable to avoid timeout on API Storj payout. New Default 90s

Tip: You should increase your scrape_interval in prometheus to avoid saturation on your node

aldelsa avatar Nov 07 '23 15:11 aldelsa

Can we merge this? I need this update since there is an error with the timeout reading metric in two of my nodes.

teemogithub avatar Dec 13 '23 10:12 teemogithub

I've been hitting this too. API Timeout would definitely help. @anclrii - Tagging for visibility.

TheChrisTech avatar Dec 13 '23 14:12 TheChrisTech

I've tested your changes by manually updating the __main__.py within my docker deployment (yikes), and was able to successfully get data to return (Took roughly 28 seconds). I've also needed to update the following within my Prometheus config for the individual JOB that scrapes the endpoint presented:

    scrape_timeout:     90s
    scrape_interval:    90s

and this works as expected. My dashboard now repopulated that Storj node, and metrics were as expected.

TheChrisTech avatar Dec 23 '23 16:12 TheChrisTech

I've implemented this change and pushed to docker here: https://github.com/TheChrisTech/Storj-Exporter (I don't want to drive traffic from this repo, but I got tired of waiting).

TheChrisTech avatar Mar 11 '24 18:03 TheChrisTech