grobidOrg / grobid / 31736153029

44%

push

github

Sort the full-image runtime stage stable-first (#1545)

The runtime stage copied the built Java distribution in as its first step, above
the DeLFT install, the JEP build and the glove embeddings preload. That copy
changes with every grobid commit, so all three expensive-and-stable layers were
rebuilt on every build.

Move the grobid COPY to the bottom of the stage. The /data symlink moves up with
the preload, since resources-registry.json writes the lmdb to a path that
resolves through it. The JEP build bind-mounted the JDK from `builder`, tying it
to a stage that is rebuilt per commit; it now mounts from a one-line `jdk` stage
on the same pinned base image -- same content, no extra pull. The preload still
takes two files from the builder, but BuildKit keys COPY --from on their content,
so it rebuilds when the registry changes rather than per commit.

Also move the /data permission fixing into the RUN that fills it. A recursive
chmod in a later layer copies up every file it touches, which put a second copy
of the multi-GB embeddings database in the image.

The TensorFlow install already sat above the COPY and is left where it is.

The GitHub Actions cache is capped at 10GB per repository with LRU eviction, and
mode=max on this image exports well past that -- the torch/CUDA wheels are ~5GB
and the embeddings lmdb ~2GB. Entries were evicted between runs, so layers the
Dockerfile ordering makes cacheable were rebuilt anyway.

Push the cache to lfoppiano/grobid:buildcache, over the Docker Hub login the job
already does. ignore-error keeps a failed export from failing the build.

The build matrix ran `assemble test jacocoTestReport` on all four runners, but
only ubuntu-latest publishes coverage. Two kinds of waste followed.

jacocoTestReport writes per-subproject reports that nothing downstream reads --
coveralls is fed by the aggregate codeCoverageReport task, which depends on the
test tasks and derives from their execution data. Drop it from every leg.

... (continued)

10126 of 25690 branches covered (39.42%)

Branch coverage included in aggregate %.

21222 of 46365 relevant lines covered (45.77%)

1.86 hits per line