-
Notifications
You must be signed in to change notification settings - Fork 33
[Telemetry] Apply shared logging module #288
Copy link
Copy link
Closed
Labels
telemetryeverything related to collection telemetryeverything related to collection telemetrytoolingtooling to support development, testing et altooling to support development, testing et al
Milestone
Description
Objective
Use the foundations to unify and standardize the logging approach across the entire codebase created in #164 and apply to a subset of files to use this framework.
Origin Document
See the github discussion for full details: #214
The implementation of this was started by @derrandz here: #162.
This ticket can either build on top of the existing WIP or use those learnings and apply them to a different approach.
Goals / Deliverables
- Select subset of files to use the same logging framework
- Add support for multiple log levels
- Add support for attaching labels to logs
-
Build a log panel in Grafana where logs can be toggled based on level or label(covered in [Telemetry] Dashboard for LocalNet logs viewing & filtering #189)
General issue checklist
- Update the appropriate CHANGELOG
- Update the appropriate README on how to use and test this feature
- Update any relevant global documentation & references
- Add a video and/or gif on how to use feature
- Document small issues / TODOs along the way
Non-goals
- Add new logs throughout the codebase
- Improve the explicitness or correctness of existing logs
- Add new business logic functionality
Testing Methodology
- All tests:
make test_all - LocalNet: verify a
LocalNetis still functioning correctly by following the instructions at docs/development/README.md
Remove
Creator: @Olshansk
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
telemetryeverything related to collection telemetryeverything related to collection telemetrytoolingtooling to support development, testing et altooling to support development, testing et al
Type
Projects
Status
Done