ceph-ansible/roles/ceph-docker-common/tasks/misc/ntp_atomic.yml

---
- name: include ../checks/check_ntp_atomic.yml
  include: ../checks/check_ntp_atomic.yml
  when: is_atomic

- name: start the ntp service
  service:
    name: chronyd
    enabled: yes
    state: started
  when:
    - ntp_pkg_query.rc == 0
Refact code `ceph-docker-common`: At the moment there is a lot of duplicated tasks in each `./roles/ceph-<role>/tasks/docker/main.yml` that could be refactored in `./roles/ceph-docker-common/tasks/main.yml`. `_containerized_deployment` variables: All `_containerized_deployment` have been refactored to a single variable `containerized_deployment` duplicate `cephx` variables in `group_vars/* have been removed. Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com> 2017-04-13 01:57:33 +08:00			`---`
name includes and set_fact for clarity When Ansible is not run with verbose options it's difficult to see which include and/or set_fact does what. So adding a name for each clarifies. Signed-off-by: Sébastien Han <seb@redhat.com> 2017-09-15 06:48:53 +08:00			`- name: include ../checks/check_ntp_atomic.yml`
			`include: ../checks/check_ntp_atomic.yml`
Refact code `ceph-docker-common`: At the moment there is a lot of duplicated tasks in each `./roles/ceph-<role>/tasks/docker/main.yml` that could be refactored in `./roles/ceph-docker-common/tasks/main.yml`. `_containerized_deployment` variables: All `_containerized_deployment` have been refactored to a single variable `containerized_deployment` duplicate `cephx` variables in `group_vars/* have been removed. Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com> 2017-04-13 01:57:33 +08:00			`when: is_atomic`

			`- name: start the ntp service`
			`service:`
			`name: chronyd`
			`enabled: yes`
			`state: started`
			`when:`
			`- ntp_pkg_query.rc == 0`