fix: Update sr_fingerprint task calls to use new structured parameters

The sr_fingerprint module was rewritten to accept structured parameters
(status, role_name, role_path, etc.) instead of a free-form sr_message.
Update the role tasks and tests to match the new module interface.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: spetrosi

tasks/main.yml

@@ -179,6 +179,10 @@
 
 - name: Record role success fingerprint
   sr_fingerprint:
-    sr_message: >-
-      success system_role:bootloader ansible_version={{ ansible_version.full }}
-      {{ ansible_facts['distribution'] }}-{{ ansible_facts['distribution_version'] }}
+    status: success
+    role_name: bootloader
+    role_path: "{{ role_path }}"
+    ansible_play_hosts_all: "{{ ansible_play_hosts_all }}"
+    distribution: "{{ ansible_facts['distribution'] }}"
+    distribution_version: "{{ ansible_facts['distribution_version'] }}"
+    write_log_file: "{{ __bootloader_write_log_file }}"

tasks/set_vars.yml

@@ -8,9 +8,13 @@
 
 - name: Record role begin fingerprint
   sr_fingerprint:
-    sr_message: >-
-      begin system_role:bootloader ansible_version={{ ansible_version.full }}
-      {{ ansible_facts['distribution'] }}-{{ ansible_facts['distribution_version'] }}
+    status: begin
+    role_name: bootloader
+    role_path: "{{ role_path }}"
+    ansible_play_hosts_all: "{{ ansible_play_hosts_all }}"
+    distribution: "{{ ansible_facts['distribution'] }}"
+    distribution_version: "{{ ansible_facts['distribution_version'] }}"
+    write_log_file: "{{ __bootloader_write_log_file }}"
 
 - name: Determine if system is ostree and set flag
   when: not __bootloader_is_ostree is defined

tests/tests_default.yml

@@ -18,18 +18,48 @@
 
     - name: Run with default parameters
       include_tasks: tasks/run_role_with_clear_facts.yml
+      vars:
+        __bootloader_write_log_file: true
 
     # look for the exact module invocation, not some other message that might contain the string
-    - name: Check system journal contains role fingerprints
+    - name: Get fingerprint entries from journal
       ansible.builtin.shell:
         executable: /bin/bash
         cmd: >-
           set -eo pipefail;
           journalctl --since "{{ __journal_start_time }}" --no-pager |
-          grep -v " Invoked with" | grep "sr_fingerprint.*begin system_role:bootloader" ||
-          { echo ERROR: BEGIN fingerprint not found; exit 1; };
-          journalctl --since "{{ __journal_start_time }}" --no-pager |
-          grep -v " Invoked with" | grep "sr_fingerprint.*success system_role:bootloader" ||
-          { echo ERROR: SUCCESS fingerprint not found; exit 1; }
+          grep -v " Invoked with" |
+          grep "sr_fingerprint.*role_name=bootloader"
+      register: __register_journal_fingerprints
       changed_when: false
       when: __register_dev_log.stat.exists
+
+    - name: Check that the log file was written
+      ansible.builtin.slurp:
+        path: /var/log/sysroles.jsonl
+      register: __register_log_file
+
+    - name: Verify log file and journal fingerprints
+      when: __register_dev_log.stat.exists
+      vars:
+        _journal_lines: "{{ __register_journal_fingerprints.stdout_lines }}"
+        _journal_begin: "{{ _journal_lines | select('search', 'status=begin') | list }}"
+        _journal_success: "{{ _journal_lines | select('search', 'status=success') | list }}"
+        _begin_date: "{{ (_journal_begin[0] | regex_search('date=([^ ]+)', '\\1'))[0] }}"
+        _success_date: "{{ (_journal_success[0] | regex_search('date=([^ ]+)', '\\1'))[0] }}"
+        _file_content: "{{ __register_log_file.content | b64decode }}"
+      block:
+        - name: Print contents of logs
+          debug:
+            var: item
+          loop:
+            - "{{ _file_content }}"
+            - "{{ _journal_lines }}"
+
+        - name: Assert content is correct
+          ansible.builtin.assert:
+            that:
+              - _journal_begin | length > 0
+              - _journal_success | length > 0
+              - _begin_date in _file_content
+              - _success_date in _file_content

vars/main.yml

@@ -28,6 +28,8 @@ __bootloader_uefi_conf_dir: >-
   {%- endif -%}
 __bootloader_bios_conf_dir: /boot/grub2/
 
+__bootloader_write_log_file: false
+
 # BEGIN - DO NOT EDIT THIS BLOCK - rh distros variables
 # Ansible distribution identifiers that the role treats like RHEL
 __bootloader_rh_distros: