address linter errors

Author: sjpb

handlers/main.yml

@@ -66,3 +66,7 @@
     - openhpc_slurm_service_started | bool
     - openhpc_enable.batch | default(false) | bool
     # 2nd condition required as notification happens on controller, which isn't necessarily a compute note
+
+- name: Reload facts
+  ansible.builtin.setup:
+    filter: ansible_local

tasks/facts.yml

@@ -2,6 +2,7 @@
   # this includes any dynamically-generated config, not just what is set in
   # slurm.conf
   ansible.builtin.command: scontrol show config
+  changed_when: false
   register: _scontrol_config
 
 - name: Create facts directory
@@ -20,8 +21,4 @@
     group: slurm
     mode: ug=rw,o=r # any user can run scontrol show config anyway
   register: _template_facts
-
-- name: Reload facts if necessary
-  ansible.builtin.setup:
-    filter: ansible_local
-  when: _template_facts.changed
+  notify: Reload facts