From e77961d36bff9be883400e3c467998b450079655 Mon Sep 17 00:00:00 2001
From: Pavel Raiskup <praiskup@redhat.com>
Date: Wed, 30 Oct 2024 11:31:26 +0100
Subject: [PATCH] copr-backend: backup log-detective "as copr"

There's no need to hide the backup tarballs, since we download them from
a public place.
---
 roles/log-detective-backup/defaults/main.yml |  2 +-
 roles/log-detective-backup/tasks/main.yml    | 17 +++++++++--------
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/roles/log-detective-backup/defaults/main.yml b/roles/log-detective-backup/defaults/main.yml
index 3f08cc35e5..a263e45a4b 100644
--- a/roles/log-detective-backup/defaults/main.yml
+++ b/roles/log-detective-backup/defaults/main.yml
@@ -1,5 +1,5 @@
 ---
-ld_backup_user: log-detective-backups
+ld_backup_user: copr
 ld_backup_path: /var/lib/copr/public_html/log-detective-backups
 ld_backup_script: /usr/bin/log-detective-backup
 ld_dump_url: https://log-detective.com/download
diff --git a/roles/log-detective-backup/tasks/main.yml b/roles/log-detective-backup/tasks/main.yml
index f67856b562..fb72954972 100644
--- a/roles/log-detective-backup/tasks/main.yml
+++ b/roles/log-detective-backup/tasks/main.yml
@@ -1,11 +1,11 @@
 ---
-- name: Add the user 'log-detective-backup'
-  ansible.builtin.user:
-    name: "{{ ld_backup_user }}"
-    comment: User used for backing-up log-detective.com
-    uid: 15151
-  tags:
-    - log_detective_backup
+# - name: Add the user 'log-detective-backup'
+#   ansible.builtin.user:
+#     name: "{{ ld_backup_user }}"
+#     comment: User used for backing-up log-detective.com
+#     uid: 15151
+#   tags:
+#     - log_detective_backup
 
 - name: create the log-detective.com backup directory
   ansible.builtin.file:
@@ -13,7 +13,8 @@
     state: directory
     owner: "{{ ld_backup_user }}"
     group: "{{ ld_backup_user }}"
-    mode: '0700'
+    # public tarball -> public backups
+    mode: '0755'
   tags:
     - log_detective_backup