Skip to content

Commit 3c71f76

Browse files
committed
Revert "experiment: don't checkout the repo unless we have to build the docker image"
technically, it is an option to delay checking out the repo, however we'd need to duplicate the logic from the Makefile around the expected image name. i think not worth it. This reverts commit 95a8a18.
1 parent 057ebb1 commit 3c71f76

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

.github/workflows/vulnerability-scans.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,14 @@ jobs:
4848
image: ${{ steps.create-image-identifier.outputs.image }}
4949

5050
steps:
51+
- uses: actions/checkout@v4
52+
5153
- name: Create image identifier
5254
id: create-image-identifier
5355
run: |
54-
echo "image=${{ inputs.app_name }}-docker-image-${{ github.sha }}" >> "$GITHUB_OUTPUT"
56+
IMAGE_NAME=$(make APP_NAME=${{ inputs.app_name }} release-image-name)
57+
IMAGE_TAG=$(make release-image-tag)
58+
echo "image=$IMAGE_NAME:$IMAGE_TAG" >> "$GITHUB_OUTPUT"
5559
5660
- name: Check whether the image has already been built
5761
id: check-image-already-exists
@@ -61,8 +65,6 @@ jobs:
6165
key: ${{ steps.create-image-identifier.outputs.image }}
6266
lookup-only: true
6367

64-
- uses: actions/checkout@v4
65-
6668
- name: Build and tag Docker image for scanning
6769
if: steps.check-image-already-exists.outputs.cache-hit != 'true'
6870
run: |

0 commit comments

Comments
 (0)