mirror of
https://github.com/spantaleev/matrix-docker-ansible-deploy.git
synced 2026-09-24 16:50:08 +00:00
Slavi Pantaleev
e6e19a20b4
The ownership repair added in e5b8de8c2 is not sufficient on every Ansible privilege-escalation setup. Git may still see a different effective owner after the checkout has been recursively chowned, as demonstrated by #5065.
Pass an exact, task-scoped safe.directory setting to all 56 server-side git tasks. Keep the ownership repair as well, because it remains necessary for filesystem permissions. The two controller-side theme checkouts already use the same protection.
101 lines
4.2 KiB
YAML
101 lines
4.2 KiB
YAML
# SPDX-FileCopyrightText: 2023 - 2025 Slavi Pantaleev
|
|
# SPDX-FileCopyrightText: 2025 Suguru Hirahara
|
|
#
|
|
# SPDX-License-Identifier: AGPL-3.0-or-later
|
|
|
|
---
|
|
- name: Ensure Matrix.to path exists
|
|
ansible.builtin.file:
|
|
path: "{{ item }}"
|
|
state: directory
|
|
mode: "0750"
|
|
owner: "{{ matrix_user_name }}"
|
|
group: "{{ matrix_group_name }}"
|
|
with_items:
|
|
- "{{ matrix_matrixto_base_path }}"
|
|
|
|
- name: Ensure Matrix.to support files installed
|
|
ansible.builtin.template:
|
|
src: "{{ role_path }}/templates/{{ item }}.j2"
|
|
dest: "{{ matrix_matrixto_base_path }}/{{ item }}"
|
|
mode: "0640"
|
|
owner: "{{ matrix_user_name }}"
|
|
group: "{{ matrix_group_name }}"
|
|
with_items:
|
|
- env
|
|
- labels
|
|
register: matrix_matrixto_support_files_result
|
|
|
|
# A checkout owned by a different user (a uid change, an earlier clone by another user, etc.) would make the git task below fail on ownership or permissions.
|
|
- name: Ensure Matrix.to repository ownership is correct on self-build
|
|
ansible.builtin.file:
|
|
path: "{{ matrix_matrixto_container_image_self_build_src_files_path }}"
|
|
state: directory
|
|
owner: "{{ matrix_user_name }}"
|
|
group: "{{ matrix_group_name }}"
|
|
recurse: true
|
|
|
|
- name: Ensure Matrix.to repository is present on self-build
|
|
ansible.builtin.git:
|
|
repo: "{{ matrix_matrixto_container_image_self_build_repo }}"
|
|
version: "{{ matrix_matrixto_container_image_self_build_repo_version }}"
|
|
dest: "{{ matrix_matrixto_container_image_self_build_src_files_path }}"
|
|
force: "yes"
|
|
become: true
|
|
become_user: "{{ matrix_user_name }}"
|
|
# Keep this even though the task above normalizes ownership. Git may still see an ownership mismatch when Ansible becomes an unprivileged user (see #5065).
|
|
environment:
|
|
GIT_CONFIG_COUNT: "1"
|
|
GIT_CONFIG_KEY_0: safe.directory
|
|
GIT_CONFIG_VALUE_0: "{{ matrix_matrixto_container_image_self_build_src_files_path }}"
|
|
register: matrix_matrixto_git_pull_results
|
|
|
|
- name: Ensure Matrix.to container image is built
|
|
community.docker.docker_image_build:
|
|
name: "{{ matrix_matrixto_container_image_self_build_name }}"
|
|
dockerfile: Dockerfile
|
|
path: "{{ matrix_matrixto_container_image_self_build_src_files_path }}"
|
|
pull: true
|
|
rebuild: "{{ 'always' if matrix_matrixto_git_pull_results.changed | bool else 'never' }}"
|
|
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'
|
|
- matrix_matrixto_container_network != 'host'
|
|
community.docker.docker_network:
|
|
enable_ipv6: "{{ devture_systemd_docker_base_ipv6_enabled }}"
|
|
name: "{{ matrix_matrixto_container_network }}"
|
|
driver: bridge
|
|
driver_options: "{{ devture_systemd_docker_base_container_networks_driver_options }}"
|
|
|
|
- name: Ensure Matrix.to container network is created via ansible.builtin.command
|
|
when:
|
|
- devture_systemd_docker_base_container_network_creation_method == 'command'
|
|
- matrix_matrixto_container_network != 'host'
|
|
ansible.builtin.command:
|
|
cmd: >-
|
|
{{ devture_systemd_docker_base_host_command_docker }} network create
|
|
{% if devture_systemd_docker_base_ipv6_enabled %}--ipv6{% endif %}
|
|
{{ devture_systemd_docker_base_container_networks_driver_options_string }}
|
|
{{ matrix_matrixto_container_network }}
|
|
register: network_creation_result
|
|
changed_when: network_creation_result.rc == 0
|
|
failed_when: network_creation_result.rc != 0 and 'already exists' not in network_creation_result.stderr
|
|
|
|
- name: Ensure Matrix.to systemd service is present
|
|
ansible.builtin.template:
|
|
src: "{{ role_path }}/templates/systemd/matrix-matrixto.service.j2"
|
|
dest: "{{ devture_systemd_docker_base_systemd_path }}/{{ matrix_matrixto_identifier }}.service"
|
|
mode: "0644"
|
|
register: matrix_matrixto_systemd_service_result
|
|
|
|
- name: Determine whether Matrix.to needs a restart
|
|
ansible.builtin.set_fact:
|
|
matrix_matrixto_restart_necessary: >-
|
|
{{
|
|
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)
|
|
}}
|