Skip to content

Commit

Permalink
[SRIOV/IOMMU]Fixing conditional to enable for variance in /proc/cpuin…
Browse files Browse the repository at this point in the history
…fo outputs

[SRIOV] Sytanx fix
  • Loading branch information
grzegorzkoper committed May 10, 2024
1 parent 8a81227 commit a005a8f
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion roles/iommu/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
- intel_iommu=on
kernel_cmdline_remove: # noqa: var-naming[no-role-prefix]
- ^intel_iommu=
when: "'Intel' in ansible_facts.processor.0"
when: ansible_facts.processor | select('search','Intel')

- name: Set iommu=pt
ansible.builtin.include_role:
Expand Down
2 changes: 1 addition & 1 deletion roles/sriov/tasks/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
kernel_cmdline_remove: # noqa var-naming[no-role-prefix]
- ^intel_iommu=
kernel_restart_handler: "{{ sriov_restart_handler }}"
when: "'Intel' in ansible_facts.processor.0"
when: ansible_facts.processor | select('search','Intel')

- name: Set iommu=pt
ansible.builtin.include_role:
Expand Down
2 changes: 1 addition & 1 deletion roles/sriov/tasks/main.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
---

Check failure on line 1 in roles/sriov/tasks/main.yml

View workflow job for this annotation

GitHub Actions / lint / Ansible 2.14 lint

schema[tasks]

Replace 'include' with either 'ansible.builtin.include_tasks' or 'ansible.builtin.import_tasks'. See https://docs.ansible.com/ansible/latest/collections/ansible/builtin/include_module.html
- name: Include action tasks

Check failure on line 2 in roles/sriov/tasks/main.yml

View workflow job for this annotation

GitHub Actions / lint / Ansible 2.12 lint

deprecated-module Deprecated module include

Check failure on line 2 in roles/sriov/tasks/main.yml

View workflow job for this annotation

GitHub Actions / lint / Ansible 2.14 lint

deprecated-module

Deprecated module. include

Check failure on line 2 in roles/sriov/tasks/main.yml

View workflow job for this annotation

GitHub Actions / lint / Ansible 2.14 lint

fqcn[action-core]

Avoid deprecated module (ansible.builtin.include)
ansible.builtin._include: "{{ sriov_action | default('all') }}.yml"
ansible.builtin.include: "{{ sriov_action | default('all') }}.yml"

0 comments on commit a005a8f

Please sign in to comment.