Skip to content

Commit 3cbb596

Browse files
authored
Merge pull request #30 from stackhpc/fact_fix
[SRIOV/IOMMU]Fixing conditional
2 parents 8a81227 + 0ebb351 commit 3cbb596

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

roles/iommu/tasks/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
- intel_iommu=on
88
kernel_cmdline_remove: # noqa: var-naming[no-role-prefix]
99
- ^intel_iommu=
10-
when: "'Intel' in ansible_facts.processor.0"
10+
when: ansible_facts.processor | select('search','Intel')
1111

1212
- name: Set iommu=pt
1313
ansible.builtin.include_role:

roles/sriov/tasks/config.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
kernel_cmdline_remove: # noqa var-naming[no-role-prefix]
2727
- ^intel_iommu=
2828
kernel_restart_handler: "{{ sriov_restart_handler }}"
29-
when: "'Intel' in ansible_facts.processor.0"
29+
when: ansible_facts.processor | select('search','Intel')
3030

3131
- name: Set iommu=pt
3232
ansible.builtin.include_role:

roles/sriov/tasks/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
---
22
- name: Include action tasks
3-
ansible.builtin._include: "{{ sriov_action | default('all') }}.yml"
3+
ansible.builtin.include_tasks: "{{ sriov_action | default('all') }}.yml"

0 commit comments

Comments
 (0)