Skip to content

Commit 79260da

Browse files
authored
Revert "[8.19] Standardize on docker image arch classifier (#130643) (#131754)"
This reverts commit 519122d.
1 parent 62e8503 commit 79260da

File tree

3 files changed

+5
-7
lines changed

3 files changed

+5
-7
lines changed

build-tools/src/main/java/org/elasticsearch/gradle/Architecture.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,17 +11,15 @@
1111

1212
public enum Architecture {
1313

14-
X64("x86_64", "linux/amd64", "amd64"),
15-
AARCH64("aarch64", "linux/arm64", "arm64");
14+
X64("x86_64", "linux/amd64"),
15+
AARCH64("aarch64", "linux/arm64");
1616

1717
public final String classifier;
1818
public final String dockerPlatform;
19-
public final String dockerClassifier;
2019

21-
Architecture(String classifier, String dockerPlatform, String dockerClassifier) {
20+
Architecture(String classifier, String dockerPlatform) {
2221
this.classifier = classifier;
2322
this.dockerPlatform = dockerPlatform;
24-
this.dockerClassifier = dockerClassifier;
2523
}
2624

2725
public static Architecture current() {

distribution/docker/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -639,7 +639,7 @@ subprojects { Project subProject ->
639639
it.setCompression(Compression.GZIP)
640640
it.getArchiveBaseName().set("elasticsearch${base.suffix}-${VersionProperties.elasticsearch}-docker-image")
641641
it.getArchiveVersion().set("")
642-
it.getArchiveClassifier().set(architecture.dockerClassifier)
642+
it.getArchiveClassifier().set(architecture == Architecture.AARCH64 ? 'aarch64' : '')
643643
it.getDestinationDirectory().set(new File(project.parent.buildDir, 'distributions'))
644644
it.dependsOn(exportTask)
645645
}

qa/packaging/src/test/java/org/elasticsearch/packaging/test/DockerTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ public void test022InstallPluginsFromLocalArchive() {
206206

207207
listPluginArchive().forEach(System.out::println);
208208
assertThat("Expected " + plugin + " to not be installed", listPlugins(), not(hasItems(plugin)));
209-
assertThat("Expected " + plugin + " available in archive", listPluginArchive(), hasItems(containsString(plugin)));
209+
assertThat("Expected " + plugin + " available in archive", listPluginArchive(), hasSize(16));
210210

211211
// Stuff the proxy settings with garbage, so any attempt to go out to the internet would fail
212212
sh.getEnv()

0 commit comments

Comments
 (0)