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

Logging seems to be split #286

Closed
jscheidtmann opened this issue Feb 19, 2025 · 2 comments
Closed

Logging seems to be split #286

jscheidtmann opened this issue Feb 19, 2025 · 2 comments

Comments

@jscheidtmann
Copy link
Contributor

DeadricGems reported a hang after a failed upgrade in Discord at this point:

Image

But the log file, that we could access via SMB was empty.

I seems that the log configuration is switched / wrong at that point in the boot process.

@jscheidtmann
Copy link
Contributor Author

I cannot find that "WiFi:Client" message in the source. Not even find find . -type f -print0 |xargs -0 grep -i wifi:client.

@brickbots
Copy link
Owner

The splash screen is handled in the file splash.py which is called via a different start up service before networking and other bits are online.

Even if it crashes, the main code should start up, so I don't think having proper logging on the splash screen is needed....

This still does not really help with why the software ended up in an un-usable state during the update and the main.py file crashed before logging anything 🤔 Given that the state of the code/sd card could end up very corrupted if the updated is terminated part way through I'm not surprised it can end up in some broken conditions.

Making the update process more robust, with a fail-safe rollback, is definitely needed

brickbots pushed a commit that referenced this issue Feb 24, 2025
* Make UIConsole a singleton.

* Add log statements for start of sub processes.

* Handle ValueError, if cedar_detect cannot be found.

* Log something at start of log process
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

No branches or pull requests

2 participants