This repository has been archived by the owner on Oct 15, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 86
Shrink Docker image sizes via OpenJDK 8 JRE 'slim' base images #27
Labels
Comments
We could also switch to alpine. |
@farodin91 — good call! Looking at all available tags, here are our OpenJDK 8 Alpine-based base image candidates:
so we can use one of the versioned base images to create smaller images. |
farodin91
added a commit
to farodin91/janusgraph-docker
that referenced
this issue
Jan 20, 2021
Part of JanusGraph#60, JanusGraph#27 * Add security scanning using AquaSecurity Trivy * Security scan results can be found here: https://github.com/JanusGraph/janusgraph-docker/security/code-scanning * Use openjdk:8-jre-slim-buster instead of openjdk:8-jdk Signed-off-by: Jan Jansen <[email protected]>
farodin91
added a commit
to farodin91/janusgraph-docker
that referenced
this issue
Feb 2, 2021
Part of JanusGraph#60, JanusGraph#27 * Add security scanning using AquaSecurity Trivy * Security scan results can be found here: https://github.com/JanusGraph/janusgraph-docker/security/code-scanning * Use openjdk:8-jre-slim-buster instead of openjdk:8-jdk * Use debian:buster-slim as builder layer Signed-off-by: Jan Jansen <[email protected]>
farodin91
added a commit
that referenced
this issue
Feb 3, 2021
Part of #60, #27 * Add security scanning using AquaSecurity Trivy * Security scan results can be found here: https://github.com/JanusGraph/janusgraph-docker/security/code-scanning * Use openjdk:8-jre-slim-buster instead of openjdk:8-jdk * Use debian:buster-slim as builder layer Signed-off-by: Jan Jansen <[email protected]>
FlorianHockmann
added
enhancement
New feature or request
help wanted
Extra attention is needed
labels
Jun 16, 2022
lionelfleury
added a commit
to lionelfleury/janusgraph-docker
that referenced
this issue
Jun 17, 2022
Fixes JanusGraph#27 Signed-off-by: Lionel Fleury <[email protected]>
lionelfleury
added a commit
to lionelfleury/janusgraph-docker
that referenced
this issue
Jun 17, 2022
Fixes JanusGraph#27 Signed-off-by: Lionel Fleury <[email protected]>
lionelfleury
added a commit
to lionelfleury/janusgraph-docker
that referenced
this issue
Jun 17, 2022
Fixes JanusGraph#27 Signed-off-by: Lionel Fleury <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The currently-built JanusGraph Docker images are way too large. While recent releases are on the order of 260-270MB, the built containers are much larger.
Compare:
It may be because the current Docker images are built using the base image
openjdk:8-jdk
but we don't need the JDK for running JanusGraph, that's what we need to compile it. Thus, we can simply use the JRE instead of the JDK for running it, so we can start usingopenjdk:8-jre
.Further, we can get rid of even more unneeded support in the base image by switching to
openjdk:8-jre-slim
which should be sufficient for running JanusGraph.These image versions are listed on OpenJDK image files.
For details on these image versions, see this SO answer which breaks down the various types of OpenJDK Docker image base images.
The text was updated successfully, but these errors were encountered: