Skip to content

Commit 279c6a6

Browse files
author
root
committed
Standardize s390x naming in UI and use CPUArch in KVM
1 parent f5b613a commit 279c6a6

File tree

5 files changed

+7
-5
lines changed

5 files changed

+7
-5
lines changed

plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
import java.util.List;
2828
import java.util.Map;
2929

30+
import com.cloud.cpu.CPU;
3031
import org.apache.cloudstack.api.ApiConstants.IoDriverPolicy;
3132
import org.apache.cloudstack.utils.qemu.QemuObject;
3233
import org.apache.commons.lang.StringEscapeUtils;
@@ -420,7 +421,7 @@ public String toString() {
420421
guestDef.append("<boot dev='" + bo + "'/>\n");
421422
}
422423
}
423-
if (!(_arch != null && _arch.equals("s390x"))) {
424+
if (!CPU.CPUArch.s390x.getType().equalsIgnoreCase(_arch)) {
424425
guestDef.append("<smbios mode='sysinfo'/>\n");
425426
}
426427
guestDef.append("</os>\n");

plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResourceTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@
6161
import javax.xml.xpath.XPathExpressionException;
6262
import javax.xml.xpath.XPathFactory;
6363

64+
import com.cloud.cpu.CPU;
6465
import com.cloud.utils.net.NetUtils;
6566

6667
import com.cloud.vm.VmDetailConstants;
@@ -865,7 +866,7 @@ private void verifyPoliticOn_(Document domainDoc) {
865866
private void verifyFeatures(Document domainDoc) {
866867
assertNodeExists(domainDoc, "/domain/features/pae");
867868
assertNodeExists(domainDoc, "/domain/features/apic");
868-
if (!"s390x".equals(System.getProperty("os.arch"))) {
869+
if (!CPU.CPUArch.s390x.getType().equalsIgnoreCase(System.getProperty("os.arch"))) {
869870
assertNodeExists(domainDoc, "/domain/features/acpi");
870871
}
871872
}

ui/src/utils/plugins.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -558,7 +558,7 @@ export const cpuArchitectureUtilPlugin = {
558558
const architectures = [
559559
{ id: 'x86_64', name: 'Intel/AMD 64 bits (x86_64)' },
560560
{ id: 'aarch64', name: 'ARM 64 bits (aarch64)' },
561-
{ id: 's390x', name: 'IBM Z/Architecture 64 bits (s390x)' }
561+
{ id: 's390x', name: 'IBM Z 64 bits (s390x)' }
562562
]
563563
return architectures.map(item => ({ ...item, description: item.name }))
564564
}

ui/src/views/infra/ClusterUpdate.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ export default {
170170
})
171171
typesList.push({
172172
id: 's390x',
173-
description: 'S390X 64 bits (s390x)'
173+
description: 'IBM Z 64 bits (s390x)'
174174
})
175175
this.architectureTypes.opts = typesList
176176
},

ui/src/views/infra/zone/ZoneWizardAddResources.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -867,7 +867,7 @@ export default {
867867
description: 'ARM 64 bits (aarch64)'
868868
}, {
869869
id: 's390x',
870-
description: 'S390X 64 bits (s390x)'
870+
description: 'IBM Z 64 bits (s390x)'
871871
}],
872872
storageProviders: [],
873873
currentStep: null,

0 commit comments

Comments
 (0)