Skip to content

Commit d5aa9bb

Browse files
authored
Use adoptopenjdk base image instead of openjdk in the Jib CLI (#3108)
* Use adoptopenjdk base image instead of openjdk * Update CHANGELOG
1 parent fece5c7 commit d5aa9bb

File tree

3 files changed

+9
-8
lines changed

3 files changed

+9
-8
lines changed

jib-cli/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ All notable changes to this project will be documented in this file.
66
### Added
77

88
### Changed
9+
- Changed the default base image of the Jib CLI jar command from the `openjdk` images to the `adoptopenjdk` images on Docker Hub. ([#3108](https://github.com/GoogleContainerTools/jib/pull/3108]))
910

1011
### Fixed
1112

jib-cli/src/main/java/com/google/cloud/tools/jib/cli/jar/JarFiles.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,8 @@ public static JibContainerBuilder toJibContainerBuilder(
5959
} else {
6060
containerBuilder =
6161
(processor.getJarJavaVersion() <= 8)
62-
? Jib.from("openjdk:8-jre-slim")
63-
: Jib.from("openjdk:11-jre-slim");
62+
? Jib.from("adoptopenjdk:8-jre")
63+
: Jib.from("adoptopenjdk:11-jre");
6464
}
6565

6666
List<FileEntriesLayer> layers = processor.createLayers();

jib-cli/src/test/java/com/google/cloud/tools/jib/cli/jar/JarFilesTest.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ public void testToJibContainer_defaultBaseImage_java8()
7171
mockStandardExplodedProcessor, mockJarCommand, mockCommonCliOptions, mockLogger);
7272
ContainerBuildPlan buildPlan = containerBuilder.toContainerBuildPlan();
7373

74-
assertThat(buildPlan.getBaseImage()).isEqualTo("openjdk:8-jre-slim");
74+
assertThat(buildPlan.getBaseImage()).isEqualTo("adoptopenjdk:8-jre");
7575
}
7676

7777
@Test
@@ -83,7 +83,7 @@ public void testToJibContainer_defaultBaseImage_java9()
8383
mockStandardExplodedProcessor, mockJarCommand, mockCommonCliOptions, mockLogger);
8484
ContainerBuildPlan buildPlan = containerBuilder.toContainerBuildPlan();
8585

86-
assertThat(buildPlan.getBaseImage()).isEqualTo("openjdk:11-jre-slim");
86+
assertThat(buildPlan.getBaseImage()).isEqualTo("adoptopenjdk:11-jre");
8787
}
8888

8989
@Test
@@ -108,7 +108,7 @@ public void testToJibContainerBuilder_explodedStandard_basicInfo()
108108
mockStandardExplodedProcessor, mockJarCommand, mockCommonCliOptions, mockLogger);
109109
ContainerBuildPlan buildPlan = containerBuilder.toContainerBuildPlan();
110110

111-
assertThat(buildPlan.getBaseImage()).isEqualTo("openjdk:8-jre-slim");
111+
assertThat(buildPlan.getBaseImage()).isEqualTo("adoptopenjdk:8-jre");
112112
assertThat(buildPlan.getPlatforms()).isEqualTo(ImmutableSet.of(new Platform("amd64", "linux")));
113113
assertThat(buildPlan.getCreationTime()).isEqualTo(Instant.EPOCH);
114114
assertThat(buildPlan.getFormat()).isEqualTo(ImageFormat.Docker);
@@ -153,7 +153,7 @@ public void testToJibContainerBuilder_packagedStandard_basicInfo()
153153
mockStandardPackagedProcessor, mockJarCommand, mockCommonCliOptions, mockLogger);
154154
ContainerBuildPlan buildPlan = containerBuilder.toContainerBuildPlan();
155155

156-
assertThat(buildPlan.getBaseImage()).isEqualTo("openjdk:8-jre-slim");
156+
assertThat(buildPlan.getBaseImage()).isEqualTo("adoptopenjdk:8-jre");
157157
assertThat(buildPlan.getPlatforms()).isEqualTo(ImmutableSet.of(new Platform("amd64", "linux")));
158158
assertThat(buildPlan.getCreationTime()).isEqualTo(Instant.EPOCH);
159159
assertThat(buildPlan.getFormat()).isEqualTo(ImageFormat.Docker);
@@ -199,7 +199,7 @@ public void testToJibContainerBuilder_explodedLayeredSpringBoot_basicInfo()
199199
mockSpringBootExplodedProcessor, mockJarCommand, mockCommonCliOptions, mockLogger);
200200
ContainerBuildPlan buildPlan = containerBuilder.toContainerBuildPlan();
201201

202-
assertThat(buildPlan.getBaseImage()).isEqualTo("openjdk:8-jre-slim");
202+
assertThat(buildPlan.getBaseImage()).isEqualTo("adoptopenjdk:8-jre");
203203
assertThat(buildPlan.getPlatforms()).isEqualTo(ImmutableSet.of(new Platform("amd64", "linux")));
204204
assertThat(buildPlan.getCreationTime()).isEqualTo(Instant.EPOCH);
205205
assertThat(buildPlan.getFormat()).isEqualTo(ImageFormat.Docker);
@@ -244,7 +244,7 @@ public void testToJibContainerBuilder_packagedSpringBoot_basicInfo()
244244
mockSpringBootPackagedProcessor, mockJarCommand, mockCommonCliOptions, mockLogger);
245245
ContainerBuildPlan buildPlan = containerBuilder.toContainerBuildPlan();
246246

247-
assertThat(buildPlan.getBaseImage()).isEqualTo("openjdk:8-jre-slim");
247+
assertThat(buildPlan.getBaseImage()).isEqualTo("adoptopenjdk:8-jre");
248248
assertThat(buildPlan.getPlatforms()).isEqualTo(ImmutableSet.of(new Platform("amd64", "linux")));
249249
assertThat(buildPlan.getCreationTime()).isEqualTo(Instant.EPOCH);
250250
assertThat(buildPlan.getFormat()).isEqualTo(ImageFormat.Docker);

0 commit comments

Comments
 (0)