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

CPU usage increases with time in v2.2.0<= #36

Closed
jakjus opened this issue Dec 15, 2024 · 3 comments
Closed

CPU usage increases with time in v2.2.0<= #36

jakjus opened this issue Dec 15, 2024 · 3 comments
Assignees
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@jakjus
Copy link
Contributor

jakjus commented Dec 15, 2024

Bug reason is not yet confirmed and this is opened to keep a track on it.

Describe the bug
There is an OBSERVATION raised by @mostoview and myself. CPU usage increases with time in v2.2.0<= , possibly due to changed wrtc library (last properly working version is v2.1.3)
I have noticed increased CPU usage as well with newer haxball.js versions, but not yet confirmed the reason.

CPU load rises with empty rooms with time, after one day increasing ~100x (e.g. from 0.4% to 40%).

To Reproduce
Run a room with haxball.js==2.1.3 and haxball.js@latest and observe the process CPU usage.

Expected behavior
Not increasing CPU load.

Device (please complete the following information):

  • node 18, node 20, node 22
  • ubuntu 24, ubuntu 22

Additional context
Benchmarking the process between versions will take some time, but Issue is opened to see if someone has a similar overview and could share more findings.

@jakjus jakjus added the bug Something isn't working label Dec 15, 2024
@jakjus jakjus changed the title CPU usage increases with time in v2.1.4<= CPU usage increases with time in v2.2.0<= Dec 15, 2024
@jakjus
Copy link
Contributor Author

jakjus commented Dec 16, 2024

I'm currently running all servers with koush/wrtc and I didn't notice any CPU usage improvement.

@mertushka mertushka added the help wanted Extra attention is needed label Dec 17, 2024
@mertushka
Copy link
Owner

Please follow-up in #38

@jakjus
Copy link
Contributor Author

jakjus commented Jan 6, 2025

I didn't come to any conclusion, as I didn't have time to test it, so I'll close it. You may reopen if you find it reasonable.

@jakjus jakjus closed this as completed Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants