From b9437ed68a81b6f907ab0dab5fa0784dd6ae3c12 Mon Sep 17 00:00:00 2001 From: Michael Schiemer Date: Sun, 9 Nov 2025 00:09:51 +0100 Subject: [PATCH] fix: use immutable list operations in Jinja2 template Ansible doesn't allow .append() on lists in Jinja2 templates. Changed to use select() filter with regex match instead of loop with append. This is the same filtering logic but using immutable operations. --- deployment/ansible/playbooks/deploy-image.yml | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/deployment/ansible/playbooks/deploy-image.yml b/deployment/ansible/playbooks/deploy-image.yml index 1e5ff613..9755eafd 100644 --- a/deployment/ansible/playbooks/deploy-image.yml +++ b/deployment/ansible/playbooks/deploy-image.yml @@ -248,12 +248,7 @@ ansible.builtin.set_fact: registries_to_login: >- {%- set found_registries = actual_registry_urls_full.stdout | trim | split('\n') | select('match', '.+') | list -%} - {%- set filtered_registries = [] -%} - {%- for reg in found_registries -%} - {%- if reg | regex_search('\.(de|com|org|net|io|dev)') or reg | regex_search(':[0-9]+') or reg == 'localhost' -%} - {%- set _ = filtered_registries.append(reg) -%} - {%- endif -%} - {%- endfor -%} + {%- set filtered_registries = found_registries | select('match', '.*\.(de|com|org|net|io|dev).*|.*:[0-9]+.*|^localhost$') | list -%} {%- set default_registry = [docker_registry] -%} {%- if filtered_registries | length > 0 -%} {{ filtered_registries | unique | list }}