diff options
| author | Egor Tensin <Egor.Tensin@gmail.com> | 2023-09-22 11:04:34 +0200 |
|---|---|---|
| committer | Egor Tensin <Egor.Tensin@gmail.com> | 2023-09-22 11:19:06 +0200 |
| commit | 33847d19fb2a76e91486e63b473afb4aa13de4cb (patch) | |
| tree | ef126214e62889c253086b7eaa5af218a26c7ea3 /roles/docker/tasks/logging.yml | |
| parent | docker: refactoring (diff) | |
| download | infra-ansible-33847d19fb2a76e91486e63b473afb4aa13de4cb.tar.gz infra-ansible-33847d19fb2a76e91486e63b473afb4aa13de4cb.zip | |
docker: avoid duplicate service restarts
Diffstat (limited to 'roles/docker/tasks/logging.yml')
| -rw-r--r-- | roles/docker/tasks/logging.yml | 38 |
1 files changed, 31 insertions, 7 deletions
diff --git a/roles/docker/tasks/logging.yml b/roles/docker/tasks/logging.yml index b40da18..f93efa0 100644 --- a/roles/docker/tasks/logging.yml +++ b/roles/docker/tasks/logging.yml @@ -16,10 +16,34 @@ - name: Modify Docker config become: true - ansible.builtin.copy: - dest: /etc/docker/daemon.json - content: '{{ docker_current_config | combine({"log-driver": docker_logging_driver}) | to_nice_json }}' - owner: root - group: root - mode: '644' - notify: docker_restart + block: + - name: Create /etc/docker + ansible.builtin.file: + path: /etc/docker + state: directory + owner: root + group: root + mode: '755' + + - name: Write daemon.json + ansible.builtin.copy: + dest: /etc/docker/daemon.json + content: '{{ docker_current_config | combine({"log-driver": docker_logging_driver}) | to_nice_json }}' + owner: root + group: root + mode: '644' + register: docker_config + +- name: Restart Docker if necessary + become: true + when: docker_config.changed + block: + - name: Collect service facts + ansible.builtin.service_facts: + + - name: Restart Docker service + when: | + 'docker' in ansible_facts.services and ansible_facts.services.docker.state == 'running' + ansible.builtin.systemd_service: + name: docker + state: restarted |