Skip to content

Commit

Permalink
fix: adding github action parallel runs to cli e2e tests (#3136)
Browse files Browse the repository at this point in the history
* fix: adding parallelism to cli e2e tests

* Fixing CI

* Update CI

* Update CI

* Update CI

* Update CI

* Fix CI script

* Fix CI script

* Updating CLI e2e tests
  • Loading branch information
danielbdias authored Sep 11, 2023
1 parent c1851fa commit 3f73f31
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 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:
total_splits: [8]
index: [0,1,2,3,4,5,6,7]
steps:
- name: Checkout
uses: actions/checkout@v3
Expand All @@ -348,14 +352,17 @@ jobs:
docker load --input dist/image.tar
- name: Run tests
run: |
curl -sfL https://raw.githubusercontent.com/Songmu/gotesplit/main/install.sh | sh -s
find ./dist -name 'tracetest' -exec cp {} ./dist \;
chmod +x ./dist/tracetest
export TRACETEST_CLI=$PWD/dist/tracetest
export TEST_ENVIRONMENT=jaeger
cd ./testing/cli-e2etest
make test
../../bin/gotesplit -total ${{ matrix.total_splits }} -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 3f73f31

Please sign in to comment.