Skip to content

Commit 0e9f74e

Browse files
saratvemulapallisaratvemulapalli
authoredApr 26, 2021
Renaming CPU architecture to have consistent naming (opensearch-project#612)
* Renaming CPU architecture from x86_64, aarch64 to x64, arm64 Signed-off-by: Sarat Vemulapalli <vemulapallisarat@gmail.com>
·
2.2.01.0.0-alpha2
1 parent c5a3c3c commit 0e9f74e

File tree

24 files changed

+102
-88
lines changed

24 files changed

+102
-88
lines changed
 

‎buildSrc/build.gradle

+1-1
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ if (project != rootProject) {
185185
distribution project(':distribution:archives:windows-zip')
186186
distribution project(':distribution:archives:darwin-tar')
187187
distribution project(':distribution:archives:linux-tar')
188-
distribution project(':distribution:archives:linux-aarch64-tar')
188+
distribution project(':distribution:archives:linux-arm64-tar')
189189

190190
integTestRuntimeOnly(project(":libs:opensearch-core"))
191191
}

‎buildSrc/src/integTest/groovy/org/opensearch/gradle/DistributionDownloadPluginFuncTest.groovy

+1-1
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ class DistributionDownloadPluginFuncTest extends AbstractGradleFuncTest {
119119

120120
then:
121121
result.tasks.size() == 3
122-
result.output.count("Unpacking opensearch-${version}-linux-x86_64.tar.gz " +
122+
result.output.count("Unpacking opensearch-${version}-linux-x64.tar.gz " +
123123
"using SymbolicLinkPreservingUntarTransform.") == 1
124124
}
125125

‎buildSrc/src/integTest/groovy/org/opensearch/gradle/fixtures/DistributionDownloadFixture.groovy

+1-1
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ class DistributionDownloadFixture {
6565
private static String urlPath(String version, OpenSearchDistribution.Platform platform) {
6666
String fileType = ((platform == OpenSearchDistribution.Platform.LINUX ||
6767
platform == OpenSearchDistribution.Platform.DARWIN)) ? "tar.gz" : "zip"
68-
"/releases/core/opensearch/${version}/opensearch-${version}-${platform}-x86_64.$fileType"
68+
"/releases/core/opensearch/${version}/opensearch-${version}-${platform}-x64.$fileType"
6969
}
7070

7171
private static byte[] filebytes(String urlPath) throws IOException {

‎buildSrc/src/integTest/groovy/org/opensearch/gradle/internal/InternalDistributionBwcSetupPluginFuncTest.groovy

+1-1
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ class InternalDistributionBwcSetupPluginFuncTest extends AbstractGradleFuncTest
102102
result.output.contains("[8.0.1] > Task :distribution:archives:darwin-tar:assemble")
103103
normalizedOutput(result.output)
104104
.contains("distfile /distribution/bwc/bugfix/build/bwc/checkout-8.0/distribution/archives/darwin-tar/" +
105-
"build/distributions/opensearch-8.0.1-SNAPSHOT-darwin-x86_64.tar.gz")
105+
"build/distributions/opensearch-8.0.1-SNAPSHOT-darwin-x64.tar.gz")
106106
}
107107

108108
def "bwc expanded distribution folder can be resolved as bwc project artifact"() {

‎buildSrc/src/integTest/resources/org/opensearch/gradle/internal/fake_git/remote/distribution/archives/build.gradle

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ subprojects {
3636
destinationDirectory.set(file('build/distributions'))
3737
archiveBaseName.set("opensearch")
3838
archiveVersion.set("8.0.1-SNAPSHOT")
39-
archiveClassifier.set("darwin-x86_64")
39+
archiveClassifier.set("darwin-x64")
4040
archiveExtension.set('tar.gz')
4141
}
4242

‎buildSrc/src/main/groovy/org/opensearch/gradle/test/ClusterFormationTasks.groovy

+1-1
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ class ClusterFormationTasks {
206206

207207
Version version = Version.fromString(opensearchVersion)
208208
String os = getOs()
209-
String classifier = "-${os}-x86_64"
209+
String classifier = "-${os}-x64"
210210
String packaging = os.equals('windows') ? 'zip' : 'tar.gz'
211211
String artifactName = 'opensearch'
212212
Object dependency

‎buildSrc/src/main/java/org/opensearch/gradle/Architecture.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
public enum Architecture {
3636

3737
X64,
38-
AARCH64;
38+
ARM64;
3939

4040
public static Architecture current() {
4141
final String architecture = System.getProperty("os.arch", "");
@@ -44,7 +44,7 @@ public static Architecture current() {
4444
case "x86_64":
4545
return X64;
4646
case "aarch64":
47-
return AARCH64;
47+
return ARM64;
4848
default:
4949
throw new IllegalArgumentException("can not determine architecture from [" + architecture + "]");
5050
}

‎buildSrc/src/main/java/org/opensearch/gradle/DistributionDownloadPlugin.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -187,11 +187,11 @@ private String dependencyNotation(OpenSearchDistribution distribution) {
187187

188188
Version distroVersion = Version.fromString(distribution.getVersion());
189189
String extension = distribution.getType().toString();
190-
String classifier = ":x86_64";
190+
String classifier = ":x64";
191191
if (distribution.getType() == Type.ARCHIVE) {
192192
extension = distribution.getPlatform() == Platform.WINDOWS ? "zip" : "tar.gz";
193193
if (distroVersion.onOrAfter("7.0.0")) {
194-
classifier = ":" + distribution.getPlatform() + "-x86_64";
194+
classifier = ":" + distribution.getPlatform() + "-x64";
195195
} else {
196196
classifier = "";
197197
}

‎buildSrc/src/main/java/org/opensearch/gradle/Jdk.java

+10-2
Original file line numberDiff line numberDiff line change
@@ -120,9 +120,10 @@ public String getArchitecture() {
120120
}
121121

122122
public void setArchitecture(final String architecture) {
123-
if (ALLOWED_ARCHITECTURES.contains(architecture) == false) {
123+
String jdkArchitecture = translateJdkArchitecture(architecture);
124+
if (ALLOWED_ARCHITECTURES.contains(jdkArchitecture) == false) {
124125
throw new IllegalArgumentException(
125-
"unknown architecture [" + architecture + "] for jdk [" + name + "], must be one of " + ALLOWED_ARCHITECTURES
126+
"unknown architecture [" + jdkArchitecture + "] for jdk [" + name + "], must be one of " + ALLOWED_ARCHITECTURES
126127
);
127128
}
128129
this.architecture.set(architecture);
@@ -229,4 +230,11 @@ private void parseVersion(String version) {
229230
hash = jdkVersionMatcher.group(5);
230231
}
231232

233+
private String translateJdkArchitecture(String architecture) {
234+
/*
235+
* Jdk uses aarch64 from ARM. Translating from arm64 to aarch64 which Jdk understands.
236+
*/
237+
return architecture == "arm64" ? "aarch64" : architecture;
238+
}
239+
232240
}

‎buildSrc/src/main/java/org/opensearch/gradle/internal/InternalDistributionBwcSetupPlugin.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ private void registerDistributionArchiveArtifact(Project bwcProject, Distributio
135135
String artifactName = "opensearch";
136136

137137
String suffix = artifactFileName.endsWith("tar.gz") ? "tar.gz" : artifactFileName.substring(artifactFileName.length() - 3);
138-
int archIndex = artifactFileName.indexOf("x86_64");
138+
int archIndex = artifactFileName.indexOf("x64");
139139

140140
bwcProject.getConfigurations().create(distributionProject.name);
141141
bwcProject.getArtifacts().add(distributionProject.name, distributionProject.getDistFile(), artifact -> {
@@ -146,7 +146,7 @@ private void registerDistributionArchiveArtifact(Project bwcProject, Distributio
146146
String classifier = "";
147147
if (archIndex != -1) {
148148
int osIndex = artifactFileName.lastIndexOf('-', archIndex - 2);
149-
classifier = "-" + artifactFileName.substring(osIndex + 1, archIndex - 1) + "-x86_64";
149+
classifier = "-" + artifactFileName.substring(osIndex + 1, archIndex - 1) + "-x64";
150150
}
151151
artifact.setClassifier(classifier);
152152
});
@@ -171,15 +171,15 @@ private static List<DistributionProject> resolveArchiveProjects(File checkoutDir
171171
if (name.contains("zip") || name.contains("tar")) {
172172
int index = name.lastIndexOf('-');
173173
String baseName = name.substring(0, index);
174-
classifier = "-" + baseName + "-x86_64";
174+
classifier = "-" + baseName + "-x64";
175175
extension = name.substring(index + 1);
176176
if (extension.equals("tar")) {
177177
extension += ".gz";
178178
}
179179
} else if (name.contains("deb")) {
180180
classifier = "-amd64";
181181
} else if (name.contains("rpm")) {
182-
classifier = "-x86_64";
182+
classifier = "-x64";
183183
}
184184
} else {
185185
extension = name.substring(4);

‎buildSrc/src/main/java/org/opensearch/gradle/test/DistroTestPlugin.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -381,8 +381,8 @@ private List<OpenSearchDistribution> configureDistributions(Project project) {
381381
)) {
382382
for (boolean bundledJdk : Arrays.asList(true, false)) {
383383
if (bundledJdk == false) {
384-
// We'll never publish an ARM (aarch64) build without a bundled JDK.
385-
if (architecture == Architecture.AARCH64) {
384+
// We'll never publish an ARM (arm64) build without a bundled JDK.
385+
if (architecture == Architecture.ARM64) {
386386
continue;
387387
}
388388
// All our Docker images include a bundled JDK so it doesn't make sense to test without one.

‎distribution/archives/build.gradle

+15-15
Original file line numberDiff line numberDiff line change
@@ -89,51 +89,51 @@ distribution_archives {
8989
}
9090

9191
windowsZip {
92-
archiveClassifier = 'windows-x86_64'
92+
archiveClassifier = 'windows-x64'
9393
content {
94-
archiveFiles(modulesFiles('windows-x86_64'), 'zip', 'windows', 'x64', true)
94+
archiveFiles(modulesFiles('windows-x64'), 'zip', 'windows', 'x64', true)
9595
}
9696
}
9797

9898
noJdkWindowsZip {
99-
archiveClassifier = 'no-jdk-windows-x86_64'
99+
archiveClassifier = 'no-jdk-windows-x64'
100100
content {
101-
archiveFiles(modulesFiles('windows-x86_64'), 'zip', 'windows', 'x64', false)
101+
archiveFiles(modulesFiles('windows-x64'), 'zip', 'windows', 'x64', false)
102102
}
103103
}
104104

105105
darwinTar {
106-
archiveClassifier = 'darwin-x86_64'
106+
archiveClassifier = 'darwin-x64'
107107
content {
108-
archiveFiles(modulesFiles('darwin-x86_64'), 'tar', 'darwin', 'x64', true)
108+
archiveFiles(modulesFiles('darwin-x64'), 'tar', 'darwin', 'x64', true)
109109
}
110110
}
111111

112112
noJdkDarwinTar {
113-
archiveClassifier = 'no-jdk-darwin-x86_64'
113+
archiveClassifier = 'no-jdk-darwin-x64'
114114
content {
115-
archiveFiles(modulesFiles('darwin-x86_64'), 'tar', 'darwin', 'x64', false)
115+
archiveFiles(modulesFiles('darwin-x64'), 'tar', 'darwin', 'x64', false)
116116
}
117117
}
118118

119-
linuxAarch64Tar {
120-
archiveClassifier = 'linux-aarch64'
119+
linuxArm64Tar {
120+
archiveClassifier = 'linux-arm64'
121121
content {
122-
archiveFiles(modulesFiles('linux-aarch64'), 'tar', 'linux', 'aarch64', true)
122+
archiveFiles(modulesFiles('linux-arm64'), 'tar', 'linux', 'arm64', true)
123123
}
124124
}
125125

126126
linuxTar {
127-
archiveClassifier = 'linux-x86_64'
127+
archiveClassifier = 'linux-x64'
128128
content {
129-
archiveFiles(modulesFiles('linux-x86_64'), 'tar', 'linux', 'x64', true)
129+
archiveFiles(modulesFiles('linux-x64'), 'tar', 'linux', 'x64', true)
130130
}
131131
}
132132

133133
noJdkLinuxTar {
134-
archiveClassifier = 'no-jdk-linux-x86_64'
134+
archiveClassifier = 'no-jdk-linux-x64'
135135
content {
136-
archiveFiles(modulesFiles('linux-x86_64'), 'tar', 'linux', 'x64', false)
136+
archiveFiles(modulesFiles('linux-x64'), 'tar', 'linux', 'x64', false)
137137
}
138138
}
139139
}

‎distribution/build.gradle

+10-4
Original file line numberDiff line numberDiff line change
@@ -312,7 +312,7 @@ configure(subprojects.findAll { ['archives', 'packages'].contains(it.name) }) {
312312
modulesFiles = { platform ->
313313
copySpec {
314314
eachFile {
315-
if (it.relativePath.segments[-2] == 'bin' || (platform == 'darwin-x86_64' && it.relativePath.segments[-2] == 'MacOS')) {
315+
if (it.relativePath.segments[-2] == 'bin' || (platform == 'darwin-x64' && it.relativePath.segments[-2] == 'MacOS')) {
316316
// bin files, wherever they are within modules (eg platform specific) should be executable
317317
// and MacOS is an alternative to bin on macOS
318318
it.mode = 0755
@@ -321,7 +321,7 @@ configure(subprojects.findAll { ['archives', 'packages'].contains(it.name) }) {
321321
}
322322
}
323323
def buildModules = buildModulesTaskProvider
324-
List excludePlatforms = ['linux-x86_64', 'linux-aarch64', 'windows-x86_64', 'darwin-x86_64']
324+
List excludePlatforms = ['linux-x64', 'linux-arm64', 'windows-x64', 'darwin-x64']
325325
if (platform != null) {
326326
excludePlatforms.remove(excludePlatforms.indexOf(platform))
327327
} else {
@@ -408,6 +408,12 @@ configure(subprojects.findAll { ['archives', 'packages'].contains(it.name) }) {
408408

409409
jdkFiles = { Project project, String platform, String architecture ->
410410
return copySpec {
411+
/*
412+
* Jdk uses aarch64 from ARM. Translating from arm64 to aarch64 which Jdk understands.
413+
*/
414+
if ("arm64".equals(architecture)) {
415+
architecture = "aarch64"
416+
}
411417
from project.jdks."bundled_${platform}_${architecture}"
412418
exclude "demo/**"
413419
/*
@@ -585,11 +591,11 @@ subprojects {
585591

586592
['archives:windows-zip',
587593
'archives:darwin-tar',
588-
'archives:linux-aarch64-tar',
594+
'archives:linux-arm64-tar',
589595
'archives:linux-tar',
590596
'archives:integ-test-zip',
591597
'packages:rpm', 'packages:deb',
592-
'packages:aarch64-rpm', 'packages:aarch64-deb'
598+
'packages:arm64-rpm', 'packages:arm64-deb'
593599
].forEach { subName ->
594600
Project subproject = project("${project.path}:${subName}")
595601
Configuration configuration = configurations.create(subproject.name)

‎distribution/docker/build.gradle

+12-12
Original file line numberDiff line numberDiff line change
@@ -24,22 +24,22 @@ apply plugin: 'opensearch.rest-resources'
2424
testFixtures.useFixture()
2525

2626
configurations {
27-
aarch64DockerSource
27+
arm64DockerSource
2828
dockerSource
2929
}
3030

3131
dependencies {
32-
aarch64DockerSource project(path: ":distribution:archives:linux-aarch64-tar", configuration:"default")
32+
arm64DockerSource project(path: ":distribution:archives:linux-arm64-tar", configuration:"default")
3333
dockerSource project(path: ":distribution:archives:linux-tar", configuration:"default")
3434
}
3535

3636
ext.expansions = { Architecture architecture, DockerBase base, boolean local ->
3737
String classifier
3838
if (local) {
39-
if (architecture == Architecture.AARCH64) {
40-
classifier = "linux-aarch64"
39+
if (architecture == Architecture.ARM64) {
40+
classifier = "linux-arm64"
4141
} else if (architecture == Architecture.X64) {
42-
classifier = "linux-x86_64"
42+
classifier = "linux-x64"
4343
} else {
4444
throw new IllegalArgumentException("Unsupported architecture [" + architecture + "]")
4545
}
@@ -82,13 +82,13 @@ RUN curl --retry 8 -S -L \\
8282

8383
private static String buildPath(Architecture architecture, DockerBase base) {
8484
return 'build/' +
85-
(architecture == Architecture.AARCH64 ? 'aarch64-' : '') +
85+
(architecture == Architecture.ARM64 ? 'arm64-' : '') +
8686
'docker'
8787
}
8888

8989
private static String taskName(String prefix, Architecture architecture, DockerBase base, String suffix) {
9090
return prefix +
91-
(architecture == Architecture.AARCH64 ? 'Aarch64' : '') +
91+
(architecture == Architecture.ARM64 ? 'Arm64' : '') +
9292
suffix
9393
}
9494

@@ -123,8 +123,8 @@ void addCopyDockerContextTask(Architecture architecture, DockerBase base) {
123123

124124
with dockerBuildContext(architecture, base, true)
125125

126-
if (architecture == Architecture.AARCH64) {
127-
from configurations.aarch64DockerSource
126+
if (architecture == Architecture.ARM64) {
127+
from configurations.arm64DockerSource
128128
} else {
129129
from configurations.dockerSource
130130
}
@@ -142,7 +142,7 @@ def createAndSetWritable(Object... locations) {
142142

143143
opensearch_distributions {
144144
Architecture.values().each { eachArchitecture ->
145-
"docker${ eachArchitecture == Architecture.AARCH64 ? '_aarch64' : '' }" {
145+
"docker${ eachArchitecture == Architecture.ARM64 ? '_arm64' : '' }" {
146146
architecture = eachArchitecture
147147
type = 'docker'
148148
version = VersionProperties.getOpenSearch()
@@ -225,10 +225,10 @@ subprojects { Project subProject ->
225225
if (subProject.name.endsWith('-export')) {
226226
apply plugin: 'distribution'
227227

228-
final Architecture architecture = subProject.name.contains('aarch64-') ? Architecture.AARCH64 : Architecture.X64
228+
final Architecture architecture = subProject.name.contains('arm64-') ? Architecture.ARM64 : Architecture.X64
229229
final DockerBase base = DockerBase.CENTOS
230230

231-
final String arch = architecture == Architecture.AARCH64 ? '-aarch64' : ''
231+
final String arch = architecture == Architecture.ARM64 ? '-arm64' : ''
232232
final String extension = 'docker.tar'
233233
final String artifactName = "opensearch${arch}_test"
234234

‎distribution/packages/build.gradle

+8-8
Original file line numberDiff line numberDiff line change
@@ -116,21 +116,21 @@ Closure commonPackageConfig(String type, boolean jdk, String architecture) {
116116
if (architecture == 'x64') {
117117
arch('amd64')
118118
} else {
119-
assert architecture == 'aarch64' : architecture
119+
assert architecture == 'arm64' : architecture
120120
arch('arm64')
121121
}
122122
} else {
123123
assert type == 'rpm' : type
124124
if (architecture == 'x64') {
125125
arch('X86_64')
126126
} else {
127-
assert architecture == 'aarch64' : architecture
127+
assert architecture == 'arm64' : architecture
128128
arch('aarch64')
129129
}
130130
}
131131
// Follow opensearch's file naming convention
132132
String jdkString = jdk ? "" : "-no-jdk"
133-
String prefix = "${architecture == 'aarch64' ? 'aarch64-' : ''}${jdk ? '' : 'no-jdk-'}${type}"
133+
String prefix = "${architecture == 'arm64' ? 'arm64-' : ''}${jdk ? '' : 'no-jdk-'}${type}"
134134
destinationDirectory = file("${prefix}/build/distributions")
135135

136136
// SystemPackagingTask overrides default archive task convention mappings, but doesn't provide a setter so we have to override the convention mapping itself
@@ -166,7 +166,7 @@ Closure commonPackageConfig(String type, boolean jdk, String architecture) {
166166
with libFiles()
167167
}
168168
into('modules') {
169-
with modulesFiles('linux-' + ((architecture == 'x64') ? 'x86_64' : architecture))
169+
with modulesFiles('linux-' + ((architecture == 'x64') ? 'x64' : architecture))
170170
}
171171
if (jdk) {
172172
into('jdk') {
@@ -345,8 +345,8 @@ Closure commonDebConfig(boolean jdk, String architecture) {
345345
}
346346
}
347347

348-
tasks.register('buildAarch64Deb', Deb) {
349-
configure(commonDebConfig(true, 'aarch64'))
348+
tasks.register('buildArm64Deb', Deb) {
349+
configure(commonDebConfig(true, 'arm64'))
350350
}
351351

352352
tasks.register('buildDeb', Deb) {
@@ -382,8 +382,8 @@ Closure commonRpmConfig(boolean jdk, String architecture) {
382382
}
383383
}
384384

385-
tasks.register('buildAarch64Rpm', Rpm) {
386-
configure(commonRpmConfig(true, 'aarch64'))
385+
tasks.register('buildArm64Rpm', Rpm) {
386+
configure(commonRpmConfig(true, 'arm64'))
387387
}
388388

389389
tasks.register('buildRpm', Rpm) {

‎distribution/tools/plugin-cli/src/test/java/org/opensearch/plugins/InstallPluginCommandTests.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -647,7 +647,7 @@ public void testPluginPermissions() throws Exception {
647647
final Path pluginDir = createPluginDir(temp);
648648
final Path resourcesDir = pluginDir.resolve("resources");
649649
final Path platformDir = pluginDir.resolve("platform");
650-
final Path platformNameDir = platformDir.resolve("linux-x86_64");
650+
final Path platformNameDir = platformDir.resolve("linux-x64");
651651
final Path platformBinDir = platformNameDir.resolve("bin");
652652
Files.createDirectories(platformBinDir);
653653

@@ -664,7 +664,7 @@ public void testPluginPermissions() throws Exception {
664664
final Path fake = env.v2().pluginsFile().resolve("fake");
665665
final Path resources = fake.resolve("resources");
666666
final Path platform = fake.resolve("platform");
667-
final Path platformName = platform.resolve("linux-x86_64");
667+
final Path platformName = platform.resolve("linux-x64");
668668
final Path bin = platformName.resolve("bin");
669669
assert755(fake);
670670
assert644(fake.resolve("fake-" + Version.CURRENT + ".jar"));

‎modules/ingest-user-agent/src/main/resources/regexes.yml

+13-13
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ user_agent_parsers:
9292
# Pingdom
9393
- regex: '(Pingdom\.com_bot_version_)(\d+)\.(\d+)'
9494
family_replacement: 'PingdomBot'
95-
# 'Mozilla/5.0 (Unknown; Linux x86_64) AppleWebKit/534.34 (KHTML, like Gecko) PingdomTMS/0.8.5 Safari/534.34'
95+
# 'Mozilla/5.0 (Unknown; Linux x64) AppleWebKit/534.34 (KHTML, like Gecko) PingdomTMS/0.8.5 Safari/534.34'
9696
- regex: '(PingdomTMS)/(\d+)\.(\d+)\.(\d+)'
9797
family_replacement: 'PingdomBot'
9898

@@ -1093,8 +1093,8 @@ os_parsers:
10931093
# Note, this needs to come before the windows parsers as the app doesn't
10941094
# properly identify as Chrome OS
10951095
#
1096-
# ex: Mozilla/5.0 (X11; Windows aarch64 10718.88.2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/68.0.3440.118 Safari/537.36 CitrixChromeApp
1097-
- regex: '(x86_64|aarch64)\ (\d+)\.(\d+)\.(\d+).*Chrome.*(?:CitrixChromeApp)$'
1096+
# ex: Mozilla/5.0 (X11; Windows arm64 10718.88.2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/68.0.3440.118 Safari/537.36 CitrixChromeApp
1097+
- regex: '(x64|arm64)\ (\d+)\.(\d+)\.(\d+).*Chrome.*(?:CitrixChromeApp)$'
10981098
os_replacement: 'Chrome OS'
10991099

11001100
##########
@@ -1209,27 +1209,27 @@ os_parsers:
12091209
os_v2_replacement: '$2'
12101210
os_v3_replacement: '$3'
12111211
# Leopard
1212-
- regex: ' (Dar)(win)/(9).(\d+).*\((?:i386|x86_64|Power Macintosh)\)'
1212+
- regex: ' (Dar)(win)/(9).(\d+).*\((?:i386|x64|Power Macintosh)\)'
12131213
os_replacement: 'Mac OS X'
12141214
os_v1_replacement: '10'
12151215
os_v2_replacement: '5'
12161216
# Snow Leopard
1217-
- regex: ' (Dar)(win)/(10).(\d+).*\((?:i386|x86_64)\)'
1217+
- regex: ' (Dar)(win)/(10).(\d+).*\((?:i386|x64)\)'
12181218
os_replacement: 'Mac OS X'
12191219
os_v1_replacement: '10'
12201220
os_v2_replacement: '6'
12211221
# Lion
1222-
- regex: ' (Dar)(win)/(11).(\d+).*\((?:i386|x86_64)\)'
1222+
- regex: ' (Dar)(win)/(11).(\d+).*\((?:i386|x64)\)'
12231223
os_replacement: 'Mac OS X'
12241224
os_v1_replacement: '10'
12251225
os_v2_replacement: '7'
12261226
# Mountain Lion
1227-
- regex: ' (Dar)(win)/(12).(\d+).*\((?:i386|x86_64)\)'
1227+
- regex: ' (Dar)(win)/(12).(\d+).*\((?:i386|x64)\)'
12281228
os_replacement: 'Mac OS X'
12291229
os_v1_replacement: '10'
12301230
os_v2_replacement: '8'
12311231
# Mavericks
1232-
- regex: ' (Dar)(win)/(13).(\d+).*\((?:i386|x86_64)\)'
1232+
- regex: ' (Dar)(win)/(13).(\d+).*\((?:i386|x64)\)'
12331233
os_replacement: 'Mac OS X'
12341234
os_v1_replacement: '10'
12351235
os_v2_replacement: '9'
@@ -1339,15 +1339,15 @@ os_parsers:
13391339
# CFNetwork macOS Apps (must be before CFNetwork iOS Apps
13401340
# @ref: https://en.wikipedia.org/wiki/Darwin_(operating_system)#Release_history
13411341
##########
1342-
- regex: 'CFNetwork/.* Darwin/17\.\d+.*\(x86_64\)'
1342+
- regex: 'CFNetwork/.* Darwin/17\.\d+.*\(x64\)'
13431343
os_replacement: 'Mac OS X'
13441344
os_v1_replacement: '10'
13451345
os_v2_replacement: '13'
1346-
- regex: 'CFNetwork/.* Darwin/16\.\d+.*\(x86_64\)'
1346+
- regex: 'CFNetwork/.* Darwin/16\.\d+.*\(x64\)'
13471347
os_replacement: 'Mac OS X'
13481348
os_v1_replacement: '10'
13491349
os_v2_replacement: '12'
1350-
- regex: 'CFNetwork/8.* Darwin/15\.\d+.*\(x86_64\)'
1350+
- regex: 'CFNetwork/8.* Darwin/15\.\d+.*\(x64\)'
13511351
os_replacement: 'Mac OS X'
13521352
os_v1_replacement: '10'
13531353
os_v2_replacement: '11'
@@ -1625,7 +1625,7 @@ os_parsers:
16251625
# Wget/x.x.x (linux-gnu)
16261626
- regex: '\(linux-gnu\)'
16271627
os_replacement: 'Linux'
1628-
- regex: '\(x86_64-redhat-linux-gnu\)'
1628+
- regex: '\(x64-redhat-linux-gnu\)'
16291629
os_replacement: 'Red Hat'
16301630
- regex: '\((freebsd)(\d+)\.(\d+)\)'
16311631
os_replacement: 'FreeBSD'
@@ -5099,7 +5099,7 @@ device_parsers:
50995099
model_replacement: '$1$2,$3'
51005100
# @note: newer desktop applications don't show device info
51015101
# This is here so as to not have them recorded as iOS-Device
5102-
- regex: 'CFNetwork/.* Darwin/\d+\.\d+\.\d+ \(x86_64\)'
5102+
- regex: 'CFNetwork/.* Darwin/\d+\.\d+\.\d+ \(x64\)'
51035103
device_replacement: 'Mac'
51045104
brand_replacement: 'Apple'
51055105
model_replacement: 'Mac'

‎server/src/main/java/org/opensearch/plugins/Platforms.java

+6-6
Original file line numberDiff line numberDiff line change
@@ -69,13 +69,13 @@ public static Path nativeControllerPath(Path plugin) {
6969

7070
/**
7171
* Return the platform name based on the OS name and architecture, for example:
72-
* - darwin-x86_64
73-
* - linux-x86-64
74-
* - windows-x86_64
72+
* - darwin-x64
73+
* - linux-x64
74+
* - windows-x64
7575
* For *nix platforms this is more-or-less `uname -s`-`uname -m` converted to lower case.
7676
* However, for consistency between different operating systems on the same architecture
77-
* "amd64" is replaced with "x86_64" and "i386" with "x86".
78-
* For Windows it's "windows-" followed by either "x86" or "x86_64".
77+
* "amd64" is replaced with "x64" and "i386" with "x86".
78+
* For Windows it's "windows-" followed by either "x86" or "x64".
7979
*/
8080
public static String platformName(final String osName, final String osArch) {
8181
final String lowerCaseOs = osName.toLowerCase(Locale.ROOT);
@@ -91,7 +91,7 @@ public static String platformName(final String osName, final String osArch) {
9191
final String lowerCaseArch = osArch.toLowerCase(Locale.ROOT);
9292
final String normalizedArch;
9393
if (lowerCaseArch.equals("amd64")) {
94-
normalizedArch = "x86_64";
94+
normalizedArch = "x64";
9595
} else if (lowerCaseArch.equals("i386")) {
9696
normalizedArch = "x86";
9797
} else {

‎server/src/test/java/org/opensearch/plugins/PluginsTests.java

+5-5
Original file line numberDiff line numberDiff line change
@@ -50,13 +50,13 @@ public void testMakePlatformName() {
5050
}
5151

5252
public void testMakeSpecificPlatformNames() {
53-
assertEquals("darwin-x86_64", Platforms.platformName("Mac OS X", "x86_64"));
54-
assertEquals("linux-x86_64", Platforms.platformName("Linux", "amd64"));
53+
assertEquals("darwin-x64", Platforms.platformName("Mac OS X", "x64"));
54+
assertEquals("linux-x64", Platforms.platformName("Linux", "amd64"));
5555
assertEquals("linux-x86", Platforms.platformName("Linux", "i386"));
56-
assertEquals("windows-x86_64", Platforms.platformName("Windows Server 2008 R2", "amd64"));
56+
assertEquals("windows-x64", Platforms.platformName("Windows Server 2008 R2", "amd64"));
5757
assertEquals("windows-x86", Platforms.platformName("Windows Server 2008", "x86"));
58-
assertEquals("windows-x86_64", Platforms.platformName("Windows 8.1", "amd64"));
59-
assertEquals("sunos-x86_64", Platforms.platformName("SunOS", "amd64"));
58+
assertEquals("windows-x64", Platforms.platformName("Windows 8.1", "amd64"));
59+
assertEquals("sunos-x64", Platforms.platformName("SunOS", "amd64"));
6060
}
6161

6262
}

‎settings.gradle

+5-5
Original file line numberDiff line numberDiff line change
@@ -33,18 +33,18 @@ List projects = [
3333
'distribution:archives:no-jdk-windows-zip',
3434
'distribution:archives:darwin-tar',
3535
'distribution:archives:no-jdk-darwin-tar',
36-
'distribution:archives:linux-aarch64-tar',
36+
'distribution:archives:linux-arm64-tar',
3737
'distribution:archives:linux-tar',
3838
'distribution:archives:no-jdk-linux-tar',
3939
'distribution:docker',
40-
'distribution:docker:docker-aarch64-build-context',
41-
'distribution:docker:docker-aarch64-export',
40+
'distribution:docker:docker-arm64-build-context',
41+
'distribution:docker:docker-arm64-export',
4242
'distribution:docker:docker-build-context',
4343
'distribution:docker:docker-export',
44-
'distribution:packages:aarch64-deb',
44+
'distribution:packages:arm64-deb',
4545
'distribution:packages:deb',
4646
'distribution:packages:no-jdk-deb',
47-
'distribution:packages:aarch64-rpm',
47+
'distribution:packages:arm64-rpm',
4848
'distribution:packages:rpm',
4949
'distribution:packages:no-jdk-rpm',
5050
'distribution:bwc:bugfix',

0 commit comments

Comments
 (0)
Please sign in to comment.