From 07c894ea213778935120a05a472eaffc688f9a1b Mon Sep 17 00:00:00 2001 From: Kevin Fenzi Date: Thu, 29 May 2025 15:28:21 -0700 Subject: [PATCH] pkgs: drop recursive call to facl setting In a0046b5b4b7 there was a bunch of fixes for ansible lint to the distgit/pagure roles. However, it seems like a recursive: true was added to the facl call, when it wasn't present before. I noticed this when my playbook run on pkgs was running for several hours. ;( I don't think there's any reason to run this recursively, as permissions should inherit from the top level one and have always worked without it. Signed-off-by: Kevin Fenzi --- roles/distgit/pagure/tasks/main.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/roles/distgit/pagure/tasks/main.yml b/roles/distgit/pagure/tasks/main.yml index 40b82e4b17..45ec4ebcf0 100644 --- a/roles/distgit/pagure/tasks/main.yml +++ b/roles/distgit/pagure/tasks/main.yml @@ -241,7 +241,6 @@ permissions: "rx" name: /srv/git state: present - recursive: yes register: acl_updates tags: - pagure