Integrated: 8371503: RETAIN_IMAGE_AFTER_TEST do not work for some tests

SendaoYan syan at openjdk.org
Mon Dec 15 14:21:46 UTC 2025


On Sat, 8 Nov 2025 08:10:45 GMT, SendaoYan <syan at openjdk.org> wrote:

> Hi all,
> 
> The DockerTestUtils.RETAIN_IMAGE_AFTER_TEST variable which read the property from "jdk.test.docker.retain.image" do not work for some of the docker tests, such as jdk/internal/platform/docker/TestDockerCpuMetrics.java, only works for some of the docker tests, such as jdk/internal/platform/docker/TestPidsLimit.java.
> 
> This PR read the DockerTestUtils.RETAIN_IMAGE_AFTER_TEST inside function removeDockerImage instead of before all the fucntion removeDockerImage. This will make all the docker tests receive the property jdk.test.docker.retain.image.
> 
> Change has been verified locally on linux-x64 by run the all touched tests.

This pull request has now been integrated.

Changeset: 34f24131
Author:    SendaoYan <syan at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/34f241317ecd7473cfb6dcc2e6e5cf3a40299e2c
Stats:     32 lines in 11 files changed: 2 ins; 20 del; 10 mod

8371503: RETAIN_IMAGE_AFTER_TEST do not work for some tests

Reviewed-by: lmesnik, dholmes

-------------

PR: https://git.openjdk.org/jdk/pull/28208


More information about the serviceability-dev mailing list