-
-
Notifications
You must be signed in to change notification settings - Fork 274
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
Provide full logs #2155
Comments
From an idea perspective, it's surely a nice to have Currently - logs are stored in-process - and are rotated out at around 1000 lines iirc - so increasing this to "unlimited" would introduce an (in this case intentional) memory leak - which could end up crashing the bot once it actually runs out of memory (we can't control the verbosity - or what logs a strategy writes). secondly, we can't rely on the user using Lastly (this is the frontend one) - assuming we have log-files available - you can't really fetch 100Mb of logs (of anything, really) into a frontend process and expect the browser / UI to still perform. If that user is just browsing through the logs - you'd you'd need time-based pagination based on ever-changing logs (The bot will continue to write logs), including eventual log rotation (which means you'll potentially load 10x 10MB files to get to the timestamp your'e looking for). Filtering get's actually worse - as a filter could return anything between 1 and 1000nds of rows (it's more like 600k, but still) - so you'd again need to filter and paginate - which again would mean loading the data from disk on every call (we don't want to keep this in memory for sure - we won't know if the user needs it, and it may be outdated by the time it's requested again) The idea - great |
Is your feature request related to a problem? Please describe.
The current freqUI Logs displays the most recent 500-1000 lines, which is sufficient for monitoring the bot's activity in real-time and over the past few hours. However this may be inadequate if you need to review the bot's status over a longer period or at specific moments in the past.
While users can achieve this by connecting to the server via SSH or implementing their custom solutions, it would be highly convenient if freqUI provided this functionality out of the box.
Describe the solution you'd like
The text was updated successfully, but these errors were encountered: