-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add nodeid to logs in OperationalDeviceProxy.cpp #21019
Add nodeid to logs in OperationalDeviceProxy.cpp #21019
Conversation
PR #21019: Size comparison from 4a56906 to f213acf Increases (36 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Decreases (4 builds for cc13x2_26x2)
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
PR #21019: Size comparison from 4a56906 to 4a95a1d Increases (5 builds for esp32, mbed, nrfconnect)
Decreases (1 build for esp32)
Full report (5 builds for esp32, mbed, nrfconnect)
|
PR #21019: Size comparison from 6d54807 to 0d57958 Increases (25 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Decreases (5 builds for cc13x2_26x2, cyw30739)
Full report (32 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use more consistent and complete logging format as already defined in this file.
0d57958
to
4ab8eb8
Compare
PR #21019: Size comparison from 1d88b32 to 4ab8eb8 Increases (40 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Decreases (6 builds for cc13x2_26x2)
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
I updated the logs to what was requested.
Problem
Some logs would benefit from knowing the nodeid for which they occur.
Fixes #16140
Change overview
Added extra logging (added nodeid to logs)
Testing
Compile only - change is trivial.