Details
-
Enhancement
-
Resolution: Fixed
-
P3
-
17, 18
-
b11
Backports
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8279855 | 17.0.3-oracle | Harold Seigel | P3 | Resolved | Fixed | b02 |
JDK-8279828 | 17.0.3 | Matthias Baesken | P3 | Resolved | Fixed | b01 |
JDK-8325819 | 11.0.24-oracle | ANANT SINGH | P3 | Resolved | Fixed | b01 |
JDK-8298900 | 11.0.19-oracle | Calvin Cheung | P3 | Resolved | Fixed | b01 |
JDK-8288939 | 11.0.17 | Matthias Baesken | P3 | Resolved | Fixed | b01 |
JDK-8302447 | 8u381 | Ivan Bereziuk | P3 | Resolved | Fixed | b01 |
Description
The initial intent was to extend the buildJdkDockerImage() API of DockerTestUtils to accept custom Dockerfile content.
As I analyzed the usage of buildJdkDockerImage() I realized that:
- 2nd argument "dockerfile" is always the same: "Dockerfile-BasicTest"
its use has been obsolete for some time, in favor of Dockerfile generated by DockerTestUtils
- 3rd argument "buildDirName" is also always the same: "jdk-docker"
Hence I thought it would be a good idea to simplify this API and make it up-to-date.
Also, since the method signature is being updated, I thought it would be a good idea to also change the name to use more generic container terminology:
buildJdkDockerImage() --> buildJdkContainerImage()
Attachments
Issue Links
- backported by
-
JDK-8279828 Update DockerTestUtils.buildJdkDockerImage()
- Resolved
-
JDK-8279855 Update DockerTestUtils.buildJdkDockerImage()
- Resolved
-
JDK-8288939 Update DockerTestUtils.buildJdkDockerImage()
- Resolved
-
JDK-8298900 Update DockerTestUtils.buildJdkDockerImage()
- Resolved
-
JDK-8302447 Update DockerTestUtils.buildJdkDockerImage()
- Resolved
-
JDK-8325819 Update DockerTestUtils.buildJdkDockerImage()
- Resolved
- blocks
-
JDK-8195809 [TESTBUG] jps and jcmd -l support for containers is not tested
- Resolved
- links to
-
Commit openjdk/jdk11u-dev/c10e0de1
-
Commit openjdk/jdk17u-dev/ab0ade6f
-
Commit openjdk/jdk/ec63957f
-
Review openjdk/jdk11u-dev/1156
-
Review openjdk/jdk17u-dev/71
-
Review openjdk/jdk/5134