-
Notifications
You must be signed in to change notification settings - Fork 2
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
disable debug logging #5
Comments
My node is fixed. The node's configuration file that handles this syslog connection is at: I suggest all nodes be patched. |
See related projects https://github.com/sudomesh/sudowrt-firmware and https://github.com/sudomesh/makenode . |
@wwwhtml and @jhpoelen i found the offending file in makenode. To reproduce:
Notice the line that reads,
Expected results:After removing the lines regarding logging in the file, /etc/config/system, you should no long see the connection to syslog server when you run |
Can confirm that latest commit to makenode fixes this issue on newly configured home nodes. |
@paidforby nice! |
@jhpoelen This is tagged with helpwanted. What kind is needed, given @paidforby last comment about makenode? @paidforby do you know off hand if latest firmware (including your epic noconfig improvements) also produces home nodes that don't debug log? |
@gobengo so this bug was originally introduced by makenode via the Of course, this was never an issue directly with the firmware repo, but I introduced that possibility with the noconfig additions, so you are right to ask about that. However, it is easy enough to check, you can see that files/etc/config/system does not reintroduce this bug. Additionally, there is no files/opt/mesh/templates/etc/config/system which would overwrite that first file after autoconfiging. Extender nodes also seme free of this bug by checking files_extender-node It should also be noted that new makenode does not introduce this either, since that is mostly just an offline method for triggering autoconf,. For future reference, this bug was triggered by the following lines in
I believe the conversation for leaving this opening had something to do with visibility for node patching, but the issue as a whole seems pretty well buttoned up. Though the N750 and TP-Link routers still need to be tested. My feeling is that as soon as we have N750s, or are able to consistently flash TP-Link routers, then we can reopen this issuie, if necessary. If anyone disagrees, feel free to reopen immediately. @gobengo thanks for pointing out that this was still open, despite being more or less resolved. |
Thanks for the detailed recap and lessons learned @paidforby . |
I believe
was also important too. Maybe the most important. There's some documentation about all these settings here: https://wiki.openwrt.org/doc/uci/system. -- @gobengo @paidforby I think this issue was left open because we have yet to write a patch that removes this config from existing nodes in the wild. I was hoping to write this patch, but I got stuck because the I opened a new issue here because there are still nodes in the wild that are calling home to log, which is a bug. |
from https://sudoroom.org/pipermail/mesh/2017-October/002600.html reported by D.
The text was updated successfully, but these errors were encountered: