Add option to enable ntp

This fixes: #845 only for non-containerized deployments

Signed-off-by: Ivan Font <ivan.font@redhat.com>

diff --git a/group_vars/all.sample b/group_vars/all.sample
index 963b4a1283c7825d6bd3da632fb06cfd5aa92b45..9820cf37ff8145e88ca307190f1ed19ccfa07d50 100644 (file)
--- a/group_vars/all.sample
+++ b/group_vars/all.sample
@@ -64,6 +64,9 @@ dummy:
 #  - ntp
 #  - python-setuptools
 
+# Enable the ntp service by default to avoid clock skew on
+# ceph nodes
+#ntp_service_enabled: true
 
 # The list of ceph packages needed for debian.
 # This variable should only be changed if packages are not available from a given

diff --git a/roles/ceph-common/defaults/main.yml b/roles/ceph-common/defaults/main.yml
index a8d135437334b3f67cd9e9564ecf1132546a526c..67a66a5f9b224a1f97fca00a55a12ca047c2f416 100644 (file)
--- a/roles/ceph-common/defaults/main.yml
+++ b/roles/ceph-common/defaults/main.yml
@@ -56,6 +56,9 @@ redhat_package_dependencies:
   - ntp
   - python-setuptools
 
+# Enable the ntp service by default to avoid clock skew on
+# ceph nodes
+ntp_service_enabled: true
 
 # The list of ceph packages needed for debian.
 # This variable should only be changed if packages are not available from a given

diff --git a/roles/ceph-common/tasks/checks/check_ntp_debian.yml b/roles/ceph-common/tasks/checks/check_ntp_debian.yml
new file mode 100644 (file)
index 0000000..21d1216
--- /dev/null
+++ b/roles/ceph-common/tasks/checks/check_ntp_debian.yml
@@ -0,0 +1,7 @@
+---
+- name: check ntp installation on debian
+  command: dpkg -s ntp
+  register: ntp_pkg_query
+  ignore_errors: true
+  changed_when: false
+  when: ansible_os_family == 'Debian'

diff --git a/roles/ceph-common/tasks/checks/check_ntp_redhat.yml b/roles/ceph-common/tasks/checks/check_ntp_redhat.yml
new file mode 100644 (file)
index 0000000..b55a0d3
--- /dev/null
+++ b/roles/ceph-common/tasks/checks/check_ntp_redhat.yml
@@ -0,0 +1,7 @@
+---
+- name: check ntp installation on redhat
+  command: rpm -q ntp
+  register: ntp_pkg_query
+  ignore_errors: true
+  changed_when: false
+  when: ansible_os_family == 'RedHat'

diff --git a/roles/ceph-common/tasks/main.yml b/roles/ceph-common/tasks/main.yml
index 0df9ea79cd89007424f9dafcd16dae86d870f4e9..587ca48922de8ea60e1d79ceeea580eb1a3de7b3 100644 (file)
--- a/roles/ceph-common/tasks/main.yml
+++ b/roles/ceph-common/tasks/main.yml
@@ -71,6 +71,16 @@
   tags:
     - package-install
 
+- include: ./misc/ntp_redhat.yml
+  when:
+    - ansible_os_family == 'RedHat'
+    - ntp_service_enabled
+
+- include: ./misc/ntp_debian.yml
+  when:
+    - ansible_os_family == 'Debian'
+    - ntp_service_enabled
+
 - include: facts.yml
 
 - set_fact:

diff --git a/roles/ceph-common/tasks/misc/ntp_debian.yml b/roles/ceph-common/tasks/misc/ntp_debian.yml
new file mode 100644 (file)
index 0000000..62db346
--- /dev/null
+++ b/roles/ceph-common/tasks/misc/ntp_debian.yml
@@ -0,0 +1,11 @@
+---
+- include: ../checks/check_ntp_debian.yml
+  when: ansible_os_family == 'Debian'
+
+- name: start the ntp service
+  service:
+    name: ntp
+    enabled: yes
+    state: started
+  when:
+    - ntp_pkg_query.rc == 0

diff --git a/roles/ceph-common/tasks/misc/ntp_redhat.yml b/roles/ceph-common/tasks/misc/ntp_redhat.yml
new file mode 100644 (file)
index 0000000..b64b9d8
--- /dev/null
+++ b/roles/ceph-common/tasks/misc/ntp_redhat.yml
@@ -0,0 +1,11 @@
+---
+- include: ../checks/check_ntp_redhat.yml
+  when: ansible_os_family == 'RedHat'
+
+- name: start the ntp service
+  service:
+    name: ntpd
+    enabled: yes
+    state: started
+  when:
+    - ntp_pkg_query.rc == 0