Skip to content

build(docker): rocksdb needs libsnappy-dev and a git context inside the image - #6

Open
Oliver-CG wants to merge 1 commit into
Miosa-osa:mainfrom
Oliver-CG:upstream/dockerfile-build-fixes
Open

build(docker): rocksdb needs libsnappy-dev and a git context inside the image#6
Oliver-CG wants to merge 1 commit into
Miosa-osa:mainfrom
Oliver-CG:upstream/dockerfile-build-fixes

Conversation

@Oliver-CG

Copy link
Copy Markdown

What

Two independent failures before first boot with the shipped deploy/Dockerfile.engine, both measured on a clean docker compose build:

  1. the rebar3 pre-hook of :rocksdb shells out to git inside /app, but the COPY list (mix.exs, lib, config, priv) never includes .git — the build dies with fatal: not a git repository. An empty git init satisfies it.
  2. the rocksdb C++ build needs snappy headers; the builder stage installs build-essential/git/libsqlite3-dev but not libsnappy-dev (the README's brew instructions have snappy — the Dockerfile was missing the apt equivalent).

One-line each.

🤖 Generated with Claude Code

…he image

Two independent failures before first boot with the shipped Dockerfile.engine:
the rebar3 pre-hook of :rocksdb runs git in /app (the COPY list never includes
.git) and the C++ build requires snappy headers. Both measured on a clean
docker compose build.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@PAMF2

PAMF2 commented Aug 27, 2026

Copy link
Copy Markdown

Cross-reference: main's CI has been red since 2026-08-17 failing exactly where this PR aims - the rocksdb NIF build (run 17/08 21:40, C++ shadow warnings escalating in deps/rocksdb/c_src/cache.cc, and the docker lane missing libsnappy). This PR never ran CI because ci.yml is push-only; #8 adds the pull_request trigger so this and the other six PRs can validate themselves. Suggest merging #8 first, then re-pushing here to get a green/red verdict on the fix.

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