search

Clon1998 / mobileraker_companion

Companion for mobileraker, enabling push notification.
MIT License
209 stars 12 forks source link

Possible memory leak in mobileraker.py #57

Closed austinrdennis closed 1 year ago

austinrdennis commented 1 year ago

Bug Report

Description

I've noticed a slow and gradual increase in the amount of memory mobileraker.py uses that points to a possible memory leak.

Steps to Reproduce

Boot the host machine and start the mobileraker service. Not sure what exactly the mechanism is.

Expected Behavior

Memory usage remains relatively stable over time and is reasonable.

Screenshots

SCR-20230909-ntbp SCR-20230909-nuls

Version Information

Debug Logs

mobileraker_2023-09-08T21-06-33.log mobileraker.log

Additional Context

After about 26 hours of runtime, it's now using around 2.1 GB of system memory which seems like way too much.

austinrdennis commented 1 year ago

I've also posted this as bug in the mobileraker repo as issue 251 as I wasn't sure which one to choose. Feel free to close this issue if it's not applicable this repo.

austinrdennis commented 1 year ago

sudo systemctl restart mobileraker.service clears out the memory usage at least temporarily. I'll see if it builds up again.

Clon1998 commented 1 year ago

Hey, thanks for reporting! This issue has been bogging me for quite a while now. However, I hope the fix I just pushed resolves it.

Duplicate of #27

Clon1998 commented 1 year ago

Please let me know if you observe any changes after updating to fa99513. Either in this issue or in #27