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

Creation tx backfill scripts #1811

Merged
merged 6 commits into from
Dec 31, 2024
Merged

Creation tx backfill scripts #1811

merged 6 commits into from
Dec 31, 2024

Conversation

kuzdogan
Copy link
Member

@kuzdogan kuzdogan commented Dec 27, 2024

The script I've written to do the backfill of the #1573. It's basically a reverification script but it first queries the DB to find contracts that don't have the creation info and creates a new DB table to keep track of this. Then fetches these contracts with their deployment and compilation info in batches from the db and sends them to verification concurrently. Implements a cold start to avoid drowning the server and getting downtimes.

Start reviewing at backfill-creation.mjs

node backfill-creation.mjs --help

I also think we should refactor out the logger as it is being used in a lot of places, but something for the future.

@kuzdogan
Copy link
Member Author

Blockscout tests are failing because they've changed the API response from creation_tx_hash to creation_transaction_hashhttps://eth.blockscout.com//api/v2/addresses/0x00000000219ab540356cBB839Cbe05303d7705Fa

We can ignore tests

Copy link
Collaborator

@manuelwedler manuelwedler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me. Just something about the comments and a question.

services/database/scripts/backfill-creation.mjs Outdated Show resolved Hide resolved
@kuzdogan kuzdogan merged commit ce0429c into staging Dec 31, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: COMPLETED
Development

Successfully merging this pull request may close these issues.

2 participants