From f6be25a6ae05780282a4fc4f121fadcd7d454766 Mon Sep 17 00:00:00 2001
From: Slavi Pantaleev <s.pantaleev@gmail.com>
Date: Fri, 8 Sep 2017 16:34:40 +0300
Subject: [PATCH] Fix idempotency problem when getting rid of S3 setup

---
 roles/matrix-server/tasks/setup_s3fs.yml | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/roles/matrix-server/tasks/setup_s3fs.yml b/roles/matrix-server/tasks/setup_s3fs.yml
index f3312195..b37cc568 100644
--- a/roles/matrix-server/tasks/setup_s3fs.yml
+++ b/roles/matrix-server/tasks/setup_s3fs.yml
@@ -25,16 +25,21 @@
 #
 # Tasks related to getting rid of s3fs (if it was previously enabled)
 #
+
+- name: Check existance of matrix-s3fs service
+  stat: path="/etc/systemd/system/matrix-s3fs.service"
+  register: matrix_s3fs_service_stat
+
 - name: Ensure matrix-s3fs is stopped
   service: name=matrix-s3fs state=stopped daemon_reload=yes
   register: stopping_result
-  when: "not matrix_s3_media_store_enabled"
+  when: "not matrix_s3_media_store_enabled and matrix_s3fs_service_stat.stat.exists"
 
 - name: Ensure matrix-s3fs.service doesn't exist
   file:
-    path: "{{ matrix_base_data_path }}/s3fs-credentials"
+    path: "/etc/systemd/system/matrix-s3fs.service"
     state: absent
-  when: "not matrix_s3_media_store_enabled"
+  when: "not matrix_s3_media_store_enabled and matrix_s3fs_service_stat.stat.exists"
 
 - name: Ensure s3fs-credentials doesn't exist
   file: