Skip to content

Commit

Permalink
Reactivate delete artifact (#2296)
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 33f240b commit 8830728
Showing 1 changed file with 14 additions and 13 deletions.
27 changes: 14 additions & 13 deletions .github/workflows/test_docker_debian_codename_sub.yml
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,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 @@ -176,15 +176,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 8830728

Please sign in to comment.