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(relayer): restart indexer and crawler when they exit due to failed API requests #16465

Merged
merged 12 commits into from
Mar 18, 2024

Conversation

xiaodino
Copy link
Contributor

@xiaodino xiaodino commented Mar 16, 2024

Use https://relayer.katla.taiko.xyz/events?event=MessageSent&page=0&size=100&address=0x9e47FBb2a2A27B3b02E4a63b3eF5A3dC863c0223 as the example.

Before, the API response only returns 16 items. With the change, the API returns 19 items.

When a crawler is restarted, it will start from the last processing block height, rather than always from the genesis block.

@xiaodino xiaodino changed the title Xiaodino/alpha 6 fix crawler 1 @xiaodino fix(relayer): restart indexer when the indexer exits due to failed API requests Mar 16, 2024
@xiaodino xiaodino changed the title @xiaodino fix(relayer): restart indexer when the indexer exits due to failed API requests fix(relayer): restart indexer when the indexer exits due to failed API requests Mar 16, 2024
@xiaodino xiaodino marked this pull request as draft March 17, 2024 05:13
Copy link

codecov bot commented Mar 17, 2024

Codecov Report

Attention: Patch coverage is 21.42857% with 22 lines in your changes are missing coverage. Please review.

❗ No coverage uploaded for pull request base (alpha-6@f416ffb). Click here to learn what that means.

Files Patch % Lines
packages/relayer/indexer/indexer.go 20.83% 16 Missing and 3 partials ⚠️
...er/indexer/set_initial_processing_block_by_mode.go 25.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             alpha-6   #16465   +/-   ##
==========================================
  Coverage           ?   26.85%           
==========================================
  Files              ?      102           
  Lines              ?     6117           
  Branches           ?        0           
==========================================
  Hits               ?     1643           
  Misses             ?     4298           
  Partials           ?      176           
Flag Coverage Δ *Carryforward flag
eventindexer 17.28% <ø> (?) Carriedforward from c8da85b
guardian-prover-health-check 36.13% <ø> (?) Carriedforward from c8da85b
relayer 37.35% <21.42%> (?)

*This pull request uses carry forward flags. Click here to find out more.

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

@xiaodino xiaodino marked this pull request as ready for review March 18, 2024 07:17
@xiaodino xiaodino changed the title fix(relayer): restart indexer when the indexer exits due to failed API requests fix(relayer): restart indexer and crawler when they exit due to failed API requests Mar 18, 2024
@xiaodino xiaodino merged commit eb84395 into alpha-6 Mar 18, 2024
5 checks passed
@xiaodino xiaodino deleted the xiaodino/alpha-6-fix-crawler-1 branch March 18, 2024 07:41
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.

3 participants