google / google/sam

Convert all logs to structured logging for consistency

Open
#331 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
846
Forks
128
Avg merge
23h 7m
Merged PRs (30d)
88

Description

PR [#328](https://github.com/google/sam/pull/328) converts the six [Mesh Event] log lines to structured Warnw/Infow so machine consumers can parse them. The remaining ~400 printf-style call sites (logger.Debugf/Infof/Warnf/Errorf) are still prose.

Convert them to the w-variants with key-value fields for consistency, so any line can be consumed programmatically without regexing message text.

Notes:

* Mechanical but large; best done in per-package chunks to keep review manageable.
* Reuse the field vocabulary started in #328(event, peer, timestamp, key) where applicable.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing PR #328 and inventorying the remaining logger.Debugf/Infof/Warnf/Errorf call sites, organizing the work into per-package chunks. Reuse the event, peer, timestamp, and key field vocabulary where applicable; done means the remaining printf-style call sites use structured w-variants without requiring regex parsing.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
observability
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.