Skip to content

feat: environment-driven snapshot restore with targetHeight selection#45

Merged
bdchatham merged 2 commits intomainfrom
feat/env-driven-snapshot-restore
Apr 1, 2026
Merged

feat: environment-driven snapshot restore with targetHeight selection#45
bdchatham merged 2 commits intomainfrom
feat/env-driven-snapshot-restore

Conversation

@bdchatham
Copy link
Copy Markdown
Collaborator

Summary

  • SnapshotRestoreParams simplified to just TargetHeight int64
  • ForNode() no longer takes platform.Config — snapshot S3 coordinates are sidecar-side concerns
  • Inject SEI_SNAPSHOT_BUCKET/SEI_SNAPSHOT_REGION into sidecar containers (StatefulSet + bootstrap Job)
  • Planner no longer threads snapshotBucket/snapshotRegion through the call chain

Companion PR

Test plan

  • make lint — 0 issues
  • go test ./... — all pass
  • Snapshot restore params round-trip verified

🤖 Generated with Claude Code

Refactor snapshot restore to match the genesis convention:
- S3 bucket/region/chainID derived from sidecar environment
- Controller sends only targetHeight as task param
- SnapshotRestoreParams simplified to just TargetHeight
- ForNode() no longer takes platform.Config (snapshot coords no longer
  needed by planners)
- Inject SEI_SNAPSHOT_BUCKET/SEI_SNAPSHOT_REGION into both StatefulSet
  and bootstrap Job sidecar containers

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@bdchatham bdchatham merged commit 65fd1f5 into main Apr 1, 2026
2 checks passed
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