chore(1.1.24): Adds small fixes to previous PR (#142)

chore(1.1.24): Adds small fixes to previous PR (#142) - chore: Although a whitespace is the default parameter for a split, it seems required for the build to be successful (in section 5 there are also uses of split where the whitespace as separator is explicitly specified) - chore: Updates default option for disable usb modules --------- Co-authored-by: Victor Bolivar <[email protected]>
alivx · Apr 2, 2024 · b0c17c6 · b0c17c6
1 parent 03b60f6
commit b0c17c6
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/tasks/section_1_Initial_Setup.yaml b/tasks/section_1_Initial_Setup.yaml
@@ -492,10 +492,10 @@
           register: kernel_modules_using_usb
         - name: 1.1.24 Disable USB Storage | disables the kernel modules that are using usb storage
           command: "modprobe -r {{ item }} usb_storage"
-          loop: "{{ kernel_modules_using_usb.stdout.split() | last | split(',') }}"
+          loop: "{{ kernel_modules_using_usb.stdout.split(' ') | last | split(',') }}"
           # The last column of the 'lsmod | grep -i usb_storage' command is a comma separated list of the kernel modules that are using usb_storage
           when: kernel_modules_using_usb.stdout | length > 0
-      when: not disable_usb_kernel_modules
+      when: disable_usb_kernel_modules
 
     - name: 1.1.24 Disable USB Storage
       modprobe: