search

anclrii / Storj-Exporter

Prometheus exporter for monitoring Storj storage nodes
GNU General Public License v3.0
58 stars 19 forks source link

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

Open aldelsa opened 10 months ago

aldelsa commented 10 months ago

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

teemogithub commented 9 months ago

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

TheChrisTech commented 9 months ago

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

TheChrisTech commented 9 months ago

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 commented 6 months ago

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).