-
Notifications
You must be signed in to change notification settings - Fork 350
Open
Description
Currently logrus.StandardLogger is used through the codebase.
One if the issues with it is that it is a global per process logger and there is no way to configure/pass logger to the components of the profiler (Controller/Tracer, etc)
Another issue is that it differs from the logging library in the collector. A quick grep says zap is used there. but we need to double check.
We should think of:
- Allowing passing/injecting the logger to the profiler components.
- Switching the implementation to
zap
Metadata
Metadata
Assignees
Labels
No labels