Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

aldelsa
Copy link

@aldelsa aldelsa commented Nov 7, 2023

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
Copy link

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

@TheChrisTech
Copy link

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

@TheChrisTech
Copy link

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
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants