Skip to content

Commit

Permalink
✨ Test image build on Dockerfile change
Browse files Browse the repository at this point in the history
Signed-off-by: Scott J Dickerson <[email protected]>
  • Loading branch information
sjd78 committed May 10, 2024
1 parent 4884439 commit f340daa
Show file tree
Hide file tree
Showing 2 changed files with 103 additions and 3 deletions.
99 changes: 99 additions & 0 deletions .github/workflows/ci-Dockerfile.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,99 @@
name: CI (test image build for a PR with a Dockerfile change)

on:
pull_request:
branches:
- "main"
- "release-*"

jobs:
checks:
runs-on: ubuntu-latest
outputs:
run-test-image-build: ${{ steps.dockerChanged.outputs.flag }}

steps:
- uses: actions/checkout@v4

- name: Log the PR event
env:
GH: ${{ toJson(github.event.pull_request) }}
run: |
cat >> "$GITHUB_STEP_SUMMARY" <<EOF
\`\`\`json
$(echo "$GH" | jq '.')
\`\`\`
EOF
- name: Dockerfile changed?
id: changed-Dockerfile
uses: tj-actions/changed-files@v44
with:
files: |
Dockerfile
- name: Check if the `Dockerfile` has been changed by a PR
id: dockerChanged
env:
IS_PR: ${{ !!github.event.pull_request }}
CHANGED: ${{ steps.dockerChanged.outputs.test_any_changed }}
run: |
INCLUDES_DOCKERFILE=$(
if [[ $(git show --raw --oneline HEAD -- Dockerfile | wc --lines) -eq 0 ]]; then
echo "false"
else
echo "true"
fi
)
TEST_IMAGE_BUILD=$(
if [[ $IS_PR == true ]] && [[ $INCLUDES_DOCKERFILE == true ]]; then
echo "true"
else
echo "false"
fi
)
cat >> "$GITHUB_STEP_SUMMARY" <<EOF
PR triggered? \`$IS_PR\`
HEAD commit include a Dockerfile change? \`$INCLUDES_DOCKERFILE\`
Should the image build be tested? \`$TEST_IMAGE_BUILD\`
CHANGED: \`$CHANGED\`
EOF
echo "flag=$TEST_IMAGE_BUILD" >> "$GITHUB_OUTPUT"
#
# Based on:
# - image-build.yaml
# - konveyor/release-tools/.github/workflows/build-push-images.yaml@main
#
# Only test the image build, no push to quay is required.
#
test-image-build:
runs-on: ubuntu-latest
needs: checks
if: ${{ needs.checks.outputs.run-test-image-build == true }}

strategy:
fail-fast: true
matrix:
architecture:
- amd64
- arm64

steps:
- name: Checkout merge commit for PR${{ github.event.pull_request.number }}
uses: actions/checkout@v4

- name: Test build image on ${{ matrix.architecture }}
id: test-build
uses: redhat-actions/buildah-build@main
with:
image: "tackle2-ui"
tags: PR${{ github.event.pull_request.number }}-${{ matrix.architecture }}
extra-args: "--no-cache --rm --ulimit nofile=4096:4096"
archs: ${{ matrix.architecture }}
labels: ""
containerfiles: "./Dockerfile"
context: "."
7 changes: 4 additions & 3 deletions .github/workflows/ci-repo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,11 @@ on:
workflow_dispatch:

jobs:
unit-test-lookup-image:
unit-test-lookups:
runs-on: ubuntu-latest
outputs:
builder-image: ${{ steps.grepBuilder.outputs.builder }}

steps:
- uses: actions/checkout@v4

Expand All @@ -33,10 +34,10 @@ jobs:
unit-test:
runs-on: ubuntu-latest
needs: unit-test-lookup-image
needs: unit-test-lookups

# Use the same container as the Dockerfile's "FROM * as builder"
container: ${{ needs.unit-test-lookup-image.outputs.builder-image }}
container: ${{ needs.unit-test-lookups.outputs.builder-image }}

steps:
- uses: actions/checkout@v4
Expand Down

0 comments on commit f340daa

Please sign in to comment.