From 3f73f311276c7b05eff0d33c82e00c90efb61d4e Mon Sep 17 00:00:00 2001 From: Daniel Baptista Dias Date: Mon, 11 Sep 2023 10:54:29 -0300 Subject: [PATCH] fix: adding github action parallel runs to cli e2e tests (#3136) * 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 --- .github/workflows/pull-request.yaml | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pull-request.yaml b/.github/workflows/pull-request.yaml index e506cc07e2..0d47dafc71 100644 --- a/.github/workflows/pull-request.yaml +++ b/.github/workflows/pull-request.yaml @@ -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 @@ -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 @@ -348,6 +352,8 @@ 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 @@ -355,7 +361,8 @@ jobs: 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