From e530ae016bb3c0e38006adf2016a7f286429ed66 Mon Sep 17 00:00:00 2001 From: Semaphore Date: Wed, 11 Mar 2026 21:05:17 -0700 Subject: [PATCH] =?UTF-8?q?Fix=20linux=5Fpatch=20=E2=80=94=20normalize=20u?= =?UTF-8?q?pgradable=5Fpackages=5Flist=20to=20avoid=20stdout=5Flines=20err?= =?UTF-8?q?or=20on=20skipped=20OS=20tasks?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- roles/linux_patch/tasks/main.yml | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/roles/linux_patch/tasks/main.yml b/roles/linux_patch/tasks/main.yml index 4125864..8b28be8 100644 --- a/roles/linux_patch/tasks/main.yml +++ b/roles/linux_patch/tasks/main.yml @@ -42,14 +42,19 @@ changed_when: false when: ansible_os_family == "Alpine" -- name: Normalize upgradable count (Alpine) +- name: Normalize upgradable packages list ansible.builtin.set_fact: - upgradable_count: "{{ apk_upgradable.stdout_lines | default([]) | length }}" - when: os_family == 'alpine' - + upgradable_packages_list: "{{ upgradable_packages.stdout_lines | default([]) }}" + when: upgradable_packages is defined and upgradable_packages.stdout_lines is defined + +- name: Normalize upgradable packages list (skipped/undefined fallback) + ansible.builtin.set_fact: + upgradable_packages_list: [] + when: upgradable_packages is not defined or upgradable_packages.stdout_lines is not defined + - name: Log packages to be updated ansible.builtin.debug: - msg: "Packages to be updated on {{ inventory_hostname }}: {{ upgradable_count | default(0) }} packages" + msg: "Packages to be updated on {{ inventory_hostname }}: {{ upgradable_packages_list | length }} packages" - name: Perform full upgrade (Debian/Ubuntu) ansible.builtin.apt: