ceph-handler: remove iscsigws restart scripts

The iscsigws restart scripts for tcmu-runner and rbd-target-{api,gw}
services only call the systemctl restart command.
We don't really need to copy a shell script to do it when we can use
the ansible service module instead.

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

diff --git a/roles/ceph-handler/handlers/main.yml b/roles/ceph-handler/handlers/main.yml
index ede442c91c9334c12d610a19f11e80e541a1de02..463a272200736e4591879f2eb5f796c0dd811fb2 100644 (file)
--- a/roles/ceph-handler/handlers/main.yml
+++ b/roles/ceph-handler/handlers/main.yml
@@ -16,8 +16,6 @@
         - "restart ceph nfss"
         - "restart ceph rbdmirrors"
         - "restart ceph mgrs"
-        - "restart ceph tcmu-runner"
-        - "restart ceph rbd-target-api-gw"
       register: tmpdirpath
       when: tmpdirpath is not defined or tmpdirpath.path is not defined or tmpdirpath.state=="absent"
 
@@ -89,7 +87,5 @@
         - "restart ceph nfss"
         - "restart ceph rbdmirrors"
         - "restart ceph mgrs"
-        - "restart ceph tcmu-runner"
-        - "restart ceph rbd-target-api-gw"
       register: tmpdirpath
       when: tmpdirpath is defined

diff --git a/roles/ceph-handler/tasks/handler_rbd_target_api_gw.yml b/roles/ceph-handler/tasks/handler_rbd_target_api_gw.yml
index f792c14586aa5ddf9eb3d89f96ac90dd0dc915d5..903d333f706ef0773e54009116726bb66228be3b 100644 (file)
--- a/roles/ceph-handler/tasks/handler_rbd_target_api_gw.yml
+++ b/roles/ceph-handler/tasks/handler_rbd_target_api_gw.yml
@@ -3,16 +3,10 @@
   set_fact:
     _rbd_target_api_handler_called: True
 
-- name: copy rbd-target-api restart script
-  template:
-    src: restart_rbd_target_api.sh.j2
-    dest: "{{ tmpdirpath.path }}/restart_rbd_target_api.sh"
-    owner: root
-    group: root
-    mode: 0750
-
 - name: restart rbd-target-api
-  command: /usr/bin/env bash {{ hostvars[item]['tmpdirpath']['path'] }}/restart_rbd_target_api.sh
+  service:
+    name: rbd-target-api
+    state: restarted
   when:
     - ceph_rbd_target_api_stat.get('rc') == 0
     - hostvars[item]['_rbd_target_api_handler_called'] | default(False) | bool
@@ -29,16 +23,10 @@
   set_fact:
     _rbd_target_gw_handler_called: True
 
-- name: copy rbd-target-gw restart script
-  template:
-    src: restart_rbd_target_gw.sh.j2
-    dest: "{{ tmpdirpath.path }}/restart_rbd_target_gw.sh"
-    owner: root
-    group: root
-    mode: 0750
-
 - name: restart rbd-target-gw
-  command: /usr/bin/env bash {{ hostvars[item]['tmpdirpath']['path'] }}/restart_rbd_target_gw.sh
+  service:
+    name: rbd-target-gw
+    state: restarted
   when:
     - ceph_rbd_target_gw_stat.get('rc') == 0
     - hostvars[item]['_rbd_target_gw_handler_called'] | default(False) | bool

diff --git a/roles/ceph-handler/tasks/handler_tcmu_runner.yml b/roles/ceph-handler/tasks/handler_tcmu_runner.yml
index 70e9d50b6f2abb5c00ddb24d355bd80afadc1b1d..3dd31e0e9bc730f8edcc6b33539b8c423b9aa74c 100644 (file)
--- a/roles/ceph-handler/tasks/handler_tcmu_runner.yml
+++ b/roles/ceph-handler/tasks/handler_tcmu_runner.yml
@@ -3,16 +3,10 @@
   set_fact:
     _tcmu_runner_handler_called: True
 
-- name: copy tcmu-runner restart script
-  template:
-    src: restart_tcmu_runner.sh.j2
-    dest: "{{ tmpdirpath.path }}/restart_tcmu_runner.sh"
-    owner: root
-    group: root
-    mode: 0750
-
 - name: restart tcmu-runner
-  command: /usr/bin/env bash {{ hostvars[item]['tmpdirpath']['path'] }}/restart_tcmu_runner.sh
+  service:
+    name: tcmu-runner
+    state: restarted
   when:
     - ceph_tcmu_runner_stat.get('rc') == 0
     - hostvars[item]['_tcmu_runner_handler_called'] | default(False) | bool

diff --git a/roles/ceph-handler/templates/restart_rbd_target_api.sh.j2 b/roles/ceph-handler/templates/restart_rbd_target_api.sh.j2
deleted file mode 100644 (file)
index fd477c3..0000000
--- a/roles/ceph-handler/templates/restart_rbd_target_api.sh.j2
+++ /dev/null
@@ -1,3 +0,0 @@
-#!/bin/bash
-
-systemctl restart rbd-target-api

diff --git a/roles/ceph-handler/templates/restart_rbd_target_gw.sh.j2 b/roles/ceph-handler/templates/restart_rbd_target_gw.sh.j2
deleted file mode 100644 (file)
index 10c34bf..0000000
--- a/roles/ceph-handler/templates/restart_rbd_target_gw.sh.j2
+++ /dev/null
@@ -1,3 +0,0 @@
-#!/bin/bash
-
-systemctl restart rbd-target-gw

diff --git a/roles/ceph-handler/templates/restart_tcmu_runner.sh.j2 b/roles/ceph-handler/templates/restart_tcmu_runner.sh.j2
deleted file mode 100644 (file)
index 5dd5ff8..0000000
--- a/roles/ceph-handler/templates/restart_tcmu_runner.sh.j2
+++ /dev/null
@@ -1,3 +0,0 @@
-#!/bin/bash
-
-systemctl restart tcmu-runner