Skip to content

Commit

Permalink
split up autoskip tests (#4025)
Browse files Browse the repository at this point in the history
the autoskip+test-all target requires more than 16GB of memory, which is
causing the M1 test instance to run out of memory and fail.

Signed-off-by: Alex Couture-Beil <[email protected]>
  • Loading branch information
alexcb authored Apr 16, 2024
1 parent ec7ba23 commit bf607d2
Show file tree
Hide file tree
Showing 5 changed files with 69 additions and 1 deletion.
2 changes: 2 additions & 0 deletions .buildkite/tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,8 @@ for target in \
+test-no-qemu-group6 \
+test-no-qemu-group7 \
+test-no-qemu-group8 \
+test-no-qemu-group9 \
+test-no-qemu-group10 \
+test-no-qemu-slow \
+test-qemu \
; do
Expand Down
28 changes: 28 additions & 0 deletions .github/workflows/ci-docker-ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -291,6 +291,34 @@ jobs:
SKIP_JOB: ${{ needs.build-earthly.result != 'success' }}
secrets: inherit

docker-tests-no-qemu-group9:
needs: build-earthly
if: ${{ !failure() }}
uses: ./.github/workflows/reusable-test.yml
with:
TEST_TARGET: "+test-no-qemu-group9"
BUILT_EARTHLY_PATH: "./build/linux/amd64/earthly"
RUNS_ON: "ubuntu-latest"
BINARY: "docker"
SUDO: ""
SKIP_JOB: ${{ needs.build-earthly.result != 'success' }}
EXTRA_ARGS: "--auto-skip"
secrets: inherit

docker-tests-no-qemu-group10:
needs: build-earthly
if: ${{ !failure() }}
uses: ./.github/workflows/reusable-test.yml
with:
TEST_TARGET: "+test-no-qemu-group10"
BUILT_EARTHLY_PATH: "./build/linux/amd64/earthly"
RUNS_ON: "ubuntu-latest"
BINARY: "docker"
SUDO: ""
SKIP_JOB: ${{ needs.build-earthly.result != 'success' }}
EXTRA_ARGS: "--auto-skip"
secrets: inherit

docker-tests-no-qemu-slow:
needs: build-earthly
if: ${{ !failure() }}
Expand Down
26 changes: 26 additions & 0 deletions .github/workflows/ci-podman-ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,32 @@ jobs:
SKIP_JOB: ${{ needs.build-earthly.result != 'success' }}
secrets: inherit

podman-tests-no-qemu-group9:
needs: build-earthly
if: ${{ !failure() }}
uses: ./.github/workflows/reusable-test.yml
with:
TEST_TARGET: "+test-no-qemu-group9"
BUILT_EARTHLY_PATH: "./build/linux/amd64/earthly"
RUNS_ON: "ubuntu-latest"
BINARY: "podman"
SUDO: "sudo -E"
SKIP_JOB: ${{ needs.build-earthly.result != 'success' }}
secrets: inherit

podman-tests-no-qemu-group10:
needs: build-earthly
if: ${{ !failure() }}
uses: ./.github/workflows/reusable-test.yml
with:
TEST_TARGET: "+test-no-qemu-group10"
BUILT_EARTHLY_PATH: "./build/linux/amd64/earthly"
RUNS_ON: "ubuntu-latest"
BINARY: "podman"
SUDO: "sudo -E"
SKIP_JOB: ${{ needs.build-earthly.result != 'success' }}
secrets: inherit

podman-tests-no-qemu-slow:
needs: build-earthly
if: ${{ !failure() }}
Expand Down
8 changes: 7 additions & 1 deletion tests/Earthfile
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,6 @@ ga-no-qemu-group7:
BUILD +cache-mount-arg
BUILD +infinite-recursion
BUILD +save-artifact-selective
BUILD --pass-args ./autoskip+test-all
BUILD +if-exists
BUILD +let-set
BUILD +let-scope
Expand All @@ -203,6 +202,13 @@ ga-no-qemu-group8:
BUILD --pass-args ./pass-args-global-via-function+test
BUILD --pass-args ./pass-args-via-function-with-override+test

ga-no-qemu-group9:
BUILD --pass-args ./autoskip+test-group1

ga-no-qemu-group10:
BUILD --pass-args ./autoskip+test-group2


ga-no-qemu-slow:
BUILD +server
BUILD --pass-args ./with-docker+all
Expand Down
6 changes: 6 additions & 0 deletions tests/autoskip/Earthfile
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,10 @@ PROJECT earthly-technologies/core
WORKDIR /test

test-all:
BUILD +test-group1
BUILD +test-group2

test-group1:
BUILD +test-files
BUILD +test-with-subdir
BUILD +test-requires-project
Expand All @@ -23,6 +27,8 @@ test-all:
BUILD +test-arg-matrix
BUILD +test-try-catch
BUILD +test-push

test-group2:
BUILD +test-no-cache
BUILD +test-shell-out
BUILD +test-copy-if-exists
Expand Down

0 comments on commit bf607d2

Please sign in to comment.