Skip to content

Upload Throughput monitor - #1044

Open
DennisGarvey wants to merge 10 commits into
marcone:main-devfrom
DennisGarvey:bandwidth
Open

Upload Throughput monitor#1044
DennisGarvey wants to merge 10 commits into
marcone:main-devfrom
DennisGarvey:bandwidth

Conversation

@DennisGarvey

Copy link
Copy Markdown

Added upload speed monitor on web dashboard. Useful for ensuring archiving is happening at an adequate speed or to diagnose connectivity to archive.

DennisGarvey and others added 7 commits April 26, 2026 23:15
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
@DennisGarvey DennisGarvey changed the title Bandwidth monitor Upload Throughput monitor May 5, 2026
}

t1=$(get_tx_bytes)
sleep 1

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it seems better to get one value and return it immediately (and do calculation of delta/speed on the web interface side) so as to not block the CGI call for a whole second, especially since the web interface fires off such a call once a second

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One thing I was worried about was handling timing in the browser because variances in the request/response will cause drift in the measurement

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will try it though

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay it seems good enough now. I added timestamps on each request response

DennisGarvey and others added 2 commits May 5, 2026 23:09
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>

@marcone marcone left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this show when it's not archiving? I haven't tried it yet, but if I'm reading the code correctly it will display any value >= 0, so just looking at the tab would be showing something like "Current Upload: 0.2 KB/s (0.0 Mbit/s)" just from refreshing the web interface, which doesn't seem super useful and may be confusing.

@DennisGarvey

DennisGarvey commented May 6, 2026

Copy link
Copy Markdown
Author

Yep
image
I think its clunky to just add a threshold to it. I am also concerned about possible flicker near the threshold. Maybe once an archiving status flag is added to status.sh, this line can be conditionally displayed? Also if archiving is supposed to happen and there is genuinely something wrong that causes it to be near 0, it should be displayed.

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.

2 participants