Skip to content

Add per-room bandwidth limits. - #1107

Draft
fancycode wants to merge 15 commits into
masterfrom
room-bandwidth
Draft

Add per-room bandwidth limits.#1107
fancycode wants to merge 15 commits into
masterfrom
room-bandwidth

Conversation

@fancycode

Copy link
Copy Markdown
Member

No description provided.

@fancycode fancycode linked an issue Nov 5, 2025 that may be closed by this pull request
@codecov

codecov Bot commented Nov 6, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 65.66524% with 160 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.87%. Comparing base (69e616b) to head (2b4365f).
⚠️ Report is 62 commits behind head on master.

Files with missing lines Patch % Lines
sfu/proxy/testserver/server.go 0.00% 42 Missing ⚠️
cmd/proxy/proxy_server.go 57.77% 12 Missing and 7 partials ⚠️
server/room.go 84.67% 12 Missing and 7 partials ⚠️
server/clientsession.go 61.53% 12 Missing and 3 partials ⚠️
sfu/janus/client.go 0.00% 13 Missing ⚠️
talk/backend.go 40.00% 12 Missing ⚠️
cmd/proxy/proxy_session.go 77.55% 10 Missing and 1 partial ⚠️
sfu/test/sfu.go 0.00% 9 Missing ⚠️
grpc/server.go 81.25% 3 Missing and 3 partials ⚠️
sfu/proxy/proxy.go 86.11% 3 Missing and 2 partials ⚠️
... and 5 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1107      +/-   ##
==========================================
- Coverage   62.89%   62.87%   -0.03%     
==========================================
  Files         129      129              
  Lines       18807    19246     +439     
==========================================
+ Hits        11829    12101     +272     
- Misses       5885     6021     +136     
- Partials     1093     1124      +31     
Flag Coverage Δ
go-1.25 62.79% <65.66%> (-0.06%) ⬇️
go-1.26 62.87% <65.66%> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
root ∅ <ø> (∅)
api 81.70% <ø> (ø)
async 86.77% <ø> (ø)
client 70.38% <ø> (ø)
cmd/client 6.34% <ø> (ø)
cmd/proxy 60.89% <68.08%> (+0.29%) ⬆️
cmd/server 0.00% <ø> (ø)
config 82.14% <ø> (ø)
container 93.79% <ø> (ø)
dns 90.82% <ø> (ø)
etcd 81.96% <ø> (ø)
geoip 68.09% <ø> (ø)
grpc 75.81% <80.39%> (+0.24%) ⬆️
internal 96.12% <ø> (ø)
log 94.44% <ø> (ø)
metrics 90.00% <ø> (ø)
mock ∅ <ø> (∅)
nats 73.55% <ø> (ø)
pool 81.91% <ø> (ø)
proxy 81.35% <ø> (ø)
security 77.50% <ø> (-1.25%) ⬇️
server 72.32% <79.66%> (-0.12%) ⬇️
session 78.57% <ø> (ø)
sfu 43.48% <32.03%> (-0.18%) ⬇️
talk 69.11% <65.85%> (+0.12%) ⬆️
test 86.04% <ø> (ø)
Files with missing lines Coverage Δ
etcd/api.go 100.00% <ø> (ø)
proxy/api.go 81.35% <ø> (ø)
server/session.go 50.00% <ø> (ø)
sfu/common.go 0.00% <ø> (ø)
sfu/janus/janus.go 57.54% <66.66%> (+0.06%) ⬆️
grpc/client.go 72.83% <88.23%> (+0.44%) ⬆️
grpc/test/server.go 83.72% <0.00%> (-4.09%) ⬇️
server/hub.go 67.79% <85.71%> (-0.15%) ⬇️
talk/backend_storage_static.go 84.48% <90.47%> (+0.59%) ⬆️
sfu/proxy/proxy.go 63.83% <86.11%> (+0.53%) ⬆️
... and 9 more

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fancycode
fancycode force-pushed the room-bandwidth branch 2 times, most recently from 2137334 to 115593c Compare November 6, 2025 12:06
@fancycode
fancycode force-pushed the room-bandwidth branch 9 times, most recently from e80fa32 to 4315384 Compare November 20, 2025 09:55
@fancycode
fancycode force-pushed the room-bandwidth branch 2 times, most recently from c60a802 to 13ed3b5 Compare November 24, 2025 14:23
@fancycode
fancycode force-pushed the room-bandwidth branch 4 times, most recently from c9311b0 to 7a168ae Compare December 5, 2025 10:47
@fancycode
fancycode force-pushed the room-bandwidth branch 2 times, most recently from 313e0d3 to a08381a Compare January 13, 2026 15:14
@fancycode
fancycode force-pushed the room-bandwidth branch 2 times, most recently from 2587cf2 to ea1a767 Compare April 9, 2026 14:33
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.

Dynamically adjust bandwidth while in call

1 participant