Skip to content

Commit

Permalink
Reactivate delete artifact (MiczFlor#2297)
Browse files Browse the repository at this point in the history
* activated delete-artifacts again with v5

* deactivate failOnError for deletion
  • Loading branch information
AlvinSchiller authored Mar 19, 2024
1 parent 4d05c75 commit cfd72b2
Showing 1 changed file with 14 additions and 13 deletions.
27 changes: 14 additions & 13 deletions .github/workflows/test_docker_debian_codename_sub_v3.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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

0 comments on commit cfd72b2

Please sign in to comment.