Problem#
Storage paths are hardcoded to ./tmp/blobs and ./tmp/manifests throughout the codebase. This appears in:
src/health.rs:82-83- Health check pathssrc/health.rs:114- Storage accessibility checksrc/health.rs:119- Writable checksrc/gc.rs:80-81- GC manifest scanningsrc/gc.rs:158- GC blob scanningsrc/gc.rs:233- GC blob deletion pathssrc/storage.rs(likely) - Actual blob/manifest storage
The args.rs CLI parser may already have a --storage or --data-dir flag, but the hardcoded paths bypass it.
Proposed Solution#
- Add a
--data-dirCLI argument (or use the existing one if present) - Pass the configured path through
Appstate - Replace all hardcoded
./tmp/references with the configured path - Default to
./dataor./tmpfor backwards compatibility