Skip to content

fix: add zstd to Docker image and validate required binaries at startup#147

Merged
alecthomas merged 1 commit intomainfrom
aat/fix-zstd-required
Feb 24, 2026
Merged

fix: add zstd to Docker image and validate required binaries at startup#147
alecthomas merged 1 commit intomainfrom
aat/fix-zstd-required

Conversation

@alecthomas
Copy link
Copy Markdown
Collaborator

@alecthomas alecthomas commented Feb 24, 2026

Add zstd to the Alpine package list in the Dockerfile so snapshot operations work out of the box.

Validate that required binaries (git, tar, zstd) are present at startup with clear error messages, rather than failing at runtime.

✓ Built cachew:local
→ Starting cachew at http://localhost:8080 (log-level=info)
INF Constructing disk cache limit-mb=250000 evict-interval=1m0s root=./state/cache max-ttl=8h0m0s
INF Constructing tiered cache tiers=1
DBG Cache backend cache=disk:/app/state/cache
DBG Registered strategy handler strategy=apiv1 pattern="GET /api/v1/object/{key}"
DBG Registered strategy handler strategy=apiv1 pattern="HEAD /api/v1/object/{key}"
DBG Registered strategy handler strategy=apiv1 pattern="POST /api/v1/object/{key}"
DBG Registered strategy handler strategy=apiv1 pattern="DELETE /api/v1/object/{key}"
DBG Registered strategy handler strategy=apiv1 pattern="GET /api/v1/stats"
cachewd: error: load config: /app/cachew.hcl:24:1: zstd is required for snapshots (snapshot-interval > 0) but not found in PATH
error: Recipe `run` failed on line 47 with exit code 1

Add zstd to the Alpine package list in the Dockerfile so snapshot
operations work out of the box.

Validate that required binaries (git, tar, zstd) are present at startup
with clear error messages, rather than failing at runtime.
@alecthomas alecthomas requested a review from a team as a code owner February 24, 2026 02:23
@alecthomas alecthomas requested review from js-murph and removed request for a team February 24, 2026 02:23
@alecthomas alecthomas enabled auto-merge (squash) February 24, 2026 02:23
@alecthomas alecthomas requested a review from nssherpa February 24, 2026 02:23
@alecthomas alecthomas merged commit 4872acb into main Feb 24, 2026
5 checks passed
@alecthomas alecthomas deleted the aat/fix-zstd-required branch February 24, 2026 02:24
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