Skip to content

Commit ec1f2d1

Browse files
committed
Cleanup Ironbank only dockerfile
1 parent 4c0c9b6 commit ec1f2d1

File tree

6 files changed

+140
-279
lines changed

6 files changed

+140
-279
lines changed

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/DockerBase.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ public enum DockerBase {
1717
DEFAULT("redhat/ubi9-minimal:latest", "", "microdnf", "dockerfiles/default/Dockerfile"),
1818

1919
// The Iron Bank base image is UBI (albeit hardened), but we are required to parameterize the Docker build
20-
IRON_BANK("${BASE_REGISTRY}/${BASE_IMAGE}:${BASE_TAG}", "-ironbank", "yum", "Dockerfile"),
20+
IRON_BANK("${BASE_REGISTRY}/${BASE_IMAGE}:${BASE_TAG}", "-ironbank", "yum", "dockerfiles/iron_bank/Dockerfile"),
2121

2222
// Chainguard based wolfi image with latest jdk
2323
WOLFI(
@@ -28,7 +28,7 @@ public enum DockerBase {
2828
),
2929
// Based on WOLFI above, with more extras. We don't set a base image because
3030
// we programmatically extend from the wolfi image.
31-
CLOUD_ESS(null, "-cloud-ess", "apk", "Dockerfile.ess"),
31+
CLOUD_ESS(null, "-cloud-ess", "apk", "dockerfiles/cloud_ess/Dockerfile"),
3232

3333
CLOUD_ESS_FIPS(
3434
null,
@@ -42,10 +42,6 @@ public enum DockerBase {
4242
private final String packageManager;
4343
private final String dockerfile;
4444

45-
DockerBase(String image, String suffix) {
46-
this(image, suffix, "apt-get", "dockerfile");
47-
}
48-
4945
DockerBase(String image, String suffix, String packageManager, String dockerfile) {
5046
this.image = image;
5147
this.suffix = suffix;

distribution/docker/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ custom behaviour in the
134134

135135
We go to some lengths to try and make the Docker build resilient to transient
136136
network errors. This is why, when browsing the
137-
[Dockerfile](src/docker/Dockerfile), you'll see many commands wrapped in looping
137+
[Dockerfile](src/docker/dockerfiles/cloud_ess/Dockerfile), you'll see many commands wrapped in looping
138138
logic, so that if e.g. package installation fails, we try again. We also perform
139139
explicit `docker pull` commands instead of relying on `docker run` to pull an
140140
image down automatically, so that we can wrap the `pull` part in a retry.

distribution/docker/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -532,7 +532,7 @@ void addBuildCloudDockerImageTasks(Architecture architecture) {
532532
rename ~/((?:file|metric)beat)-.*\.tar\.gz$/, "\$1-${VersionProperties.elasticsearch}.tar.gz"
533533

534534
String baseSuffix = DockerBase.WOLFI.suffix
535-
from(projectDir.resolve("src/docker/Dockerfile.ess")) {
535+
from(projectDir.resolve("src/docker/Dockerfile")) {
536536
expand(
537537
[
538538
base_image : "elasticsearch${baseSuffix}:${architecture.classifier}",

distribution/docker/src/docker/Dockerfile

Lines changed: 0 additions & 271 deletions
This file was deleted.

0 commit comments

Comments
 (0)