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>
(cherry picked from commit cbe79428e687e383f9764668a56171e5582451be)

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 c8bfb12c27ab5b9a374e08ece9f9e5b90842a01a..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: /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
+  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: /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
+  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 d2a82fb923e36e38fcd053636249af4a6429c8ab..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: /tmp/restart_tcmu_runner.sh
-    owner: root
-    group: root
-    mode: 0750
-
 - name: restart tcmu-runner
-  command: /usr/bin/env bash /tmp/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