From b092e126a954b5396c3e2517c6bc0ffcccf6310e Mon Sep 17 00:00:00 2001 From: Slavi Pantaleev Date: Sun, 22 Mar 2026 06:45:47 +0200 Subject: [PATCH] Fix docker image build results not affecting conditional restart for ldap-registration-proxy and matrixto These roles had conditional restart logic (restart_necessary set_fact) but the docker_image build task result was not registered or included in the condition, so a changed image build would not trigger a service restart. Co-Authored-By: Claude Opus 4.6 (1M context) --- .../matrix-ldap-registration-proxy/tasks/setup_install.yml | 2 ++ roles/custom/matrix-matrixto/tasks/install.yml | 2 ++ 2 files changed, 4 insertions(+) diff --git a/roles/custom/matrix-ldap-registration-proxy/tasks/setup_install.yml b/roles/custom/matrix-ldap-registration-proxy/tasks/setup_install.yml index 4d8060669..ebf1212ff 100644 --- a/roles/custom/matrix-ldap-registration-proxy/tasks/setup_install.yml +++ b/roles/custom/matrix-ldap-registration-proxy/tasks/setup_install.yml @@ -40,6 +40,7 @@ path: "{{ matrix_ldap_registration_proxy_container_src_files_path }}" pull: true when: true + register: matrix_ldap_registration_proxy_container_image_build_result - name: Ensure matrix_ldap_registration_proxy config installed ansible.builtin.template: @@ -82,4 +83,5 @@ matrix_ldap_registration_proxy_config_result.changed | default(false) or matrix_ldap_registration_proxy_support_files_result.changed | default(false) or matrix_ldap_registration_proxy_systemd_service_result.changed | default(false) + or matrix_ldap_registration_proxy_container_image_build_result.changed | default(false) }} diff --git a/roles/custom/matrix-matrixto/tasks/install.yml b/roles/custom/matrix-matrixto/tasks/install.yml index 29232440e..feae4538e 100644 --- a/roles/custom/matrix-matrixto/tasks/install.yml +++ b/roles/custom/matrix-matrixto/tasks/install.yml @@ -45,6 +45,7 @@ path: "{{ matrix_matrixto_container_image_self_build_src_files_path }}" pull: true args: + register: matrix_matrixto_container_image_build_result - name: Ensure Matrix.to container network is created via community.docker.docker_network when: devture_systemd_docker_base_container_network_creation_method == 'ansible-module' @@ -79,4 +80,5 @@ {{ matrix_matrixto_support_files_result.changed | default(false) or matrix_matrixto_systemd_service_result.changed | default(false) + or matrix_matrixto_container_image_build_result.changed | default(false) }}