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

[Bug] Using a high value for DASHDOT_SPEED_TEST_INTERVAL causes issues #927

Closed
lopazopy opened this issue Oct 9, 2023 · 2 comments
Closed
Assignees

Comments

@lopazopy
Copy link

lopazopy commented Oct 9, 2023

Description of the bug

I wanted to run the speed test 1 time a month and entered a value of 43800. When I made this change, CPU and RAM went to 100% with network downloading at full speed and my system would freeze up. Finally, after removing the environment variable, things went back to normal.

I know how to "disable" the speed test by using a local file now.

How to reproduce

Running dash using docker compose on Linux Mint 21.1

environment:
DASHDOT_SPEED_TEST_INTERVAL: '43800'

Relevant log output

No response

Info output of dashdot cli

No response

What browsers are you seeing the problem on?

Firefox

Where is your instance running?

Desktop PC (Linux)

Additional context

No response

@MauriceNino
Copy link
Owner

Hi there, that is very weird, I will look into that when I get some time.

@MauriceNino MauriceNino self-assigned this Oct 25, 2023
@MauriceNino
Copy link
Owner

Okay, I have now investigated this and came to the conclusion that this happens because the number is actually too large. The maximum number javascript can handle for setInterval is 2.147.483.647 according to this. The number you passed resolves to 2.628.000.000.

Unfortunately, there is not much I can do about it for now.

But there is a feature task that plans to use a cron-like syntax to start the speed-test. If you are in need of such a feature, you can follow the progress there: #889

@MauriceNino MauriceNino closed this as not planned Won't fix, can't repro, duplicate, stale Jan 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants