ceph-iscsi: Update gateway config/template

- Remove gateway_keyring from the configuration file because it's
not used in ceph-iscsi 3.x release.
- Use config_template instead of template module for iscsi-gateway
configuration file. Because the file is an ini file and we might want
to override more parameters than those present in ceph-ansible.
- Because we can now set the pool name in the configuration, we should
use a variable for that. This is refact with the iscsi_pool_* variables
also used to configure the pool size.

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
(cherry picked from commit 1f2a4f1910d75241deb345228227a3ad7711c145)

diff --git a/group_vars/iscsigws.yml.sample b/group_vars/iscsigws.yml.sample
index 6314ff52d28df6a5d8394682c6c658860911c938..3f2814d27904131156ff45a055d566c4266255a1 100644 (file)
--- a/group_vars/iscsigws.yml.sample
+++ b/group_vars/iscsigws.yml.sample
@@ -15,7 +15,9 @@ dummy:
 # Whether or not to generate secure certificate to iSCSI gateway nodes
 #generate_crt: False
 
-#rbd_pool_size: "{{ osd_pool_default_size }}"
+#iscsi_conf_overrides: {}
+#iscsi_pool_name: rbd
+#iscsi_pool_size: "{{ osd_pool_default_size }}"
 
 ##################
 # RBD-TARGET-API #

diff --git a/roles/ceph-iscsi-gw/defaults/main.yml b/roles/ceph-iscsi-gw/defaults/main.yml
index 4c7ff98c835bef986616840c5e50a207f1b035e5..3d543248335b9e4889fd2417f2370026cf9e8ac3 100644 (file)
--- a/roles/ceph-iscsi-gw/defaults/main.yml
+++ b/roles/ceph-iscsi-gw/defaults/main.yml
@@ -7,7 +7,9 @@
 # Whether or not to generate secure certificate to iSCSI gateway nodes
 generate_crt: False
 
-rbd_pool_size: "{{ osd_pool_default_size }}"
+iscsi_conf_overrides: {}
+iscsi_pool_name: rbd
+iscsi_pool_size: "{{ osd_pool_default_size }}"
 
 ##################
 # RBD-TARGET-API #

diff --git a/roles/ceph-iscsi-gw/tasks/common.yml b/roles/ceph-iscsi-gw/tasks/common.yml
index 111ec9ea547ea9ec8277b90273ed32b110e14e43..c21ccb66b7a6d5812bb38d6c6619b91bcd1e2215 100644 (file)
--- a/roles/ceph-iscsi-gw/tasks/common.yml
+++ b/roles/ceph-iscsi-gw/tasks/common.yml
@@ -9,9 +9,11 @@
   when: cephx | bool
 
 - name: deploy gateway settings, used by the ceph_iscsi_config modules
-  template:
+  config_template:
     src: "{{ role_path }}/templates/iscsi-gateway.cfg.j2"
     dest: /etc/ceph/iscsi-gateway.cfg
+    config_type: ini
+    config_overrides: '{{ iscsi_conf_overrides }}'
 
 - name: set_fact container_exec_cmd
   set_fact:
@@ -22,24 +24,24 @@
 - name: check if a rbd pool exists
   command: "{{ container_exec_cmd | default('') }} ceph --cluster {{ cluster }} osd pool ls --format json"
   changed_when: false
-  register: rbd_pool_exists
+  register: iscsi_pool_exists
   delegate_to: "{{ groups[mon_group_name][0] }}"
 
-- name: rbd pool related tasks
-  when: "'rbd' not in (rbd_pool_exists.stdout | from_json)"
+- name: iscsi pool related tasks
+  when: "iscsi_pool_name not in (iscsi_pool_exists.stdout | from_json)"
   block:
-    - name: create a rbd pool if it doesn't exist
-      command: "{{ container_exec_cmd | default('') }} ceph --cluster {{ cluster }} osd pool create rbd {{ osd_pool_default_pg_num }}"
+    - name: create a iscsi pool if it doesn't exist
+      command: "{{ container_exec_cmd | default('') }} ceph --cluster {{ cluster }} osd pool create {{ iscsi_pool_name }} {{ osd_pool_default_pg_num }}"
       changed_when: false
       delegate_to: "{{ groups[mon_group_name][0] }}"
 
-    - name: assign application rbd to pool 'rbd'
-      command: "{{ container_exec_cmd }} ceph --connect-timeout 5 --cluster {{ cluster }} osd pool application enable rbd rbd"
+    - name: assign application rbd to iscsi pool
+      command: "{{ container_exec_cmd }} ceph --connect-timeout 5 --cluster {{ cluster }} osd pool application enable {{ iscsi_pool_name }} rbd"
       delegate_to: "{{ groups[mon_group_name][0] }}"
       run_once: True
 
     - name: customize pool size
-      command: "{{ container_exec_cmd | default('') }} ceph --cluster {{ cluster }} osd pool set rbd size {{ rbd_pool_size | default(osd_pool_default_size) }}"
+      command: "{{ container_exec_cmd | default('') }} ceph --cluster {{ cluster }} osd pool set {{ iscsi_pool_name }} size {{ iscsi_pool_size | default(osd_pool_default_size) }}"
       delegate_to: "{{ groups[mon_group_name][0] }}"
       changed_when: false
-      when: rbd_pool_size | default(osd_pool_default_size) != ceph_osd_pool_default_size
+      when: iscsi_pool_size | default(osd_pool_default_size) != ceph_osd_pool_default_size

diff --git a/roles/ceph-iscsi-gw/templates/iscsi-gateway.cfg.j2 b/roles/ceph-iscsi-gw/templates/iscsi-gateway.cfg.j2
index eae297603b0026b1b62af81f70df2c70e35fd5d7..25f530614078d3520e52a2db6e121cc3c7adc066 100644 (file)
--- a/roles/ceph-iscsi-gw/templates/iscsi-gateway.cfg.j2
+++ b/roles/ceph-iscsi-gw/templates/iscsi-gateway.cfg.j2
@@ -5,7 +5,6 @@
 
 [config]
 cluster_name = {{ cluster }}
-gateway_keyring = {{ cluster }}.client.admin.keyring
 
 # API settings.
 # The API supports a number of options that allow you to tailor it to your