From cfd72b2f825c69f2d81247d47483c5c6b461d27f Mon Sep 17 00:00:00 2001
From: Alvin Schiller <103769832+AlvinSchiller@users.noreply.github.com>
Date: Tue, 19 Mar 2024 14:03:47 +0100
Subject: [PATCH] Reactivate delete artifact  (#2297)

* activated delete-artifacts again with v5

* deactivate failOnError for deletion
---
 .../test_docker_debian_codename_sub_v3.yml    | 27 ++++++++++---------
 1 file changed, 14 insertions(+), 13 deletions(-)

diff --git a/.github/workflows/test_docker_debian_codename_sub_v3.yml b/.github/workflows/test_docker_debian_codename_sub_v3.yml
index 6deedb478..e3ef02bbe 100644
--- a/.github/workflows/test_docker_debian_codename_sub_v3.yml
+++ b/.github/workflows/test_docker_debian_codename_sub_v3.yml
@@ -138,7 +138,7 @@ jobs:
       with:
         name: ${{ steps.vars.outputs.image_file_name }}
         path: ${{ steps.vars.outputs.image_file_path }}
-        retention-days: 1
+        retention-days: 2
 
 
   # Run tests with build image
@@ -177,15 +177,16 @@ jobs:
         args: |
           ./${{ matrix.test_script }}
 
-  ## cleanup after test execution
-  #cleanup:
-  #  # run only if tests didn't fail: keep the artifact to make job reruns possible
-  #  if: ${{ !failure() }}
-  #  needs: [build, test]
-  #  runs-on: ${{ inputs.runs_on }}
-  #
-  #  steps:
-  #      - name: Artifact Delete Docker Image
-  #        uses: geekyeggo/delete-artifact@v4
-  #        with:
-  #          name: ${{ needs.build.outputs.image_file_name }}
+  # cleanup after test execution
+  cleanup:
+   # run only if tests didn't fail: keep the artifact to make job reruns possible
+   if: ${{ !failure() }}
+   needs: [build, test]
+   runs-on: ${{ inputs.runs_on }}
+  
+   steps:
+       - name: Artifact Delete Docker Image
+         uses: geekyeggo/delete-artifact@v5
+         with:
+           name: ${{ needs.build.outputs.image_file_name }}
+           failOnError: false