common: set journal size to 5GB

Journal size is not mandatory anymore, a default from 5GB is being
added. A simple warning message will show up if the size is set to
something below 5GB.

Signed-off-by: Sébastien Han <seb@redhat.com>

diff --git a/group_vars/all.sample b/group_vars/all.sample
index bdf8ba61fff1572c3e496c5e72e3640c2a69d1cc..d197e40122be4b0878abf600dd6cc00fe7038f1c 100644 (file)
--- a/group_vars/all.sample
+++ b/group_vars/all.sample
@@ -258,7 +258,7 @@ dummy:
 
 ## OSD options
 #
-#journal_size: 0 # OSD journal size in MB
+#journal_size: 5120 # OSD journal size in MB
 #public_network: 0.0.0.0/0
 #cluster_network: "{{ public_network }}"
 #osd_mkfs_type: xfs

diff --git a/roles/ceph-common/defaults/main.yml b/roles/ceph-common/defaults/main.yml
index a70fbcab122faf1b28cb867ac7b644d30785139d..c40eb20645f232a9e283e4950625c9695af73baa 100644 (file)
--- a/roles/ceph-common/defaults/main.yml
+++ b/roles/ceph-common/defaults/main.yml
@@ -250,7 +250,7 @@ mon_use_fqdn: false # if set to true, the MON name used will be the fqdn in the
 
 ## OSD options
 #
-journal_size: 0 # OSD journal size in MB
+journal_size: 5120 # OSD journal size in MB
 public_network: 0.0.0.0/0
 cluster_network: "{{ public_network }}"
 osd_mkfs_type: xfs

diff --git a/roles/ceph-common/tasks/checks/check_mandatory_vars.yml b/roles/ceph-common/tasks/checks/check_mandatory_vars.yml
index 2cf5ba39ddba4c2c6cfa086bbd93ee6474f10cbb..2a84c9ff869e4b47a5d2b223f83e94ac12119fcb 100644 (file)
--- a/roles/ceph-common/tasks/checks/check_mandatory_vars.yml
+++ b/roles/ceph-common/tasks/checks/check_mandatory_vars.yml
@@ -33,10 +33,10 @@
     - package-install
 
 - name: make sure journal_size configured
-  fail:
-    msg: "journal_size must be configured. See http://ceph.com/docs/master/rados/configuration/osd-config-ref/"
+  debug:
+    msg: "WARNING: journal_size is configured to less than 5GB. This is not recommended and can lead to severe issues."
   when:
-    - journal_size|int == 0
+    - journal_size|int < 5120
     - osd_objectstore != 'bluestore'
     - osd_group_name in group_names