Update storage role defaults

- Don't run checks/validations by default
- Do configure networking by default
- Skip xz-utils installation on debian when possible

Author: serpro69

metal/roles/storage/defaults/main.yml

@@ -19,7 +19,7 @@ firewall_allowed_networks: # NB! The default will only allow connections (includ
   - "{{ homelab_net_cidr }}"
 
 # Network configuration (optional)
-configure_network: false
+configure_network: true
 storage_network:
   interface: "{{ network_interface | default('eth0') }}"
   address: "{{ ansible_host }}"
@@ -29,17 +29,6 @@ storage_network:
     - "10.10.10.1"
     - "1.1.1.1"
 
-# Disk health validation
-enable_disk_health_checks: false
-prefer_native_utils: false # Use smartctl by default, set true for brand-specific tools
-fail_on_unhealthy_drives: false # Warning only by default
-
-disk_health_thresholds:
-  max_temperature_celsius: 60
-  max_reallocated_sectors: 50
-  max_pending_sectors: 0
-  max_uncorrectable_errors: 0
-
 # Snapraid maintenance configuration
 snapraid_maintenance:
   # Daily sync schedule (systemd timer format)
@@ -78,5 +67,21 @@ snapraid_maintenance:
   content_backup_enabled: true
   content_backup_count: 5 # Keep last N backups
 
+# Don't upgrade all packages on each run
 # Set to true (e.g. via cli: '-e upgrade_packages=true') to upgrade all packages on the host
 upgrade_packages: false
+
+# Healthchecks, validations, tests, etc.
+
+# Disk health validation
+enable_disk_health_checks: false
+prefer_native_utils: false # Use smartctl by default, set true for brand-specific tools
+fail_on_unhealthy_drives: false # Warning only by default
+disk_health_thresholds:
+  max_temperature_celsius: 60
+  max_reallocated_sectors: 50
+  max_pending_sectors: 0
+  max_uncorrectable_errors: 0
+
+# Run nfs validations
+validate_nfs: false

metal/roles/storage/files/.gitignore

@@ -0,0 +1,2 @@
+*.deb
+*.rpm

metal/roles/storage/tasks/network.yml

@@ -5,7 +5,7 @@
     dest: /etc/network/interfaces
     owner: root
     group: root
-    mode: '0644'
+    mode: "0644"
     backup: true
   when: ansible_os_family == 'Debian'
   notify: restart networking debian
@@ -16,7 +16,7 @@
     dest: "/etc/sysconfig/network-scripts/ifcfg-{{ storage_network.interface }}"
     owner: root
     group: root
-    mode: '0644'
+    mode: "0644"
     backup: true
   when: ansible_os_family == 'RedHat'
   notify: restart networking redhat

metal/roles/storage/tasks/nfs.yml

@@ -62,10 +62,12 @@
 
 - name: Validate NFS exports
   include_tasks: nfs_validation.yml
+  when: validate_nfs
 
 - name: Validate NFS-K8S integration
   include_tasks:
     file: k8s_validation.yml
     apply:
       become: false
       delegate_to: localhost
+  when: validate_nfs

metal/roles/storage/tasks/setup_debian.yml

@@ -8,8 +8,6 @@
 - name: Ensure base packages
   apt:
     name:
-      # temp 'apt' module dependency for installing .deb
-      - xz-utils
       # hardware monitoring
       - lm-sensors
       # nfs
@@ -30,6 +28,15 @@
     owner: root
     group: root
     mode: "0755"
+  register: snapraid_deb
+
+- name: Ensure apt module dependencies
+  apt:
+    name:
+      # temp 'apt' module dependency for installing .deb
+      - xz-utils
+    state: present
+  when: snapraid_deb is changed
 
 - name: Install snapraid
   apt: