sorentwo / oban

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

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

Closed punchcafe closed 1 month ago

punchcafe commented 1 month 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 1 month ago

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