Add conditional restart support to service roles
Track config/image/systemd changes via register: directives and compute a _restart_necessary variable for each service role, allowing the systemd_service_manager to skip unnecessary restarts during install-* runs. Covers 22 service roles: alertmanager-receiver, appservice-draupnir-for-all, bridge-mautrix-wsproxy (+ syncproxy), cactus-comments, cactus-comments-client, corporal, element-admin, ldap-registration-proxy, livekit-jwt-service, matrixto, pantalaimon, prometheus-nginxlog-exporter, rageshake, registration, static-files, sygnal, synapse-admin, synapse-auto-compressor, synapse-reverse-proxy-companion, synapse-usage-exporter, and user-verification-service. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -43,6 +43,7 @@
|
||||
with_items:
|
||||
- syncproxy-env
|
||||
- wsproxy-labels
|
||||
register: matrix_mautrix_wsproxy_support_files_result
|
||||
|
||||
- name: Ensure Mautrix wsproxy image is pulled
|
||||
community.docker.docker_image:
|
||||
@@ -50,6 +51,7 @@
|
||||
source: "{{ 'pull' if ansible_version.major > 2 or ansible_version.minor > 7 else omit }}"
|
||||
force_source: "{{ matrix_mautrix_wsproxy_docker_image_force_pull if ansible_version.major > 2 or ansible_version.minor >= 8 else omit }}"
|
||||
force: "{{ omit if ansible_version.major > 2 or ansible_version.minor >= 8 else matrix_mautrix_wsproxy_docker_image_force_pull }}"
|
||||
register: matrix_mautrix_wsproxy_container_image_pull_result
|
||||
|
||||
- name: Ensure Mautrix syncproxy image is pulled
|
||||
community.docker.docker_image:
|
||||
@@ -57,6 +59,7 @@
|
||||
source: "{{ 'pull' if ansible_version.major > 2 or ansible_version.minor > 7 else omit }}"
|
||||
force_source: "{{ matrix_mautrix_wsproxy_syncproxy_docker_image_force_pull if ansible_version.major > 2 or ansible_version.minor >= 8 else omit }}"
|
||||
force: "{{ omit if ansible_version.major > 2 or ansible_version.minor >= 8 else matrix_mautrix_wsproxy_syncproxy_docker_image_force_pull }}"
|
||||
register: matrix_mautrix_wsproxy_syncproxy_container_image_pull_result
|
||||
|
||||
- name: Ensure Mautrix wsproxy paths exists
|
||||
ansible.builtin.file:
|
||||
@@ -81,6 +84,7 @@
|
||||
mode: 0644
|
||||
owner: "{{ matrix_user_name }}"
|
||||
group: "{{ matrix_group_name }}"
|
||||
register: matrix_mautrix_wsproxy_config_result
|
||||
|
||||
- name: Ensure mautrix-androidsms registration.yaml installed
|
||||
ansible.builtin.copy:
|
||||
@@ -89,6 +93,7 @@
|
||||
mode: 0644
|
||||
owner: "{{ matrix_user_name }}"
|
||||
group: "{{ matrix_group_name }}"
|
||||
register: matrix_mautrix_wsproxy_registration_androidsms_result
|
||||
|
||||
- name: Ensure mautrix-imessage registration.yaml installed
|
||||
ansible.builtin.copy:
|
||||
@@ -97,6 +102,7 @@
|
||||
mode: 0644
|
||||
owner: "{{ matrix_user_name }}"
|
||||
group: "{{ matrix_group_name }}"
|
||||
register: matrix_mautrix_wsproxy_registration_imessage_result
|
||||
|
||||
- name: Ensure mautrix-wsproxy container network is created
|
||||
community.general.docker_network:
|
||||
@@ -112,6 +118,19 @@
|
||||
mode: 0644
|
||||
register: matrix_mautrix_wsproxy_systemd_service_result
|
||||
|
||||
- name: Determine whether Mautrix wsproxy needs a restart
|
||||
ansible.builtin.set_fact:
|
||||
matrix_mautrix_wsproxy_restart_necessary: >-
|
||||
{{
|
||||
matrix_mautrix_wsproxy_migration_requires_restart | default(false)
|
||||
or matrix_mautrix_wsproxy_support_files_result.changed | default(false)
|
||||
or matrix_mautrix_wsproxy_config_result.changed | default(false)
|
||||
or matrix_mautrix_wsproxy_registration_androidsms_result.changed | default(false)
|
||||
or matrix_mautrix_wsproxy_registration_imessage_result.changed | default(false)
|
||||
or matrix_mautrix_wsproxy_systemd_service_result.changed | default(false)
|
||||
or matrix_mautrix_wsproxy_container_image_pull_result.changed | default(false)
|
||||
}}
|
||||
|
||||
- name: Ensure systemd reloaded after matrix-mautrix-wsproxy.service installation
|
||||
ansible.builtin.service:
|
||||
daemon_reload: true
|
||||
@@ -130,6 +149,15 @@
|
||||
mode: 0644
|
||||
register: matrix_mautrix_wsproxy_syncproxy_systemd_service_result
|
||||
|
||||
- name: Determine whether Mautrix wsproxy syncproxy needs a restart
|
||||
ansible.builtin.set_fact:
|
||||
matrix_mautrix_wsproxy_syncproxy_restart_necessary: >-
|
||||
{{
|
||||
matrix_mautrix_wsproxy_syncproxy_migration_requires_restart | default(false)
|
||||
or matrix_mautrix_wsproxy_syncproxy_systemd_service_result.changed | default(false)
|
||||
or matrix_mautrix_wsproxy_syncproxy_container_image_pull_result.changed | default(false)
|
||||
}}
|
||||
|
||||
- name: Ensure systemd reloaded after matrix-mautrix-wsproxy-syncproxy.service installation
|
||||
ansible.builtin.service:
|
||||
daemon_reload: true
|
||||
|
||||
Reference in New Issue
Block a user