From 7b2d04f850cdb6655bf59e1e61ad4edb8d1497cd Mon Sep 17 00:00:00 2001
From: David Moreau Simard <dmsimard@redhat.com>
Date: Tue, 24 Apr 2018 21:12:21 -0400
Subject: [PATCH] Reload systemctl and restart ara on service unit file

We were not reloading or restarting ara after changing
the parameters.
This means that running a second time with different
parameters will not be effective automatically.

Also, in order to be effective, the unit file we setup
needs to be in /etc/systemd, not /usr/lib/systemd.

Change-Id: I53716fc2bdb7cf8afc2c5ea39ab953a51c9c4585
---
 handlers/main.yml          | 12 +++++++++++-
 tasks/embedded_server.yaml | 11 ++++++++---
 2 files changed, 19 insertions(+), 4 deletions(-)

diff --git a/handlers/main.yml b/handlers/main.yml
index 742464d..c331ce3 100644
--- a/handlers/main.yml
+++ b/handlers/main.yml
@@ -13,8 +13,18 @@
 #   License for the specific language governing permissions and limitations
 #   under the License.
 
+- name: reload systemctl daemon
+  systemd:
+    daemon_reload: yes
+
 - name: restart apache
   service:
    name: "{{ apache_service }}"
-   state: "restarted"
+   state: restarted
+  become: true
+
+- name: restart ara
+  service:
+    name: ara
+    state: restarted
   become: true
diff --git a/tasks/embedded_server.yaml b/tasks/embedded_server.yaml
index d129d6c..fe8eda1 100644
--- a/tasks/embedded_server.yaml
+++ b/tasks/embedded_server.yaml
@@ -23,15 +23,20 @@
     - name: Copy systemd service template
       template:
         src: templates/ara-service.conf.j2
-        dest: /usr/lib/systemd/system/ara.service
+        dest: /etc/systemd/system/ara.service
         owner: root
         group: root
         mode: 0644
+      notify:
+        - reload systemctl daemon
+        - restart ara
 
-    - name: Start and enable the embedded server service
+    - name: Flush handlers
+      meta: flush_handlers
+
+    - name: Ensure the embedded server service is started and enabled
       systemd:
         name: ara
         state: started
         enabled: yes
-        daemon_reload: yes
   become: true