Skip to content

feat: improvements to sync snapshot script#53

Draft
kangsorang wants to merge 1 commit intomainfrom
feat/update-sync-script
Draft

feat: improvements to sync snapshot script#53
kangsorang wants to merge 1 commit intomainfrom
feat/update-sync-script

Conversation

@kangsorang
Copy link
Copy Markdown
Contributor

Description

Speed and stability improvements to snapshot script

  • improved the speed of make snapshot, sync sync using aria2c, pigz libraries.
  • increased reliability by using docker inspect for result lookup for docker execution requests.

Comment thread sync_block.sh Outdated
Comment thread sync_block.sh Outdated
@kangsorang kangsorang force-pushed the feat/update-sync-script branch 2 times, most recently from f040112 to 7c24052 Compare January 6, 2025 07:21
Comment thread sync_block.sh Outdated
@seolaoh
Copy link
Copy Markdown
Contributor

seolaoh commented Jan 6, 2025

I hope you to test this script when Sepolia hardfork completed. Thanks!

- improved the speed of make snapshot, sync sync using aria2c, pigz libraries.
- increased reliability by using docker inspect for result lookup for docker execution requests.
@kangsorang kangsorang force-pushed the feat/update-sync-script branch from 7c24052 to 1d50dc7 Compare January 6, 2025 08:55
@kangsorang
Copy link
Copy Markdown
Contributor Author

I hope you to test this script when Sepolia hardfork completed. Thanks!

Tested with holesky, will check after sepolia hardfork.

@kangsorang
Copy link
Copy Markdown
Contributor Author

kangsorang commented Jan 10, 2025

Here's what I found out later.
There is a difference in the structure of docker usage in devops-kroma-up and kroma-up.

  • devops-kroma-up: mount to host directory and use it
  • kroma-up : uses docker volume

So we can't use the same scheme as devops-kroma-up (dealing directly with the .kroma directory on host).
I'll take this into consideration. Do not merge before then.

@kangsorang kangsorang marked this pull request as draft February 3, 2025 00:56
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