oban-bg / oban

💎 Robust job processing in Elixir, backed by modern PostgreSQL and SQLite3
https://oban.pro
Apache License 2.0
3.37k stars 314 forks source link

Request to rename [:oban, :notifier, :switch] emitted log attribute `status` to `connectivity_status` #1086

Closed punchcafe closed 6 months ago

punchcafe commented 6 months ago

Hello Again!

This is a small change request to make the default telemetry logger integrate nicer with Datadog.

The logs emitted in response to [:oban, :notifier, :switch] events have a status field, which is a reserved field for JSON logs with Datadog. The result was that all :switch events were being treated as critical logs and raising alerts. I imagine a lot of Oban users use Datadog so I thought it may be worth making this small tweak to avoid conflict.

Thanks, Luca

sorentwo commented 6 months ago

@punchcafe Thanks! This is a welcome change; whatever we can do to ease integration with external tools is wonderful.