aries-cloudagent-python icon indicating copy to clipboard operation
aries-cloudagent-python copied to clipboard

Log message processing time for better performance observability

Open jleach opened this issue 1 year ago • 4 comments

The BC Wallet often gets comments the wallet is "slow". It can be a daunting task to track down where the perceived slowness comes from: bc wallet, ledgers, mediator, issue agent, etc. We would like to introduce some basic logging to help track information flowing through the ecosystem during a debug session. In this case, our proposal is to add a log message when a message is received from a mediator client and when it is relayed to its destination. A sample is shown below:

17:50:42.634.  Message (some id or counter for the message) received from DID 12355abc.
... other stuff
... more stuff
17:50:43.121.  Message (some id or counter for the message) relayed to DID 1325d3d332.

The hope is we can correlate this with a message being sent to a connecitonId / threadId / DID on the wallet and better determine how long the message took to exit our infrastructure.

jleach avatar Jan 25 '24 22:01 jleach

This might possibly be covered under tracing

esune avatar Jan 25 '24 22:01 esune

Where did we get on per tenant logging? Is that complete? I think so... #2550 .

swcurran avatar Jan 29 '24 21:01 swcurran

Where did we get on per tenant logging? Is that complete? I think so... #2550 .

I need to pick up where we left and re-review what we have and what is missing. What @jleach is suggesting would likely require an audit of the code to include log statements in key points - if not already supported by tracing - to record entry/exit time in specific methods (with toggles to disable the statements during "normal" operation).

esune avatar Jan 30 '24 01:01 esune

This might possibly be covered under tracing

Does AFJ/Bifold support message tracing?

ianco avatar Feb 01 '24 17:02 ianco

I'm going to close this as stale. If anyone wants to re-open it leave a comment.

jamshale avatar Jul 08 '24 19:07 jamshale