From f869c0f643a170186e93f83c1eb60905bc68fdfa Mon Sep 17 00:00:00 2001 From: Adam Williamson Date: Thu, 26 May 2022 15:23:13 -0700 Subject: [PATCH] Revert "openqa/worker: handle git 'safety' check for test dir" This reverts commit 34b3d3a5cc57d84f96c5a4198529a7d2d7a9f7f1. On second thoughts it's kinda ugly and I need to think about other options... --- roles/openqa/worker/tasks/main.yml | 6 ------ 1 file changed, 6 deletions(-) diff --git a/roles/openqa/worker/tasks/main.yml b/roles/openqa/worker/tasks/main.yml index 20eb04cd37..2bbd6b4da7 100644 --- a/roles/openqa/worker/tasks/main.yml +++ b/roles/openqa/worker/tasks/main.yml @@ -170,12 +170,6 @@ - include_tasks: tap-setup.yml when: openqa_tap|bool -- name: Tell git it's OK for _openqa-worker to run 'git' on the test dir - command: git config --global --add safe.directory /var/lib/openqa/share/tests/fedora - become: yes - become_user: _openqa-worker - when: openqa_nfs_worker|bool - # this is kinda lazy - we could have a separate openqa_swtpm var so we # we could potentially separate tap worker hosts from swtpm ones - but # it makes workers.ini templating annoyingly awkward (we need way more