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

[Feature]: network alerts limited to 125MB (1gbps) #468

Open
blinkinglight opened this issue Jan 22, 2025 · 7 comments
Open

[Feature]: network alerts limited to 125MB (1gbps) #468

blinkinglight opened this issue Jan 22, 2025 · 7 comments
Labels
enhancement New feature or request

Comments

@blinkinglight
Copy link

blinkinglight commented Jan 22, 2025

Describe the feature you would like to see

hey,
i am using 20G network for now and plan to migrate to 40G so i cant use alerts because they always exceeds alert limit ( 125M ( 1G ))

Describe how you would like to see this feature implemented

network limits up to 12500MB/s ( 100G )

@blinkinglight blinkinglight added the enhancement New feature or request label Jan 22, 2025
@blinkinglight blinkinglight changed the title [Feature]: network alerts limited to 125MB (10gbps) [Feature]: network alerts limited to 125MB (1gbps) Jan 22, 2025
@Glitch3dPenguin
Copy link

I also run a 10+ Gigabit network and think this would be a valuable change.

@e3ndr
Copy link

e3ndr commented Jan 23, 2025

Maybe Beszel could look at the link speed of all of the interfaces to determine the upper limit of the slider in the UI? I think that'd be a nice compromise as it prevents needing to have numbered inputs or have a range that is useless for some users.

@henrygd
Copy link
Owner

henrygd commented Jan 23, 2025

It makes sense to do this. The only problem is that the UI is not designed for numbers that high. Especially if we change network metrics to Mbps by default.

Let me think about the best way to go about it. For now you should be able to manually change the number to whatever you want by creating the alert, then editing it through the alerts table in PocketBase.

@blinkinglight
Copy link
Author

maybe text input for hand write MB/s ?

@blinkinglight
Copy link
Author

blinkinglight commented Jan 23, 2025

or add dropdown like "multiplicator" ? :) *1 *10 *100 ?

@e3ndr
Copy link

e3ndr commented Jan 23, 2025

or add dropdown like "multiplicator" ? :) *1 *10 *100 ?

Or, you could expand on this and have the dropdown be in units.

So you can select between Kbps, Mbps, Gbps, or even Tbps. And the slider can have a range of 0->1000 (and make the UI slightly wider to accommodate the larger range)

@henrygd
Copy link
Owner

henrygd commented Jan 23, 2025

So you can select between Kbps, Mbps, Gbps, or even Tbps. And the slider can have a range of 0->1000 (and make the UI slightly wider to accommodate the larger range)

That's a good idea, I think this could work.

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

No branches or pull requests

4 participants