From 130a7a779642144934e5e3d37369283c5e0ffa01 Mon Sep 17 00:00:00 2001 From: Jayne Passmore Date: Tue, 20 May 2025 19:51:40 -0500 Subject: [PATCH 1/4] fix system.xml ownership --- tasks/main.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tasks/main.yml b/tasks/main.yml index 938c3aa..e7fa3b6 100644 --- a/tasks/main.yml +++ b/tasks/main.yml @@ -18,6 +18,8 @@ - name: deploy jellyfin system.xml file ansible.builtin.copy: + owner: jellyfin + group: jellyfin src: files/system.xml dest: /etc/jellyfin/system.xml notify: restart jellyfin -- 2.50.1 From 4160544e887f46c59a836365e5a39472207e792a Mon Sep 17 00:00:00 2001 From: Jayne Passmore Date: Sat, 24 May 2025 19:04:36 -0500 Subject: [PATCH 2/4] added pause before restarting jellyfin --- handlers/main.yml | 8 +++++++- tasks/main.yml | 12 ++++++------ 2 files changed, 13 insertions(+), 7 deletions(-) diff --git a/handlers/main.yml b/handlers/main.yml index d89abd9..ce64ed3 100644 --- a/handlers/main.yml +++ b/handlers/main.yml @@ -1,7 +1,13 @@ #SPDX-License-Identifier: MIT-0 --- # handlers file for role-jellyfin +- name: pause to ensure jellyfin has completely started on first run + ansible.builtin.pause: + seconds: 30 + listen: "restart jellyfin" + - name: restart jellyfin service: name: jellyfin - state: restarted \ No newline at end of file + state: restarted + listen: "restart jellyfin" \ No newline at end of file diff --git a/tasks/main.yml b/tasks/main.yml index e7fa3b6..f3e0cca 100644 --- a/tasks/main.yml +++ b/tasks/main.yml @@ -16,6 +16,12 @@ owner: jellyfin group: jellyfin +- name: ensure jellyfin is running + service: + name: jellyfin + state: started + enabled: yes + - name: deploy jellyfin system.xml file ansible.builtin.copy: owner: jellyfin @@ -23,9 +29,3 @@ src: files/system.xml dest: /etc/jellyfin/system.xml notify: restart jellyfin - -- name: ensure jellyfin is running - service: - name: jellyfin - state: started - enabled: yes \ No newline at end of file -- 2.50.1 From cc42a82497f0d6c27466f3802dbca0435e128446 Mon Sep 17 00:00:00 2001 From: Jayne Passmore Date: Sat, 24 May 2025 21:38:11 -0500 Subject: [PATCH 3/4] added cache dir --- handlers/main.yml | 8 +------- tasks/main.yml | 19 +++++++++++++------ 2 files changed, 14 insertions(+), 13 deletions(-) diff --git a/handlers/main.yml b/handlers/main.yml index ce64ed3..d89abd9 100644 --- a/handlers/main.yml +++ b/handlers/main.yml @@ -1,13 +1,7 @@ #SPDX-License-Identifier: MIT-0 --- # handlers file for role-jellyfin -- name: pause to ensure jellyfin has completely started on first run - ansible.builtin.pause: - seconds: 30 - listen: "restart jellyfin" - - name: restart jellyfin service: name: jellyfin - state: restarted - listen: "restart jellyfin" \ No newline at end of file + state: restarted \ No newline at end of file diff --git a/tasks/main.yml b/tasks/main.yml index f3e0cca..48ee467 100644 --- a/tasks/main.yml +++ b/tasks/main.yml @@ -2,7 +2,7 @@ --- # tasks file for role-jellyfin -- name: install nextcloud and dependencies +- name: install jellyfin and dependencies ansible.builtin.package: name: - jellyfin-server @@ -16,11 +16,12 @@ owner: jellyfin group: jellyfin -- name: ensure jellyfin is running - service: - name: jellyfin - state: started - enabled: yes +- name: ensure jellyfin cache is owned by jellyfin user + ansible.builtin.file: + path: /var/cache/jellyfin/ + recurse: yes + owner: jellyfin + group: jellyfin - name: deploy jellyfin system.xml file ansible.builtin.copy: @@ -29,3 +30,9 @@ src: files/system.xml dest: /etc/jellyfin/system.xml notify: restart jellyfin + +- name: ensure jellyfin is running + service: + name: jellyfin + state: started + enabled: yes \ No newline at end of file -- 2.50.1 From 40100006edc051f81d5b1fec2a7ccbe08c083f2f Mon Sep 17 00:00:00 2001 From: Jayne Passmore Date: Sat, 24 May 2025 21:49:21 -0500 Subject: [PATCH 4/4] added config dir --- tasks/main.yml | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/tasks/main.yml b/tasks/main.yml index 48ee467..fa928d7 100644 --- a/tasks/main.yml +++ b/tasks/main.yml @@ -9,6 +9,13 @@ - jellyfin-web state: present +- name: ensure jellyfin config is owned by jellyfin user + ansible.builtin.file: + path: /etc/jellyfin/ + recurse: yes + owner: jellyfin + group: jellyfin + - name: ensure jellyfin data is owned by jellyfin user ansible.builtin.file: path: /var/lib/jellyfin/ -- 2.50.1