feat: add suffix to the s390x templates

This change also involves renaming the templates themselves. The end-to-end (e2e) tests require minimal adjustments. The following templates will be generated for s390x: - CentOS Stream 9 - Fedora - RHEL 8 - RHEL 9 - Ubuntu Signed-off-by: Nestor Acuna Blanco <[email protected]>
kubevirt · Nov 29, 2024 · 0d85b86 · 0d85b86
1 parent 16d3b8d
commit 0d85b86
Show file tree

Hide file tree

Showing 8 changed files with 32 additions and 28 deletions.
diff --git a/automation/test-linux.sh b/automation/test-linux.sh
@@ -163,9 +163,10 @@ run_vm() {
   fi
 }
 
+target_arch_suffix=$( [ "$TARGET_ARCH" = "x86_64" ] && echo "" || echo "-$TARGET_ARCH" )
 for size in "${sizes[@]}"; do
   for workload in "${workloads[@]}"; do
-    vm_name=$template_name-$workload-$size
+    vm_name=$template_name-$workload-$size$target_arch_suffix
     run_vm "$vm_name"
   done
 done
diff --git a/generate-templates.yaml b/generate-templates.yaml
@@ -2,6 +2,8 @@
 - connection: local
   hosts: 127.0.0.1
   gather_facts: yes
+  vars:
+    suffix: "{{ '' if target_arch == 'x86_64' else '-' + target_arch }}"
   tasks:
   - name: Ensure target_arch is defined
     assert:
@@ -27,7 +29,7 @@
   - name: Generate RHEL 9 templates
     template:
       src: rhel9.tpl.yaml
-      dest: "{{ playbook_dir }}/dist/templates/{{ os }}-{{ item.workload }}-{{ item.flavor }}.yaml"
+      dest: "{{ playbook_dir }}/dist/templates/{{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}.yaml"
     with_items:
     - {flavor: tiny,   workload: server,          memsize: "1.5Gi", cpus: 1, iothreads: False, emulatorthread: False, tablet: False, default: False}
     - {flavor: tiny,   workload: desktop,         memsize: "1.5Gi", cpus: 1, iothreads: False, emulatorthread: False, tablet: True,  default: False}
@@ -62,7 +64,7 @@
   - name: Generate RHEL 8 templates
     template:
       src: rhel8.tpl.yaml
-      dest: "{{ playbook_dir }}/dist/templates/{{ os }}-{{ item.workload }}-{{ item.flavor }}.yaml"
+      dest: "{{ playbook_dir }}/dist/templates/{{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}.yaml"
     with_items:
     - {flavor: tiny,   workload: server,          memsize: "1.5Gi", cpus: 1, iothreads: False, emulatorthread: False, tablet: False, default: False}
     - {flavor: tiny,   workload: desktop,         memsize: "1.5Gi", cpus: 1, iothreads: False, emulatorthread: False, tablet: True,  default: False}
@@ -131,7 +133,7 @@
   - name: Generate CentOS Stream 9 templates
     template:
       src: centos-stream9.tpl.yaml
-      dest: "{{ playbook_dir }}/dist/templates/{{ os }}-{{ item.workload }}-{{ item.flavor }}.yaml"
+      dest: "{{ playbook_dir }}/dist/templates/{{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}.yaml"
     with_items:
     - {flavor: tiny,   workload: server,  memsize: "1.5Gi", cpus: 1, iothreads: False, emulatorthread: False, tablet: False, default: False}
     - {flavor: tiny,   workload: desktop, memsize: "1.5Gi", cpus: 1, iothreads: False, emulatorthread: False, tablet: True,  default: False}
@@ -188,7 +190,7 @@
   - name: Generate Fedora templates
     template:
       src: fedora.tpl.yaml
-      dest: "{{ playbook_dir }}/dist/templates/{{ os }}-{{ item.workload }}-{{ item.flavor }}.yaml"
+      dest: "{{ playbook_dir }}/dist/templates/{{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}.yaml"
     with_items:
     - {flavor: small,  workload: desktop,         memsize: "2Gi", cpus: 1, iothreads: False, emulatorthread: False, tablet: True,  default: False}
     - {flavor: small,  workload: server,          memsize: "2Gi", cpus: 1, iothreads: False, emulatorthread: False, tablet: False, default: True}
@@ -252,7 +254,7 @@
   - name: Generate Ubuntu templates
     template:
       src: ubuntu.tpl.yaml
-      dest: "{{ playbook_dir }}/dist/templates/{{ os }}-{{ item.workload }}-{{ item.flavor }}.yaml"
+      dest: "{{ playbook_dir }}/dist/templates/{{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}.yaml"
     with_items:
     - {flavor: small,  workload: desktop, memsize: "3Gi", cpus: 1, iothreads: False, emulatorthread: False, tablet: True, default: False}
     - {flavor: small,  workload: server,  memsize: "3Gi", cpus: 1, iothreads: False, emulatorthread: False, tablet: True, default: True}

