From: Mike Christie <mchristi@redhat.com>
Date: Thu, 30 May 2019 16:17:09 +0000 (-0500)
Subject: igw: Add check for missing iqn
X-Git-Tag: v4.0.0rc10~16
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=ce62ac7bebd1709bc899ef778fad18231e65e335;p=ceph-ansible.git

igw: Add check for missing iqn

If the user is still using the older packages and does not setup
the target iqn you will just get a vague error message later on.
This adds a check during the validate task, so it is clear to the
user.

Signed-off-by: Mike Christie <mchristi@redhat.com>
(cherry picked from commit 08a6d10c32beb5c2736b36487efb1860e868d74e)
---

diff --git a/roles/ceph-validate/tasks/check_iscsi.yml b/roles/ceph-validate/tasks/check_iscsi.yml
index b067b6259..6233a3d34 100644
--- a/roles/ceph-validate/tasks/check_iscsi.yml
+++ b/roles/ceph-validate/tasks/check_iscsi.yml
@@ -11,6 +11,14 @@
     - not containerized_deployment | bool
     - not use_new_ceph_iscsi | bool
 
+- name: make sure gateway_iqn is configured
+  fail:
+    msg: "you must set a iqn for the iSCSI target"
+  when:
+    - "gateway_iqn | default('') | length == 0"
+    - not containerized_deployment | bool
+    - not use_new_ceph_iscsi | bool
+
 - name: fail if unsupported chap configuration
   fail:
     msg: "Mixing clients with CHAP enabled and disabled is not supported."