Skip to content

Replace log with tracing on Bridge Relay related modules #9430

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 49 commits into
base: master
Choose a base branch
from
Open

Replace log with tracing on Bridge Relay related modules #9430

wants to merge 49 commits into from
+726 −710

Conversation

@raymondkfcheung
Copy link
Contributor

@raymondkfcheung raymondkfcheung commented Aug 5, 2025 •
edited
Loading

This PR replaces log with tracing instrumentation on Bridge Relay related modules by providing structured logging.

Partially addresses #9211
Similar to #9279

Key Features

  • Consistent targets: All components use predictable log targets
  • Structured fields: Uses ?variable/%variable syntax for automatic Debug/Display formatting
  • Zero runtime impact: No behavioural changes, only observability improvements

@raymondkfcheung
Replace log with tracing on Bridge relay modules
b4b1d36
@raymondkfcheung
Update rpc.rs
c9af47a
@raymondkfcheung
Update subscription.rs
4a7e34e
@raymondkfcheung
Update guard.rs
0fa8b94
@raymondkfcheung
Update transaction_tracker.rs
2f42317
@raymondkfcheung
Update block_checker.rs
efddd93
@raymondkfcheung
Update equivocation_loop.rs
acfde04
@raymondkfcheung
Update finality_loop.rs
05850b6
@raymondkfcheung
Update finality_proofs.rs
4b2331a
@raymondkfcheung
Update headers.rs
1a10721
@raymondkfcheung
Fix tracing
e79af3f
@raymondkfcheung
Update relay_messages.rs
df9dcfa
@raymondkfcheung
Update mod.rs
98c0798
@raymondkfcheung
Update initialize.rs
bf1928e
@raymondkfcheung
Update target.rs
b7f4f37
@raymondkfcheung
Update engine.rs
b893ba5
@raymondkfcheung
Update mod.rs
1de97d3
@raymondkfcheung
Update metrics.rs
a941ecb
@raymondkfcheung
Update mod.rs
f38c18f
@raymondkfcheung
Update source.rs
54cdf9e
@raymondkfcheung
Update headers.rs
22c4459
@raymondkfcheung
Update parachains.rs
7d2a2f5
@raymondkfcheung
Update source.rs
1035141
@raymondkfcheung
Update target.rs
8bc1158
@raymondkfcheung
Update message_lane_loop.rs
a45c550
@raymondkfcheung
Update message_race_delivery.rs
4835f89
@raymondkfcheung
Update message_race_loop.rs
c60d5d4
@raymondkfcheung
Update message_race_receiving.rs
7dce315
@raymondkfcheung
Update message_race_strategy.rs
9d52588
@raymondkfcheung
Update parachains_loop.rs
2911457
@raymondkfcheung raymondkfcheung self-assigned this Aug 5, 2025
@raymondkfcheung raymondkfcheung added the T15-bridges This PR/Issue is related to bridges. label Aug 5, 2025
@paritytech-review-bot paritytech-review-bot bot requested a review from a team August 5, 2025 15:05
raymondkfcheung added a commit that referenced this pull request Aug 5, 2025
@raymondkfcheung
Revert Bridge Relay related modules #9430
ad707a8
@raymondkfcheung raymondkfcheung mentioned this pull request Aug 5, 2025
Open
23 tasks
@raymondkfcheung raymondkfcheung added the C2-good-first-issue A task for a first time contributor to become familiar with the Polkadot-SDK. label Aug 6, 2025
@paritytech-workflow-stopper
Copy link

paritytech-workflow-stopper bot commented Aug 7, 2025

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/16803223605
Failed job name: run-frame-omni-bencher

@acatangiu acatangiu enabled auto-merge October 21, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@acatangiu acatangiu acatangiu approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

@raymondkfcheung raymondkfcheung

Labels

C2-good-first-issue A task for a first time contributor to become familiar with the Polkadot-SDK. T15-bridges This PR/Issue is related to bridges.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@raymondkfcheung @acatangiu