diff --git a/makefile b/makefile
@@ -30,7 +30,7 @@ release: dist/common-templates.yaml
 	cp dist/common-templates.yaml dist/common-templates-$(VERSION).yaml
 
 e2e-tests:
-	./automation/test.sh
+	TARGET_ARCH=$(TARGET_ARCH) ./automation/test.sh
 
 go-tests:
 	go test -v ./tests/
@@ -44,7 +44,8 @@ validate-no-offensive-lang:
 generate: generate-templates.yaml $(METASOURCES)
 	# Just build the XML files, no need to export to tarball
 	make -C osinfo-db/ OSINFO_DB_EXPORT=echo
-	ansible-playbook generate-templates.yaml -e "target_arch=$(TARGET_ARCH)"
+	ansible-playbook generate-templates.yaml -e "target_arch=x86_64"
+	ansible-playbook generate-templates.yaml -e "target_arch=s390x"
 
 update-osinfo-db:
 	git submodule init

diff --git a/templates/centos-stream9.tpl.yaml b/templates/centos-stream9.tpl.yaml
@@ -1,7 +1,7 @@
 apiVersion: template.openshift.io/v1
 kind: Template
 metadata:
-  name: {{ os }}-{{ item.workload }}-{{ item.flavor }}
+  name: {{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}
   annotations:
     openshift.io/display-name: "CentOS Stream 9 VM"
     description: >-
@@ -36,11 +36,11 @@ metadata:
       /objects[0].spec.template.spec.volumes
       /objects[0].spec.template.spec.networks
 {% for osl in oslabels %}
-    name.os.template.kubevirt.io/{{ osl }}: {{ lookup('osinfo', oslabels[0]).name }} or higher
+    name.os.template.kubevirt.io/{{ osl }}{{ suffix }}: {{ lookup('osinfo', oslabels[0]).name }} or higher
 {% endfor %}
   labels:
 {% for osl in oslabels %}
-    os.template.kubevirt.io/{{ osl }}: "true"
+    os.template.kubevirt.io/{{ osl }}{{ suffix }}: "true"
 {% endfor %}
     workload.template.kubevirt.io/{{ item.workload }}: "true"
     flavor.template.kubevirt.io/{{ item.flavor }}: "true"
@@ -55,7 +55,7 @@ objects:
   metadata:
     name: ${NAME}
     labels:
-      vm.kubevirt.io/template: {{ os }}-{{ item.workload }}-{{ item.flavor }}
+      vm.kubevirt.io/template: {{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}
       vm.kubevirt.io/template.version: "{{ lookup('env', 'VERSION') | default('devel', true) }}"
       vm.kubevirt.io/template.revision: "{{ lookup('env', 'REVISION') | default(1, true) }}"
       app: ${NAME}

diff --git a/templates/fedora.tpl.yaml b/templates/fedora.tpl.yaml
@@ -1,7 +1,7 @@
 apiVersion: template.openshift.io/v1
 kind: Template
 metadata:
-  name: {{ os }}-{{ item.workload }}-{{ item.flavor }}
+  name: {{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}
   annotations:
     openshift.io/display-name: "Fedora VM"
     description: >-
@@ -35,11 +35,11 @@ metadata:
       /objects[0].spec.template.spec.volumes
       /objects[0].spec.template.spec.networks
 {% for osl in oslabels %}
-    name.os.template.kubevirt.io/{{ osl }}: {{ lookup('osinfo', oslabels[0]).name }} or higher
+    name.os.template.kubevirt.io/{{ osl }}{{ suffix }}: {{ lookup('osinfo', oslabels[0]).name }} or higher
 {% endfor %}
   labels:
 {% for osl in oslabels %}
-    os.template.kubevirt.io/{{ osl }}: "true"
+    os.template.kubevirt.io/{{ osl }}{{ suffix }}: "true"
 {% endfor %}
     workload.template.kubevirt.io/{{ item.workload }}: "true"
     flavor.template.kubevirt.io/{{ item.flavor }}: "true"
@@ -54,7 +54,7 @@ objects:
   metadata:
     name: ${NAME}
     labels:
-      vm.kubevirt.io/template: {{ os }}-{{ item.workload }}-{{ item.flavor }}
+      vm.kubevirt.io/template: {{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}
       vm.kubevirt.io/template.version: "{{ lookup('env', 'VERSION') | default('devel', true) }}"
       vm.kubevirt.io/template.revision: "{{ lookup('env', 'REVISION') | default(1, true) }}"
       app: ${NAME}

diff --git a/templates/rhel8.tpl.yaml b/templates/rhel8.tpl.yaml
@@ -1,7 +1,7 @@
 apiVersion: template.openshift.io/v1
 kind: Template
 metadata:
-  name: {{ os }}-{{ item.workload }}-{{ item.flavor }}
+  name: {{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}
   annotations:
     openshift.io/display-name: "Red Hat Enterprise Linux 8 VM"
     description: >-
@@ -32,11 +32,11 @@ metadata:
       /objects[0].spec.template.spec.volumes
       /objects[0].spec.template.spec.networks
 {% for osl in oslabels %}
-    name.os.template.kubevirt.io/{{ osl }}: {{ lookup('osinfo', oslabels[0]).name }} or higher
+    name.os.template.kubevirt.io/{{ osl }}{{ suffix }}: {{ lookup('osinfo', oslabels[0]).name }} or higher
 {% endfor %}
   labels:
 {% for osl in oslabels %}
-    os.template.kubevirt.io/{{ osl }}: "true"
+    os.template.kubevirt.io/{{ osl }}{{ suffix }}: "true"
 {% endfor %}
     workload.template.kubevirt.io/{{ item.workload }}: "true"
     flavor.template.kubevirt.io/{{ item.flavor }}: "true"
@@ -51,7 +51,7 @@ objects:
   metadata:
     name: ${NAME}
     labels:
-      vm.kubevirt.io/template: {{ os }}-{{ item.workload }}-{{ item.flavor }}
+      vm.kubevirt.io/template: {{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}
       vm.kubevirt.io/template.version: "{{ lookup('env', 'VERSION') | default('devel', true) }}"
       vm.kubevirt.io/template.revision: "{{ lookup('env', 'REVISION') | default(1, true) }}"
       app: ${NAME}

diff --git a/templates/rhel9.tpl.yaml b/templates/rhel9.tpl.yaml
@@ -1,7 +1,7 @@
 apiVersion: template.openshift.io/v1
 kind: Template
 metadata:
-  name: {{ os }}-{{ item.workload }}-{{ item.flavor }}
+  name: {{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}
   annotations:
     openshift.io/display-name: "Red Hat Enterprise Linux 9 VM"
     description: >-
@@ -32,11 +32,11 @@ metadata:
       /objects[0].spec.template.spec.volumes
       /objects[0].spec.template.spec.networks
 {% for osl in oslabels %}
-    name.os.template.kubevirt.io/{{ osl }}: {{ lookup('osinfo', oslabels[0]).name }} or higher
+    name.os.template.kubevirt.io/{{ osl }}{{ suffix }}: {{ lookup('osinfo', oslabels[0]).name }} or higher
 {% endfor %}
   labels:
 {% for osl in oslabels %}
-    os.template.kubevirt.io/{{ osl }}: "true"
+    os.template.kubevirt.io/{{ osl }}{{ suffix }}: "true"
 {% endfor %}
     workload.template.kubevirt.io/{{ item.workload }}: "true"
     flavor.template.kubevirt.io/{{ item.flavor }}: "true"
@@ -51,7 +51,7 @@ objects:
   metadata:
     name: ${NAME}
     labels:
-      vm.kubevirt.io/template: {{ os }}-{{ item.workload }}-{{ item.flavor }}
+      vm.kubevirt.io/template: {{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}
       vm.kubevirt.io/template.version: "{{ lookup('env', 'VERSION') | default('devel', true) }}"
       vm.kubevirt.io/template.revision: "{{ lookup('env', 'REVISION') | default(1, true) }}"
       app: ${NAME}

diff --git a/templates/ubuntu.tpl.yaml b/templates/ubuntu.tpl.yaml
@@ -1,7 +1,7 @@
 apiVersion: template.openshift.io/v1
 kind: Template
 metadata:
-  name: {{ os }}-{{ item.workload }}-{{ item.flavor }}
+  name: {{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}
   annotations:
     openshift.io/display-name: "Ubuntu {{ lookup('osinfo', oslabels[0]).version }}+ VM"
     description: >-
@@ -36,11 +36,11 @@ metadata:
       /objects[0].spec.template.spec.volumes
       /objects[0].spec.template.spec.networks
 {% for osl in oslabels %}
-    name.os.template.kubevirt.io/{{ osl }}: Ubuntu {{ lookup('osinfo', oslabels[0]).version }} or higher
+    name.os.template.kubevirt.io/{{ osl }}{{ suffix }}: Ubuntu {{ lookup('osinfo', oslabels[0]).version }} or higher
 {% endfor %}
   labels:
 {% for osl in oslabels %}
-    os.template.kubevirt.io/{{ osl }}: "true"
+    os.template.kubevirt.io/{{ osl }}{{ suffix }}: "true"
 {% endfor %}
     workload.template.kubevirt.io/{{ item.workload }}: "true"
     flavor.template.kubevirt.io/{{ item.flavor }}: "true"
@@ -55,7 +55,7 @@ objects:
   metadata:
     name: ${NAME}
     labels:
-      vm.kubevirt.io/template: {{ os }}-{{ item.workload }}-{{ item.flavor }}
+      vm.kubevirt.io/template: {{ os }}-{{ item.workload }}-{{ item.flavor }}{{ suffix }}
       vm.kubevirt.io/template.version: "{{ lookup('env', 'VERSION') | default('devel', true) }}"
       vm.kubevirt.io/template.revision: "{{ lookup('env', 'REVISION') | default(1, true) }}"
       app: ${NAME}