Skip to content

increase the default log level

Charles Hall requested to merge charles/tracing-levels into main

Having the default log level be at warn means that we omit almost all span information from the logs, which is not great. It also incentivizes programmers to use warn!() by default so their events actually show up, which demonstrably results in miscategorized events, some of which are cleaned up here.

To-do list

  • Review event info!("Upgrading {} to timeline pdu", incoming_pdu.event_id)
    • Reduced to debug
  • Review event warn!(error = %e, "Invalid 200 response",)
    • Reduced to debug
  • Review event info!(origin = %origin, "Returning stale keys")
    • Not sure what to make of this one; leaving it alone for now
  • Review event info!("Returning an error: {}: {}", status_code, message)
    • Promoted to warn, changed to be structured
  • Figure out if it's possible to make the stdout logger drop otel.name spans
  • Other stuff
Edited by Charles Hall

Merge request reports