moved string parsing of ceph version into ansbile for better error handling

diff --git a/roles/ceph-common/tasks/main.yml b/roles/ceph-common/tasks/main.yml
index 02d3339209ca177d39a4c2c32cd01616fcfedfa7..8048bddb9cd1406aa569341a5bc778bc07119067 100644 (file)
--- a/roles/ceph-common/tasks/main.yml
+++ b/roles/ceph-common/tasks/main.yml
@@ -65,15 +65,18 @@
     - package-install
 
 - name: get ceph version
-  shell: ceph --version | cut -f3 -d ' '
+  command: ceph --version
   changed_when: false
   register: ceph_version
 
+- set_fact:
+    ceph_version: "{{ ceph_version.stdout.split(' ')[2] }}"
+
 # NOTE (leseb/jsaintrocc): Your supposed to qoute variables
 # that follow colons to avoid confusion with dicts but this
 # causes issues with the boolean, so we keep this syntax styling...
 - set_fact:
-    is_ceph_infernalis={{ ceph_version.stdout | version_compare('9.2.0', '>=') }}
+    is_ceph_infernalis={{ ceph_version | version_compare('9.2.0', '>=') }}
 
 - set_fact:
     dir_owner: ceph