Skip to content

Improve logging in the codebase #417

@korniltsev

Description

@korniltsev

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

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions