Skip to content

Commit

Permalink
buildkite: split up autoskip tests (#4035)
Browse files Browse the repository at this point in the history
The autoskip tests are still periodically OOMing; this splits them up
further.

Signed-off-by: Alex Couture-Beil <[email protected]>
  • Loading branch information
alexcb authored Apr 17, 2024
1 parent 2718b79 commit 73f596b
Show file tree
Hide file tree
Showing 6 changed files with 37 additions and 2 deletions.
1 change: 1 addition & 0 deletions .buildkite/tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,7 @@ for target in \
+test-no-qemu-group8 \
+test-no-qemu-group9 \
+test-no-qemu-group10 \
+test-no-qemu-group11 \
+test-no-qemu-slow \
+test-qemu \
; do
Expand Down
14 changes: 14 additions & 0 deletions .github/workflows/ci-docker-ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -319,6 +319,20 @@ jobs:
EXTRA_ARGS: "--auto-skip"
secrets: inherit

docker-tests-no-qemu-group11:
needs: build-earthly
if: ${{ !failure() }}
uses: ./.github/workflows/reusable-test.yml
with:
TEST_TARGET: "+test-no-qemu-group11"
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
13 changes: 13 additions & 0 deletions .github/workflows/ci-podman-ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -196,6 +196,19 @@ jobs:
SKIP_JOB: ${{ needs.build-earthly.result != 'success' }}
secrets: inherit

podman-tests-no-qemu-group11:
needs: build-earthly
if: ${{ !failure() }}
uses: ./.github/workflows/reusable-test.yml
with:
TEST_TARGET: "+test-no-qemu-group11"
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
1 change: 1 addition & 0 deletions Earthfile
Original file line number Diff line number Diff line change
Expand Up @@ -660,6 +660,7 @@ test-no-qemu:
BUILD --pass-args +test-no-qemu-group8
BUILD --pass-args +test-no-qemu-group9
BUILD --pass-args +test-no-qemu-group10
BUILD --pass-args +test-no-qemu-group11
BUILD --pass-args +test-no-qemu-slow

# test-misc runs misc (non earthly-in-earthly) tests
Expand Down
3 changes: 3 additions & 0 deletions tests/Earthfile
Original file line number Diff line number Diff line change
Expand Up @@ -209,6 +209,9 @@ ga-no-qemu-group9:
ga-no-qemu-group10:
BUILD --pass-args ./autoskip+test-group2

ga-no-qemu-group11:
BUILD --pass-args ./autoskip+test-group3


ga-no-qemu-slow:
BUILD +server
Expand Down
7 changes: 5 additions & 2 deletions tests/autoskip/Earthfile
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ WORKDIR /test
test-all:
BUILD +test-group1
BUILD +test-group2
BUILD +test-group3

test-group1:
BUILD +test-files
Expand All @@ -23,18 +24,20 @@ test-group1:
BUILD +test-expand-args
BUILD +test-build-args
BUILD +test-pass-args

test-group2:
BUILD +test-copy-target-args
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
BUILD +test-remote-targets
BUILD +test-from-dockerfile
BUILD +test-import

test-group3:
BUILD +test-copy-target-args-quoted
BUILD +test-build-flag
BUILD +test-flag-conflict
Expand Down

0 comments on commit 73f596b

Please sign in to comment.