Page MenuHomePhabricator

[Identity] Add tracing to the Identity service
ClosedPublic

Authored by varun on Apr 25 2022, 8:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 5, 7:19 PM
Unknown Object (File)
Thu, Oct 31, 8:45 PM
Unknown Object (File)
Thu, Oct 31, 8:35 PM
Unknown Object (File)
Thu, Oct 31, 6:20 PM
Unknown Object (File)
Mon, Oct 28, 12:06 PM
Unknown Object (File)
Oct 27 2024, 6:05 PM
Unknown Object (File)
Oct 27 2024, 6:04 PM
Unknown Object (File)
Oct 27 2024, 6:04 PM

Details

Summary

Tracing provides structured logging and diagnostics info. It's especially helpful for asynchronous programs where log lines for various tasks can get intermixed.

Test Plan

Checked that detailed error logs appeared in my terminal

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun requested review of this revision.Apr 25 2022, 8:51 PM
services/identity/src/main.rs
19 ↗(On Diff #11903)

The string slice passed to expect shows up in the panic message, so we actually want to express that loading config failed here

services/identity/src/main.rs
12 ↗(On Diff #11903)

Can we just omit this?

tomek added a reviewer: ashoat. tomek added 1 blocking reviewer(s): karol.

Adding @ashoat as we're adding a new library

Acceptin' as this looks good and my comment is just a nit, but please respond, thanks!

This revision is now accepted and ready to land.Apr 27 2022, 11:58 PM
services/identity/src/main.rs
12 ↗(On Diff #11903)

yeah, changed it to tracing::subscriber::set_global_default(subscriber)?;

varun marked an inline comment as done.May 4 2022, 1:16 PM