Skip to content

Commit bf4a804

Browse files
Merge pull request #447 from eifrach/PPC_for_older_OCP
fix issue where performance profile generator fails
2 parents 6b71b22 + c2a36e0 commit bf4a804

5 files changed

Lines changed: 27 additions & 5 deletions

File tree

playbooks/compute/delete_old_clusters.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
password: "{{ hostvars[item].bmc_password }}"
1515
when: hostvars[item].vendor | default('') | lower != 'kvm'
1616
loop: "{{ groups['nodes'] | default([]) }}"
17-
17+
1818
- name: Get all Running VMs
1919
community.libvirt.virt:
2020
command: list_vms

playbooks/compute/roles/configurecluster/defaults/main.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,3 +13,4 @@ day0_installation: false
1313
mcp_wait_retries: 180
1414
mcp_wait_delay: 30
1515
ppc_image: "registry.redhat.io/openshift4/ose-cluster-node-tuning-rhel9-operator:v4.21"
16+
ppc_image_rhel8: "registry.redhat.io/openshift4/ose-cluster-node-tuning-operator:v4.12"

playbooks/compute/roles/configurecluster/tasks/ppc.yml

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,20 @@
2222
ocp_version_major: "{{ api_version.resources[0].status.desired.version.split('.')[0] }}"
2323
ocp_version_minor: "{{ api_version.resources[0].status.desired.version.split('.')[1] }}"
2424

25+
- name: Set performance profile creator
26+
ansible.builtin.set_fact:
27+
nto_ppc_image: "{{ ppc_image }}"
28+
29+
- name: Set performance profile creator for older OCP
30+
ansible.builtin.set_fact:
31+
nto_ppc_image: "{{ ppc_image_rhel8 }}"
32+
when:
33+
- (ocp_version_major ~ '.' ~ ocp_version_minor) is version('4.19', '<')
34+
2535
- name: Generate Performance Profile using PPC container
2636
containers.podman.podman_container:
2737
name: ppc_tool
28-
image: "{{ ppc_image }}"
38+
image: "{{ nto_ppc_image }}"
2939
rm: true
3040
entrypoint: performance-profile-creator
3141
detach: false
@@ -40,7 +50,7 @@
4050
register: ppc_output
4151

4252
- name: Save generated profile to file
43-
copy:
53+
ansible.builtin.copy:
4454
content: "{{ ppc_output.stdout }}"
4555
dest: "/tmp/tmp_ppc.yaml"
4656
mode: '0664'
@@ -51,6 +61,6 @@
5161
register: profile_file
5262

5363
- name: Extract CPU settings
54-
set_fact:
64+
ansible.builtin.set_fact:
5565
cpu_isolated: "{{ (profile_file.content | b64decode | from_yaml).spec.cpu.isolated }}"
5666
cpu_reserved: "{{ (profile_file.content | b64decode | from_yaml).spec.cpu.reserved }}"

playbooks/compute/roles/nrop_testing/defaults/main.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ rt_kernel: "true"
2727
must_gather_path: "/tmp/must-gather-data"
2828
performance_profile: "performance-profile.yaml"
2929
ppc_image: "registry.redhat.io/openshift4/ose-cluster-node-tuning-rhel9-operator:v4.21"
30+
ppc_image_rhel8: "registry.redhat.io/openshift4/ose-cluster-node-tuning-operator:v4.12"
3031

3132
# devices
3233
device_t1: tty1

playbooks/compute/roles/nrop_testing/tasks/performance_profile.yml

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,20 @@
1010
register: must_gather
1111
changed_when: must_gather.rc == 0
1212

13+
- name: Set performance profile creator
14+
ansible.builtin.set_fact:
15+
nrop_ppc_image: "{{ ppc_image }}"
16+
17+
- name: Set performance profile creator for older OCP
18+
ansible.builtin.set_fact:
19+
nrop_ppc_image: "{{ ppc_image_rhel8 }}"
20+
when:
21+
- (ocp_version_major ~ '.' ~ ocp_version_minor) is version('4.19', '<')
22+
1323
- name: Generate Performance Profile using PPC container
1424
containers.podman.podman_container:
1525
name: ppc_tool
16-
image: "{{ ppc_image }}"
26+
image: "{{ nrop_ppc_image }}"
1727
rm: true
1828
entrypoint: performance-profile-creator
1929
detach: false

0 commit comments

Comments
 (0)