Skip to content

chore(cmd): utils to support slot hash debug tooling#1203

Merged
yewman merged 8 commits intomainfrom
harnew/cmd-utils
Feb 13, 2026
Merged

chore(cmd): utils to support slot hash debug tooling#1203
yewman merged 8 commits intomainfrom
harnew/cmd-utils

Conversation

@yewman
Copy link
Contributor

@yewman yewman commented Feb 3, 2026

Summary

  • Add option to specify validator directory
  • Support genesis downloads instead of relying on local copies
  • Add ledger tool for checking ledger bounds and trimming to range
  • Add flag to log the highest completed slot in the shred tracker

@yewman yewman requested review from dnut and ultd as code owners February 3, 2026 08:02
@github-project-automation github-project-automation bot moved this to 🏗 In progress in Sig Feb 3, 2026
@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
src/accountsdb/snapshot/data.zig 89.49% <100.00%> (ø)
src/accountsdb/snapshot/load.zig 90.30% <100.00%> (ø)
src/core/hash.zig 97.76% <ø> (ø)
src/rpc/methods.zig 79.66% <ø> (ø)
src/shred_network/collector/repair_service.zig 83.63% <100.00%> (ø)
src/shred_network/collector/shred_receiver.zig 83.42% <100.00%> (ø)
src/shred_network/collector/shred_tracker.zig 86.68% <100.00%> (+0.09%) ⬆️
src/shred_network/service.zig 90.38% <100.00%> (+0.48%) ⬆️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-project-automation github-project-automation bot moved this from 🏗 In progress to 👀 In review in Sig Feb 3, 2026
@yewman yewman force-pushed the harnew/cmd-utils branch 3 times, most recently from ce12c1e to da20824 Compare February 5, 2026 19:12
@yewman yewman force-pushed the harnew/epoch-state-tracking branch 2 times, most recently from 6bb797e to e250f1f Compare February 9, 2026 05:11
@yewman yewman force-pushed the harnew/cmd-utils branch 2 times, most recently from 2f174ea to 8825886 Compare February 10, 2026 02:33
@yewman yewman force-pushed the harnew/cmd-utils branch 3 times, most recently from d80ce78 to ae30019 Compare February 10, 2026 23:17
Rexicon226
Rexicon226 previously approved these changes Feb 10, 2026
Base automatically changed from harnew/epoch-state-tracking to main February 11, 2026 05:17
@yewman yewman dismissed Rexicon226’s stale review February 11, 2026 05:17

The base branch was changed.

yewman and others added 7 commits February 12, 2026 18:01
- retain snapshot-dir and geyser-dir which override defaults within specified validator dir
- create the validator dir if it does not exist
- support sym links for snapshots
…atabase

- Add 'bounds' subcommand to print min/max slot in ledger
- Add 'retain' subcommand to purge slots outside specified range
- Add flag to log highest finished slot when it updates in setBottom
- Wire flag through cmd.zig -> config.zig -> service.zig -> shred_tracker
- Log directly in existing code path, no separate thread
- Add derivePathFromValidatorDir to config.Cmd for DRY path resolution
- Replace 7 snapshot_dir and 3 geyser.pipe_path inline derivations
- Auto-detect existing genesis.bin in validator_dir before downloading
- Minor formatting improvements for multiline help strings
- Replace @cImport with bzip2 mod via translate-c
- Simplify genesis_download to use new bzip2 module import
- Fix path handling in cmd.zig and config.zig for validator-dir
- allow ensureValidatorDir to create directory if it does not exist
- move ensureValidatorDir dir to outside sub command
- Use openFile instead of access for genesis file existence check
- Replace allocPrint with bufPrint for genesis URL to avoid allocation
- Use makeOpenPath to simplify directory creation logic
- Remove unnecessary min cap on decompression buffer size
- Add TODO for future path allocation improvements
@yewman yewman enabled auto-merge February 13, 2026 17:38
@yewman yewman added this pull request to the merge queue Feb 13, 2026
Merged via the queue into main with commit 7ca50bc Feb 13, 2026
20 checks passed
@yewman yewman deleted the harnew/cmd-utils branch February 13, 2026 18:23
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Sig Feb 13, 2026
ajw221 pushed a commit that referenced this pull request Feb 13, 2026
# Summary
- Add option to specify validator directory 
- Support genesis downloads instead of relying on local copies
- Add ledger tool for checking ledger bounds and trimming to range
- Add flag to log the highest completed slot in the shred tracker

---------

Co-authored-by: Ubuntu <ubuntu@ip-172-31-33-159.ap-southeast-2.compute.internal>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

4 participants