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

Add mainnet directory setup instructions. #2288

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

saniksin
Copy link

@saniksin saniksin commented Nov 28, 2024

Description

Problem: The original instructions did not specify creating a dedicated directory for extracting snapshots. As a result, following the instructions led to Docker pointing to an incorrect path, causing synchronization to start from the first block instead of using the snapshot data.

Solution: Added commands to create a dedicated juno_mainnet directory and updated extraction instructions. This ensures the snapshot data is properly used during sync, avoiding unnecessary full blockchain synchronization from the start.

Changes Made:

  1. Changes to directory creation command:

    - mkdir -p $HOME/snapshots
    + mkdir -p $HOME/snapshots/juno_mainnet
  2. Changes to snapshot extraction command:

    - tar -xvf juno_mainnet.tar -C $HOME/snapshots
    + tar -xvf juno_mainnet.tar -C $HOME/snapshots/juno_mainnet
  3. Example Docker run command after these changes:

docker run -d \
  --name juno \
  -p 6060:6060 \
  
  <!-- The problem is in the line: -->
  -v $HOME/snapshots/juno_mainnet:/snapshots/juno_mainnet \
  
  nethermind/juno \
  --http \
  --http-port 6060 \
  --http-host 0.0.0.0 \
  --db-path /snapshots/juno_mainnet \
  --eth-node <YOUR ETH NODE>

These changes ensure that the Docker container will use the correct path, utilizing the snapshot effectively instead of resynchronizing from the genesis block.

Copy link

codecov bot commented Jan 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.37%. Comparing base (71c7ae9) to head (7b86c92).
Report is 21 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2288   +/-   ##
=======================================
  Coverage   74.37%   74.37%           
=======================================
  Files         109      109           
  Lines       11519    11519           
=======================================
  Hits         8567     8567           
  Misses       2285     2285           
  Partials      667      667           

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

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.

1 participant