ceph-iscsi: notify rbd target services

When the iscsi gateway or the ceph configuration file change then we
need to notify the rbd target api/gw services to be restarted.
This patch also merges the rbd-target-api and rbd-target-gw handler
into a single file and listen.

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>

diff --git a/roles/ceph-config/tasks/main.yml b/roles/ceph-config/tasks/main.yml
index f6bbd7462fafa9260ab85ecc03e275a24a6e724f..a590b86d2ce2c81830ef81da2e0d78cfe03e1d66 100644 (file)
--- a/roles/ceph-config/tasks/main.yml
+++ b/roles/ceph-config/tasks/main.yml
@@ -91,6 +91,7 @@
       - restart ceph rgws
       - restart ceph mgrs
       - restart ceph rbdmirrors
+      - restart ceph rbd-target-api-gw
 
   - name: "ensure fetch directory exists"
     run_once: true
@@ -149,3 +150,4 @@
       - restart ceph rgws
       - restart ceph mgrs
       - restart ceph rbdmirrors
+      - restart ceph rbd-target-api-gw

diff --git a/roles/ceph-handler/handlers/main.yml b/roles/ceph-handler/handlers/main.yml
index 83809a294b6199b6d6d43da824af7834a0d7dab6..65b0ba9947591d20df4566c9edeae97a3f6090ad 100644 (file)
--- a/roles/ceph-handler/handlers/main.yml
+++ b/roles/ceph-handler/handlers/main.yml
@@ -49,12 +49,7 @@
       when: iscsi_gw_group_name in group_names
       listen: "restart ceph tcmu-runner"
 
-    - name: rbd-target-gw handler
-      include_tasks: handler_rbd_target_gw.yml
+    - name: rbd-target-api and rbd-target-gw handler
+      include_tasks: handler_rbd_target_api_gw.yml
       when: iscsi_gw_group_name in group_names
-      listen: "restart ceph rbd-target-gw"
-
-    - name: rbd-target-api handler
-      include_tasks: handler_rbd_target_api.yml
-      when: iscsi_gw_group_name in group_names
-      listen: "restart ceph rbd-target-api"
+      listen: "restart ceph rbd-target-api-gw"

diff --git a/roles/ceph-handler/tasks/handler_rbd_target_api.yml b/roles/ceph-handler/tasks/handler_rbd_target_api.yml
deleted file mode 100644 (file)
index 8843503..0000000
--- a/roles/ceph-handler/tasks/handler_rbd_target_api.yml
+++ /dev/null
@@ -1,26 +0,0 @@
----
-- name: set _rbd_target_api_handler_called before restart
-  set_fact:
-    _rbd_target_api_handler_called: True
-
-- name: copy rbd-target-api restart script
-  template:
-    src: restart_rbd_target_api.sh.j2
-    dest: /tmp/restart_rbd_target_api.sh
-    owner: root
-    group: root
-    mode: 0750
-
-- name: restart rbd-target-api
-  command: /usr/bin/env bash /tmp/restart_rbd_target_api.sh
-  when:
-    - ceph_rbd_target_api_stat.get('rc') == 0
-    - hostvars[item]['_rbd_target_api_handler_called'] | default(False) | bool
-    - ceph_rbd_target_api_stat.get('stdout_lines', [])|length != 0
-  with_items: "{{ groups[iscsi_gw_group_name] }}"
-  delegate_to: "{{ item }}"
-  run_once: True
-
-- name: set _rbd_target_api_handler_called after restart
-  set_fact:
-    _rbd_target_api_handler_called: False

