feat(p2p): cover Exchange with traces

[vgonkivs]

Overview

Add traces for p2p.session

[codecov-commenter]

Codecov Report

Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (446648d) 63.96% compared to head (cd263e1) 62.95%.
Report is 3 commits behind head on main.

Files	Patch %	Lines
p2p/exchange.go	81.63%	9 Missing ⚠️
p2p/session.go	90.32%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
- Coverage   63.96%   62.95%   -1.02%     
==========================================
  Files          39       39              
  Lines        3383     3593     +210     
==========================================
+ Hits         2164     2262      +98     
- Misses       1052     1157     +105     
- Partials      167      174       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Wondertan]

Would be also nice to cover Head.

[renaynay]

Just a general comment, there seems to be duplication of traces in paths where metrics are already covering. Let's reduce the amount of traces to events that are of interest -- the hottest of hottest paths (specifically Head request and get range by height, etc.) and only tracing the absolute most interesting events.

[vgonkivs]

I don't get why this is needed here even on old version tbh

context is needed to create the baggage. And we will get a race if do not copy the variable.

[Wondertan]

🙏🏻

[renaynay]

Utack