Skip to content

Conversation

@lidel
Copy link
Member

@lidel lidel commented Oct 28, 2025

This PR fixes #11035 and restores ipfs log tail and ipfs log level functionality for controlling libp2p log levels.

It adds test/cli regression test to catch any future regression if go-libp2p logs get broken again on go-libp2p update.

Context

Details in libp2p/go-libp2p#3418 and ipfs/go-log#176 (especially README).

These changes restore dynamic log level control and tail for go-libp2p subsystems after the migration to slog, fixing the regression introduced in libp2p/go-libp2p#3364

TODO

This PR depends on below PRs to land first:

update go-log to v2.8.3-0.20251027235339-5fe47a4191be (feat/slog-interop)
update go-libp2p to v0.44.1-0.20251027235033-ea2c010ece2d (feat/gologshim-use-slog-default)

these changes restore dynamic log level control and tail for go-libp2p
subsystems after the migration to slog, fixing the regression introduced
in libp2p/go-libp2p#3364

Fixes #11035
@lidel lidel requested a review from gammazero October 28, 2025 00:06
@lidel lidel added the skip/changelog This change does NOT require a changelog entry label Oct 28, 2025
- wrap cmd.Process.Kill() to explicitly ignore error
- fix const alignment (go fmt)
@lidel lidel marked this pull request as ready for review October 28, 2025 16:26
@lidel lidel requested a review from a team as a code owner October 28, 2025 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/changelog This change does NOT require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

go-libp2p v0.44 logs not present in ipfs log tail

3 participants