diff --git a/roles/ceph-handler/tasks/handler_rbd_target_api_gw.yml b/roles/ceph-handler/tasks/handler_rbd_target_api_gw.yml
new file mode 100644 (file)
index 0000000..c8bfb12
--- /dev/null
+++ b/roles/ceph-handler/tasks/handler_rbd_target_api_gw.yml
@@ -0,0 +1,52 @@
+---
+- name: set _rbd_target_api_handler_called before restart
+  set_fact:
+    _rbd_target_api_handler_called: True
+
+- name: copy rbd-target-api restart script
+  template:
+    src: restart_rbd_target_api.sh.j2
+    dest: /tmp/restart_rbd_target_api.sh
+    owner: root
+    group: root
+    mode: 0750
+
+- name: restart rbd-target-api
+  command: /usr/bin/env bash /tmp/restart_rbd_target_api.sh
+  when:
+    - ceph_rbd_target_api_stat.get('rc') == 0
+    - hostvars[item]['_rbd_target_api_handler_called'] | default(False) | bool
+    - ceph_rbd_target_api_stat.get('stdout_lines', [])|length != 0
+  with_items: "{{ groups[iscsi_gw_group_name] }}"
+  delegate_to: "{{ item }}"
+  run_once: True
+
+- name: set _rbd_target_api_handler_called after restart
+  set_fact:
+    _rbd_target_api_handler_called: False
+
+- name: set _rbd_target_gw_handler_called before restart
+  set_fact:
+    _rbd_target_gw_handler_called: True
+
+- name: copy rbd-target-gw restart script
+  template:
+    src: restart_rbd_target_gw.sh.j2
+    dest: /tmp/restart_rbd_target_gw.sh
+    owner: root
+    group: root
+    mode: 0750
+
+- name: restart rbd-target-gw
+  command: /usr/bin/env bash /tmp/restart_rbd_target_gw.sh
+  when:
+    - ceph_rbd_target_gw_stat.get('rc') == 0
+    - hostvars[item]['_rbd_target_gw_handler_called'] | default(False) | bool
+    - ceph_rbd_target_gw_stat.get('stdout_lines', [])|length != 0
+  with_items: "{{ groups[iscsi_gw_group_name] }}"
+  delegate_to: "{{ item }}"
+  run_once: True
+
+- name: set _rbd_target_gw_handler_called after restart
+  set_fact:
+    _rbd_target_gw_handler_called: False

diff --git a/roles/ceph-handler/tasks/handler_rbd_target_gw.yml b/roles/ceph-handler/tasks/handler_rbd_target_gw.yml
deleted file mode 100644 (file)
index 17cbb09..0000000
--- a/roles/ceph-handler/tasks/handler_rbd_target_gw.yml
+++ /dev/null
@@ -1,26 +0,0 @@
----
-- name: set _rbd_target_gw_handler_called before restart
-  set_fact:
-    _rbd_target_gw_handler_called: True
-
-- name: copy rbd-target-gw restart script
-  template:
-    src: restart_rbd_target_gw.sh.j2
-    dest: /tmp/restart_rbd_target_gw.sh
-    owner: root
-    group: root
-    mode: 0750
-
-- name: restart rbd-target-gw
-  command: /usr/bin/env bash /tmp/restart_rbd_target_gw.sh
-  when:
-    - ceph_rbd_target_gw_stat.get('rc') == 0
-    - hostvars[item]['_rbd_target_gw_handler_called'] | default(False) | bool
-    - ceph_rbd_target_gw_stat.get('stdout_lines', [])|length != 0
-  with_items: "{{ groups[iscsi_gw_group_name] }}"
-  delegate_to: "{{ item }}"
-  run_once: True
-
-- name: set _rbd_target_gw_handler_called after restart
-  set_fact:
-    _rbd_target_gw_handler_called: False

diff --git a/roles/ceph-iscsi-gw/tasks/common.yml b/roles/ceph-iscsi-gw/tasks/common.yml
index c7e8d8cdf8dd24dadc2bd36a3828633247e99da4..93c8ba9b02bd5bc88b5faf951cfedce8ddd2833c 100644 (file)
--- a/roles/ceph-iscsi-gw/tasks/common.yml
+++ b/roles/ceph-iscsi-gw/tasks/common.yml
@@ -28,6 +28,7 @@
     dest: /etc/ceph/iscsi-gateway.cfg
     config_type: ini
     config_overrides: '{{ iscsi_conf_overrides }}'
+  notify: restart ceph rbd-target-api-gw
 
 - name: set_fact container_exec_cmd
   set_fact:

diff --git a/roles/ceph-iscsi-gw/tasks/systemd.yml b/roles/ceph-iscsi-gw/tasks/systemd.yml
index 6768716b059f5b1c8f8d32c19bf1032d2240c549..359342260eef4d6a2a0b865decb5bd6bc0b11ae4 100644 (file)
--- a/roles/ceph-iscsi-gw/tasks/systemd.yml
+++ b/roles/ceph-iscsi-gw/tasks/systemd.yml
@@ -11,4 +11,6 @@
     - tcmu-runner
     - rbd-target-gw
     - rbd-target-api
-  notify: restart ceph {{ item }}
+  notify:
+    - restart ceph tcmu-runner
+    - restart ceph rbd-target-api-gw