Skip to content

chore(infra): remove dead honeycomb monitoring and log processor - #56226

Merged
vimtor merged 2 commits into
devfrom
remove-honeycomb-monitoring
Oct 17, 2026
Merged

vimtor merged 2 commits into
devfrom
remove-honeycomb-monitoring

Conversation

@vimtor

@vimtor vimtor commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Removes the SST-managed Honeycomb triggers, the Console /honeycomb/webhook route, and the LogProcessor tail worker.

The triggers query event_type = completions, which has had no data in Honeycomb for 90+ days since the log processor stopped exporting. Alerting now lives in the hand-made triggers on the inference dataset. Confirmed with Daniel.

LogProcessor lost its last destination when the S3 lake was retired (#52514) and only console.logs to a worker nobody tails. The Console no longer emits _metric: logs since /zen/* proxies to inference.

  • delete infra/monitoring.ts, infra/secret.ts, webhook route
  • drop DISCORD_INCIDENT_WEBHOOK_URL, HONEYCOMB_API_KEY, HoneycombWebhookSecret links
  • delete LogProcessor worker, Console tailConsumers, and packages/console/function

The honeycomb and random providers and HONEYCOMB_API_KEY in deploy.yml stay so the next production deploy can delete the resources. Remove them in a follow-up.

@opencode-agent opencode-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found no problems with this change. The console function and console app type-check cleanly. Nothing in the code still uses the removed resources or the /honeycomb/webhook route. I ran the old and new log processor tail handlers on the same fake event and they printed identical output, with later metrics still overriding earlier ones. Keeping the honeycomb and random providers and HONEYCOMB_API_KEY in deploy.yml means the next production deploy can still delete the triggers and the RandomPassword resource. I couldn't run a real deploy here because there are no credentials.

Public API: unchanged. This only touches infra, the internal Console webhook route and the log processor worker.

@vimtor vimtor changed the title chore(infra): remove dead honeycomb monitoring chore(infra): remove dead honeycomb monitoring and log processor Oct 17, 2026
@vimtor
vimtor enabled auto-merge (squash) October 17, 2026 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant