-
Notifications
You must be signed in to change notification settings - Fork 44
chore: update versions and add zstd compression support #142
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
base: master
Are you sure you want to change the base?
chore: update versions and add zstd compression support #142
Conversation
- Update STACKS_BLOCKCHAIN_VERSION to 3.3.0.0.3 - Update STACKS_SIGNER_VERSION to 3.3.0.0.3.0 - Update STACKS_BLOCKCHAIN_API_VERSION to 8.13.6 - Update POSTGRES_VERSION to 17 - Change chainstate archive format from tar.gz to tar.zst - Add zstd extraction support with availability check Co-Authored-By: Claude Opus 4.5 <[email protected]>
c1f7013 to
55c32a5
Compare
| if command -v zstd &> /dev/null; then | ||
| zstd -d -c "${CHAINDATA_DEST}" | tar -xvf - -C "${SCRIPTPATH}/persistent-data/${NETWORK}/stacks-blockchain/" || exit_error "${COLRED}Error${COLRESET} extracting stacks-blockchain chainstate data" | ||
| else | ||
| exit_error "${COLRED}Error${COLRESET} - zstd is required for extraction. Install with: brew install zstd (macOS) or apt install zstd (Ubuntu)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| exit_error "${COLRED}Error${COLRESET} - zstd is required for extraction. Install with: brew install zstd (macOS) or apt install zstd (Ubuntu)" | |
| exit_error "${COLRED}Error${COLRESET} - zstd is required for extraction. Install with: brew install zstd (macOS) or apt install zstd (Debian based systems)" |
| STACKS_BLOCKCHAIN_VERSION=3.3.0.0.1 | ||
| STACKS_SIGNER_VERSION=3.3.0.0.1.0 | ||
| STACKS_BLOCKCHAIN_API_VERSION=8.13.4 | ||
| STACKS_BLOCKCHAIN_VERSION=3.3.0.0.3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| STACKS_BLOCKCHAIN_VERSION=3.3.0.0.3 | |
| STACKS_BLOCKCHAIN_VERSION=3.3.0.0.4 |
| STACKS_SIGNER_VERSION=3.3.0.0.1.0 | ||
| STACKS_BLOCKCHAIN_API_VERSION=8.13.4 | ||
| STACKS_BLOCKCHAIN_VERSION=3.3.0.0.3 | ||
| STACKS_SIGNER_VERSION=3.3.0.0.3.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| STACKS_SIGNER_VERSION=3.3.0.0.3.0 | |
| STACKS_SIGNER_VERSION=3.3.0.0.4.0 |
wileyj
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few nits, but overall good to go.
Description
As a Stacks node operator, I would like to use the latest archive files from Hiro to seed my chainstate. The Hiro archive has updated to newer versions and changed the compression format from tar.gz to tar.zst for better compression ratios.
seed-chainstate.shwill now needzstdinstalled for extractionType of Change
Does this introduce a breaking change?
zstdinstalled for chainstate extractionzstdis not availableAre documentation updates required?
Testing information
zstdis installedsudo ./scripts/seed-chainstate.shscripts/seed-chainstate.sh,sample.envChecklist
npm run testpasses