Skip to content

Commit

Permalink
fix: adding parallelism to cli e2e tests
Browse files Browse the repository at this point in the history
  • Loading branch information
danielbdias committed Sep 8, 2023
1 parent 4e0515c commit 398e5e8
Showing 1 changed file with 13 additions and 3 deletions.
16 changes: 13 additions & 3 deletions .github/workflows/pull-request.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,6 @@ jobs:
- name: Run unit tests
run: cd agent; make test -B


unit-test-web:
name: WebUI unit tests
runs-on: ubuntu-latest
Expand Down Expand Up @@ -336,6 +335,11 @@ jobs:
name: CLI e2e tests
needs: [build-docker]
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
parallelism: [4]
index: [0,1,2,3]
steps:
- name: Checkout
uses: actions/checkout@v3
Expand All @@ -346,16 +350,22 @@ jobs:
- name: Import image
run: |
docker load --input dist/image.tar
- name: Run tests parallelly
run: |
curl -sfL https://raw.githubusercontent.com/Songmu/gotesplit/main/install.sh | sh -s
- name: Run tests
run: |
find ./dist -name 'tracetest' -exec cp {} ./dist \;
chmod +x ./dist/tracetest
cd ./testing/cli-e2etest
export TRACETEST_CLI=$PWD/dist/tracetest
export TEST_ENVIRONMENT=jaeger
export ENABLE_CLI_DEBUG=false
export TAG=$(TAG)
cd ./testing/cli-e2etest
make test
bin/gotesplit -total ${{ matrix.parallelism }} -index ${{ matrix.index }} ./... -- -v -timeout 300s -p 1
# TODO: this would be a great idea but it doesn't work on GHA with docker
# it can probablly be implemented with k8s in a separated job
Expand Down

0 comments on commit 398e5e8

Please sign in to comment.