Skip to content

feat(gossip): packet sanitize fail - switch error log to prometheus#1262

Open
dnut wants to merge 4 commits intomainfrom
dnut/gossip-sanitize-observability
Open

feat(gossip): packet sanitize fail - switch error log to prometheus#1262
dnut wants to merge 4 commits intomainfrom
dnut/gossip-sanitize-observability

Conversation

@dnut
Copy link
Contributor

@dnut dnut commented Feb 26, 2026

previously these were logged as errors which caused alert spam. these are inputs from the public internet and are expected to be malformed occasionally, so we shouldn't get spammed with alerts for this. i changed the error log to warn and moved it to counters in prometheus and added them to the gossip dashboard.

prevoiusly these were logged as errors which caused alert spam. these are inputs from the public internet and are expected to be malformed so we shouldn't get spammed with alerts for this. i changed the err to warn and moved it to counters in prometheus and added them to the gossip dashboard.
@dnut dnut requested review from ultd and yewman as code owners February 26, 2026 21:35
@github-project-automation github-project-automation bot moved this to 🏗 In progress in Sig Feb 26, 2026
Sobeston
Sobeston previously approved these changes Feb 26, 2026
@dnut dnut enabled auto-merge February 26, 2026 22:54
@github-project-automation github-project-automation bot moved this from 🏗 In progress to 👀 In review in Sig Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

3 participants