Modernizes the OpenIDM Docker images and broadens their multi-architecture build matrix#193
Open
vharseko wants to merge 2 commits into
Open
Modernizes the OpenIDM Docker images and broadens their multi-architecture build matrix#193vharseko wants to merge 2 commits into
vharseko wants to merge 2 commits into
Conversation
- Dockerfile: bump base image eclipse-temurin:25-jre-jammy -> 25-jre-noble (Ubuntu 24.04) - Dockerfile-alpine: bump JDK openjdk17 -> openjdk25-jre to align the Alpine runtime with the Ubuntu JRE 25 image and enable linux/riscv64 - build.yml / release.yml: add linux/riscv64 to both the Ubuntu and Alpine docker build platforms - release.yml: add the CDDL license header linux/386 is intentionally not added: OpenIDM is compiled for Java 17+ (maven.compiler.target=17) and Alpine ships no Java 17+ build for 32-bit x86 (only openjdk11), so a 386 image cannot run OpenIDM.
Add "docker logs test" as the final command of both Docker test steps (build-docker and build-docker-alpine) in build.yml, so the container output is captured in CI for easier diagnosis of failed health checks.
maximthomas
approved these changes
Jun 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Modernizes the OpenIDM Docker images and broadens their multi-architecture build matrix, and improves CI diagnostics for the Docker smoke tests. Ported from OpenIdentityPlatform/OpenDJ#655, adapted to OpenIDM's Java 17+ requirement.
Dockerfile): switch the base image fromeclipse-temurin:25-jre-jammytoeclipse-temurin:25-jre-noble(Ubuntu 22.04 → 24.04).Dockerfile-alpine): bump the JDK fromopenjdk17toopenjdk25-jre, aligning the Alpine runtime with the Ubuntu JRE 25 image and enablinglinux/riscv64.build.yml,release.yml): addlinux/riscv64to both the Ubuntu and Alpine docker build platforms.build.yml): printdocker logs testat the end of bothDocker teststeps so container output is captured on failed health checks.release.yml): add the CDDL license header.Platform coverage
Dockerfile)linux/amd64,linux/arm64/8,linux/ppc64le,linux/s390x,linux/riscv64Dockerfile-alpine)linux/amd64,linux/arm64/8,linux/s390x,linux/ppc64le,linux/riscv64Why
linux/386is not includedUnlike OpenDJ (which supports Java 11–25), OpenIDM is compiled for Java 17+ (
maven.compiler.target=17in the rootpom.xml). Alpine ships no Java 17+ build for 32-bit x86 (linux/386) — onlyopenjdk11— so alinux/386image could not run OpenIDM. It is therefore intentionally excluded, and no conditional/per-architecture JDK selection is needed: a singleopenjdk25-jrecovers every target architecture.Notes
linux/arm/v6andlinux/arm/v7remain excluded: neither Alpine noreclipse-temurinpublishes an OpenJDK for those architectures.continue-on-error: true, so an architecture without a published base/JDK won't fail the whole job.Testing
build-docker(Ubuntu/Noble) andbuild-docker-alpinejobs build across the listed platforms, including the existing "Docker test" smoke tests (now with container logs printed).