Fixing some ansible-lint issues

roles/netbootxyz/meta/main.yml

@@ -10,6 +10,6 @@ galaxy_info:
     - name: Centos
       versions:
         - all
-  galaxy_tags: 
+  galaxy_tags:
     - system
 dependencies: []

roles/netbootxyz/tasks/generate_disks_base.yml

@@ -30,7 +30,7 @@
   - name: Install required packages
     package:
       name: "{{ item }}"
-      state: latest
+      state: present
     with_items: "{{ netbootxyz_packages }}"
 
   - name: Check out latest iPXE sources
@@ -63,4 +63,4 @@
       - src/config/local/entropy.h
       - src/config/local/crypto.h
       - src/config/local/usb.h
-      - src/config/local/settings.h
+      - src/config/local/settings.h

roles/netbootxyz/tasks/generate_disks_efi.yml

@@ -20,6 +20,8 @@
     - make EMBED=ipxe-bootloader bin-x86_64-efi/ipxe.efi
     args:
       chdir: "{{ ipxe_source_dir }}/src"
+    tags:
+    - skip_ansible_lint
 
   - name: Generate iPXE EFI ISO Disk
     shell: "{{ item }}"
@@ -33,6 +35,8 @@
     args:
       chdir: "{{ ipxe_source_dir }}/src"
       warn: false
+    tags:
+    - skip_ansible_lint
 
   - name: Copy iPXE EFI builds to http directory
     copy:

roles/netbootxyz/tasks/generate_disks_legacy.yml

@@ -16,6 +16,8 @@
     - make EMBED=ipxe-bootloader bin/ipxe.dsk bin/ipxe.iso bin/ipxe.lkrn bin/ipxe.usb bin/ipxe.kpxe bin/undionly.kpxe
     args:
       chdir: "{{ ipxe_source_dir }}/src"
+    tags:
+    - skip_ansible_lint
 
   - name: Copy iPXE files for Legacy BIOS to http directory
     copy:

roles/netbootxyz/tasks/generate_menus.yml

@@ -1,36 +1,34 @@
 ---
   - name: Combine overrides with release defaults
-    set_fact: 
+    set_fact:
       _releases: "{{ releases|combine(release_overrides, recursive=True) }}"
+    when: release_overrides is defined
 
   - name: Set releases with user overrides
     set_fact:
       releases: "{{ _releases }}"
+    when: release_overrides is defined
 
   - name: Combine overrides with utilities defaults
-    set_fact: 
+    set_fact:
       _utilities: "{{ utilities|combine(utilities_overrides, recursive=True) }}"
+    when: utilities_overrides is defined
 
   - name: Set utility with user overrides
     set_fact:
       utilities: "{{ _utilities }}"
+    when: utilities_overrides is defined
 
   - name: Generate directories
     file:
       path: "{{ netbootxyz_root }}"
       state: directory
 
-  - name: Dump releases vars for debugging
-    debug:
-      var: releases
-
-  - name: Dump endpoint vars for debugging
-    debug:
-      var: endpoints
-
   - name: Generate netboot.xyz source files templates
     template:
       src: "{{ item.src }}"
       dest: "{{ netbootxyz_root }}/{{ item.path | regex_replace('.j2','') }}"
     with_filetree: "templates/menu/"
     when: item.state == "file"
+    tags:
+    - skip_ansible_lint

user_overrides.yml

@@ -20,10 +20,11 @@ generate_disks: true
 # set release overrides from standard netboot.xyz defaults
 #release_overrides:
 #  alpinelinux:
-#    name: "ALPINELINUX"
+#    name: "Alpine Linux"
 #  fedora:
 #    mirror: "mirrors.kernel.org"
 
 # set utilities_overrides from standard netboot.xyz defaults
 #utilities_overrides:
-  
+#  supergrub:
+#    enabled: false