Skip to content

Commit

Permalink
[Misc] Fix testcase that depend on docker build
Browse files Browse the repository at this point in the history
Summary: A previous CRaC pactch change the signature of DockerTestUtils.buildJdkContainerImage(String imageName).All testcases depend on docker build will compile failed.

Testing: TestMemoryWithCgroupV1.java

Reviewers: yansendao.ysd,lvfei.lv

Issue: #872
  • Loading branch information
lingjun-cg authored and Accelerator1996 committed Oct 21, 2024
1 parent d01f01e commit 85a2ef0
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions test/lib/jdk/test/lib/containers/docker/DockerTestUtils.java
Original file line number Diff line number Diff line change
Expand Up @@ -122,6 +122,17 @@ private static boolean isDockerEngineAvailableCheck() throws Exception {
return true;
}

/**
* Build a container image that contains JDK under test.
* The jdk will be placed under the "/jdk/" folder inside the image/container file system.
*
* @param imageName name of the image to be created, including version tag
* @throws Exception
*/
public static void buildJdkContainerImage(String imageName) throws Exception {
buildJdkContainerImage(imageName, "jdk-docker");
}

/**
* Build a container image that contains JDK under test.
* The jdk will be placed under the "/jdk/" folder inside the image/container file system.
Expand Down

0 comments on commit 85a2ef0

Please sign in to comment.