From e5924d5ecb9b79f6c5d7db8b693308053018be54 Mon Sep 17 00:00:00 2001
From: Johanna Dorothea Reichmann <transcaffeine@finally.coffee>
Date: Sat, 7 Oct 2023 16:16:53 +0200
Subject: [PATCH] chore(ghost): improve role task names and fix mount
 permissions

---
 roles/ghost/defaults/main.yml |  1 +
 roles/ghost/tasks/main.yml    | 17 +++++++++--------
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/roles/ghost/defaults/main.yml b/roles/ghost/defaults/main.yml
index 77c37d3..da8ea0d 100644
--- a/roles/ghost/defaults/main.yml
+++ b/roles/ghost/defaults/main.yml
@@ -36,3 +36,4 @@ ghost_container_restart_policy: "unless-stopped"
 ghost_container_networks: ~
 ghost_container_purge_networks: ~
 ghost_container_etc_hosts: ~
+ghost_container_state: started
diff --git a/roles/ghost/tasks/main.yml b/roles/ghost/tasks/main.yml
index 61fedff..8ce63ff 100644
--- a/roles/ghost/tasks/main.yml
+++ b/roles/ghost/tasks/main.yml
@@ -16,15 +16,16 @@
 
 - name: Ensure host paths for docker volumes exist for ghost
   ansible.builtin.file:
-    path: "{{ item }}"
+    path: "{{ item.path }}"
     state: directory
     mode: "0750"
-    owner: "{{ ghost_user }}"
-    group: "{{ ghost_user_group }}"
+    owner: "{{ item.owner | default(ghost_user) }}"
+    group: "{{ item.group | default(ghost_user_group) }}"
   loop:
-    - "{{ ghost_base_path }}"
-    - "{{ ghost_data_path }}"
-    - "{{ ghost_config_path }}"
+    - path: "{{ ghost_base_path }}"
+    - path: "{{ ghost_data_path }}"
+      owner: "1000"
+    - path: "{{ ghost_config_path }}"
 
 - name: Ensure ghost configuration file is templated
   ansible.builtin.template:
@@ -41,7 +42,7 @@
     source: pull
     force_source: "{{ ghost_container_image_tag is defined }}"
 
-- name: Ensure ghost container is running
+- name: Ensure ghost container '{{ ghost_container_name }}' is {{ ghost_container_state }}
   community.docker.docker_container:
     name: "{{ ghost_container_name }}"
     image: "{{ ghost_container_image }}"
@@ -53,4 +54,4 @@
     networks: "{{ ghost_container_networks | default(omit, true) }}"
     purge_networks: "{{ ghost_container_purge_networks | default(omit, true) }}"
     restart_policy: "{{ ghost_container_restart_policy }}"
-    state: started
+    state: "{{ ghost_container_state }}"