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

Fix/instrumentation #200

Merged
merged 4 commits into from
Dec 27, 2024
Merged

Fix/instrumentation #200

merged 4 commits into from
Dec 27, 2024

Conversation

heemankv
Copy link
Contributor

@heemankv heemankv commented Dec 27, 2024

Record latest block number for StateTransition jobs

Currently, StateTransition job metrics record the starting block number instead of the ending block when processing multiple blocks. This PR changes it to record the ending block number (external_id) specifically for StateTransition jobs, while keeping the behavior the same for other job types.

@heemankv heemankv self-assigned this Dec 27, 2024
@coveralls
Copy link

coveralls commented Dec 27, 2024

Coverage Status

coverage: 67.433% (-0.02%) from 67.449%
when pulling 5e647f3 on fix/instrumentation
into a008642 on main.

@heemankv heemankv requested review from Mohiiit and ocdbytes December 27, 2024 12:48
@heemankv heemankv merged commit 15b321b into main Dec 27, 2024
8 checks passed
@heemankv heemankv deleted the fix/instrumentation branch December 27, 2024 13:28
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.

4 participants