Skip to content

Normalize chains/ directory for Substrate and tar snapshots#22

Closed
vukomir wants to merge 2 commits intomainfrom
feat/substrate-chains-dir-snapshot-tar
Closed

Normalize chains/ directory for Substrate and tar snapshots#22
vukomir wants to merge 2 commits intomainfrom
feat/substrate-chains-dir-snapshot-tar

Conversation

@vukomir
Copy link
Copy Markdown
Contributor

@vukomir vukomir commented Mar 31, 2026

  • Add SubstrateChainsDirName (hyphens to underscores) for ChainDBDataPath and RelayChainDBDataPath so on-disk layout matches the node.
  • Extend tar extraction to strip chains//db|paritydb/ prefixes using both YAML chain_id and normalized segment; pass chain_id from SyncIfNeeded.
  • Add tests for path mapping and snapshot sync with underscore archive paths.

- Add SubstrateChainsDirName (hyphens to underscores) for ChainDBDataPath and RelayChainDBDataPath so on-disk layout matches the node.
- Extend tar extraction to strip chains/<id>/db|paritydb/ prefixes using both YAML chain_id and normalized segment; pass chain_id from SyncIfNeeded.
- Add tests for path mapping and snapshot sync with underscore archive paths.

Made-with: Cursor
Copilot AI review requested due to automatic review settings March 31, 2026 10:06
@vukomir vukomir closed this Mar 31, 2026
@vukomir vukomir deleted the feat/substrate-chains-dir-snapshot-tar branch March 31, 2026 10:07
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants