Skip to content
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.

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

feat: generate traces for solana elections #5639

Merged
merged 8 commits into from
Feb 18, 2025

Conversation

MxmUrw
Copy link
Contributor

@MxmUrw MxmUrw commented Feb 14, 2025

Pull Request

Closes: PRO-2015

Checklist

Please conduct a thorough self-review before opening the PR.

  • I am confident that the code works.
  • I have written sufficient tests.
  • I have written and tested required migrations.
  • I have updated documentation where appropriate.

Summary

This PR introduces a new binary chainflip-elections-tracker which runs alongside a chainflip node and watches the state of elections in each block. It computes which part of the state have changed and pushes this information in form of OTLP traces to a compatible backend, for example to be ingested by Grafana Tempo.

This is a first version with basic tracking of elections. For example, it does not "backfill" blocks if it was shut down for a while. Further functionality is going to be added on a by-need basis.

@MxmUrw MxmUrw force-pushed the feat/PRO-2015/generate-traces-for-solana-elections branch 2 times, most recently from 612c5cc to 00b3e2e Compare February 14, 2025 10:11
@MxmUrw MxmUrw changed the title Feat/pro 2015/generate traces for solana elections feat: generate traces for solana elections Feb 14, 2025
Copy link

codecov bot commented Feb 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@MxmUrw MxmUrw force-pushed the feat/PRO-2015/generate-traces-for-solana-elections branch 5 times, most recently from c6022f6 to aa9b28c Compare February 14, 2025 13:16
@MxmUrw MxmUrw force-pushed the feat/PRO-2015/generate-traces-for-solana-elections branch from aa9b28c to cfc2a21 Compare February 14, 2025 13:20
@MxmUrw MxmUrw marked this pull request as ready for review February 14, 2025 13:24
@MxmUrw MxmUrw requested a review from dandanlen as a code owner February 14, 2025 13:24
@MxmUrw MxmUrw force-pushed the feat/PRO-2015/generate-traces-for-solana-elections branch from 11fb67a to 47d1e5f Compare February 14, 2025 14:02
@MxmUrw MxmUrw requested review from kylezs and removed request for dandanlen February 14, 2025 14:11
@MxmUrw MxmUrw requested a review from kylezs February 18, 2025 12:21
@MxmUrw MxmUrw added this pull request to the merge queue Feb 18, 2025
Merged via the queue into main with commit 7f2ad67 Feb 18, 2025
48 checks passed
@MxmUrw MxmUrw deleted the feat/PRO-2015/generate-traces-for-solana-elections branch February 18, 2025 15:48
@marcellorigotti marcellorigotti self-requested a review February 18, 2025 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants