Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Snapshots] Add snapshot support for full nodes #1092

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

okdas
Copy link
Member

@okdas okdas commented Feb 27, 2025

Summary

Add snapshot support to full node installer script

Issue

Type of change

Select one or more from the following:

Sanity Checklist

  • I have updated the GitHub Issue assignees, reviewers, labels, project, iteration and milestone
  • For docs, I have run make docusaurus_start
  • For code, I have run make go_develop_and_test and make test_e2e
  • For code, I have added the devnet-test-e2e label to run E2E tests in CI
  • For configurations, I have update the documentation
  • I added TODOs where applicable

@okdas okdas added infra Infra or tooling related improvements, additions or fixes tooling Tooling - CLI, scripts, helpers, off-chain, etc... labels Feb 27, 2025
@okdas okdas added this to the Beta TestNet Iteration milestone Feb 27, 2025
@okdas okdas self-assigned this Feb 27, 2025
@okdas okdas force-pushed the install-full-node-from-snapshot branch from 606f6e9 to e01eb31 Compare March 7, 2025 21:46
@okdas okdas changed the title [Snapshots] Add snapshot sync support for full node installation [Snapshots] Add snapshot support for full nodes Mar 8, 2025
@okdas
Copy link
Member Author

okdas commented Mar 8, 2025

Added #1111 and #1110 to iterate on snapshots post-mainnet.

@okdas okdas requested a review from Olshansk March 8, 2025 01:52
@okdas okdas marked this pull request as ready for review March 8, 2025 01:52
@okdas
Copy link
Member Author

okdas commented Mar 8, 2025

@Olshansk just tested this before hanging over the PR:

  • Walkthrough on Beta TestNet
  • The script that is used in the cheat sheet also worked - Beta TestNet + Snapshot = good.

I would like to add support for this script in helm-charts as well, hopefully after the load test.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
infra Infra or tooling related improvements, additions or fixes tooling Tooling - CLI, scripts, helpers, off-chain, etc...
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

2 participants