From 555da331ca14281c25ffe766c09c654d1db281d8 Mon Sep 17 00:00:00 2001 From: Will Szumski Date: Tue, 9 Jul 2024 14:09:53 +0100 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Dawud <7688823+technowhizz@users.noreply.github.com> --- roles/iommu/tasks/main.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/roles/iommu/tasks/main.yml b/roles/iommu/tasks/main.yml index ae6002b..cb69eb5 100644 --- a/roles/iommu/tasks/main.yml +++ b/roles/iommu/tasks/main.yml @@ -45,9 +45,10 @@ ansible.builtin.include_role: name: stackhpc.linux.grubcmdline vars: - kernel_cmdline: "{{ ['vfio-pci.ids=' + iommu_vfio_pci_ids] if iommu_vfio_pci_ids is defined else [] }}" # noqa: var-naming[no-role-prefix] + kernel_cmdline: "{{ ['vfio-pci.ids=' + iommu_vfio_pci_ids] }}" # noqa: var-naming[no-role-prefix] kernel_cmdline_remove: # noqa: var-naming[no-role-prefix] - ^vfio-pci\.ids= + when: iommu_vfio_pci_ids is defined - name: Set iommu=pt ansible.builtin.include_role: