Skip to content

Commit 2029a69

Browse files
committed
Move image artifact cleanup to end of each workflow to avoid race condition
1 parent c4d5bf3 commit 2029a69

2 files changed

Lines changed: 23 additions & 26 deletions

File tree

.github/workflows/build-container-images.yml

Lines changed: 12 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -321,7 +321,7 @@ jobs:
321321
- name: Upload image as artifact
322322
uses: actions/upload-artifact@v4
323323
with:
324-
name: ${{ matrix.image.name }}
324+
name: build-images.${{ matrix.image.name }}
325325
path: ${{ runner.temp }}/${{ matrix.image.name }}.tar
326326
retention-days: 1
327327

@@ -341,25 +341,13 @@ jobs:
341341
echo "::add-mask::$role_arn"
342342
echo "role-arn=$role_arn" >> "$GITHUB_OUTPUT"
343343
344-
- name: Delete base-images artifact
345-
uses: GeekyEggo/delete-artifact@v5.1.0
346-
with:
347-
name: base-images
348-
failOnError: false
349-
350344
- name: Download all built image artifacts
351345
uses: actions/download-artifact@v4
352346
with:
347+
pattern: build-images.*
353348
path: ${{ runner.temp }}
354349
merge-multiple: true
355350

356-
- name: Delete all image artifacts
357-
if: ${{ inputs.cleanupImageArtifacts }}
358-
uses: GeekyEggo/delete-artifact@v5.1.0
359-
with:
360-
name: '*'
361-
failOnError: false
362-
363351
- name: Configure AWS credentials
364352
uses: aws-actions/configure-aws-credentials@v4
365353
with:
@@ -398,3 +386,13 @@ jobs:
398386
done
399387
working-directory: ${{ runner.temp }}
400388

389+
cleanup-artifacts:
390+
if: ${{ !cancelled() && !inputs.cleanupImageArtifacts }}
391+
runs-on: ubuntu-24.04
392+
steps:
393+
- name: Delete all image artifacts
394+
uses: GeekyEggo/delete-artifact@v5.1.0
395+
with:
396+
name: '*'
397+
failOnError: false
398+

.github/workflows/build-release.yml

Lines changed: 11 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -147,12 +147,6 @@ jobs:
147147
name: bfd-platform-base-java
148148
path: ${{ runner.temp }}
149149

150-
- name: Delete unnecessary image Artifacts
151-
uses: GeekyEggo/delete-artifact@v5.1.0
152-
with:
153-
name: '*'
154-
failOnError: false
155-
156150
- name: Load base image into Docker
157151
run: |
158152
docker load --input "${{ runner.temp }}/bfd-platform-base-java.tar"
@@ -267,12 +261,6 @@ jobs:
267261
name: jib-images
268262
path: ${{ runner.temp }}
269263

270-
- name: Delete jib image artifacts
271-
uses: GeekyEggo/delete-artifact@v5.1.0
272-
with:
273-
name: jib-images
274-
failOnError: false
275-
276264
- name: Configure AWS credentials
277265
uses: aws-actions/configure-aws-credentials@v4
278266
with:
@@ -399,3 +387,14 @@ jobs:
399387
tag: ${{ needs.compute-version-strings.outputs.bfd_release }}
400388
name: "v${{ needs.compute-version-strings.outputs.bfd_release }}"
401389
artifacts: "*.csv,*.json,*.xlsx,*.yaml"
390+
391+
cleanup-artifacts:
392+
if: ${{ !cancelled() }}
393+
runs-on: ubuntu-24.04
394+
steps:
395+
- name: Delete all image artifacts
396+
uses: GeekyEggo/delete-artifact@v5.1.0
397+
with:
398+
name: '*'
399+
failOnError: false
400+

0 commit comments

Comments
 (0)