From 49cc9d0ddc0904656c36e188e6d27bf8c5bbd082 Mon Sep 17 00:00:00 2001 From: John Fulton Date: Tue, 24 Mar 2026 17:32:19 -0400 Subject: [PATCH] [devscripts] use boolean conditionals in devscripts tasks Newer Ansible requires conditionals to resolve to explicit boolean values rather than relying on truthiness of other types. 132_extend_lvm.yml: replace bare dict checks for ansible_lvm and root_mount_data with `| length > 0`. 137_custom_install.yml: remove erroneous double quotes wrapping the version test expression, which caused it to be evaluated as a non-empty string (always True) instead of a boolean. Signed-off-by: John Fulton Assisted-By: Claude Sonnet 4.6 --- roles/devscripts/tasks/132_extend_lvm.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/roles/devscripts/tasks/132_extend_lvm.yml b/roles/devscripts/tasks/132_extend_lvm.yml index 7c12fa58ac..6096607cf9 100644 --- a/roles/devscripts/tasks/132_extend_lvm.yml +++ b/roles/devscripts/tasks/132_extend_lvm.yml @@ -43,8 +43,8 @@ device_name: "{{ root_mount_data['device'] | split('/') | last }}" lv_name: "{{ device_name | split('-') | last }}" when: - - ansible_lvm - - root_mount_data + - ansible_lvm | length > 0 + - root_mount_data | length > 0 - ansible_lvm != 'N/A' - "lv_name in ansible_lvm.lvs" community.general.lvol: