containerd / containerd/log

consider replacing logrus

Open
#2 8 comments 8 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
8
Forks
6
Avg merge
1d 16h
Merged PRs (30d)
7

Description

### What is the problem you're trying to solve

[logrus](https://github.com/sirupsen/logrus) is in maintenance mode and is no longer receiving significant development beyond security and bugfixes. Structured logging has continued to evolve in Go and recently the standard-library `log/slog` package was [accepted](https://github.com/golang/go/issues/56345#issuecomment-1470506816) for inclusion in a future version of Go.

### Describe the solution you'd like

With containerd 2.0, we should consider replacing logrus with an alternative. Some possibilities:

* `log/slog` (via https://pkg.go.dev/golang.org/x/exp/slog for now, until it is integrated in the standard library)
* https://github.com/rs/zerolog
* https://github.com/uber-go/zap
* https://github.com/apex/log

### Additional context

Replacing logrus has changes for clients of containerd (such as Moby) and plugins like shims. We'd also need to adapt containerd's logging helper (`log.G(ctx)`) to work with a new logging implementation.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.