Skip to content

Commit

Permalink
Make build_test_matrix workflows fail if exit code 1 (#5356)
Browse files Browse the repository at this point in the history
* run go before echo

* run go before echo

* set matrix

* try remove a suite

* env

* revert testing

* check output

---------

Co-authored-by: Dzung Do | Decentrio <[email protected]>
Co-authored-by: Long <[email protected]>
Co-authored-by: DimitrisJim <[email protected]>
  • Loading branch information
4 people authored Dec 13, 2023
1 parent 8cc5b0a commit 39e6073
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 4 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/e2e-fork.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,9 @@ jobs:
with:
go-version: '1.21'
- id: set-matrix
run: echo "matrix=$(go run cmd/build_test_matrix/main.go)" >> $GITHUB_OUTPUT
run: |
output=$(go run cmd/build_test_matrix/main.go)
echo "matrix=$output" >> $GITHUB_OUTPUT
env:
TEST_EXCLUSIONS: 'TestUpgradeTestSuite,TestGrandpaTestSuite,TestIBCWasmUpgradeTestSuite'

Expand All @@ -38,11 +40,12 @@ jobs:
with:
go-version: '1.21'
- id: set-matrix
run: echo "matrix=$(go run cmd/build_test_matrix/main.go)" >> $GITHUB_OUTPUT
run: |
output=$(go run cmd/build_test_matrix/main.go)
echo "matrix=$output" >> $GITHUB_OUTPUT
env:
TEST_ENTRYPOINT: 'TestGrandpaTestSuite'


# e2e-fork runs the e2e tests directly by using a matrix strategy.
e2e-fork:
env:
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/e2e-test-workflow-call.yml
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,9 @@ jobs:
with:
go-version: '1.21'
- id: set-matrix
run: echo "matrix=$(go run cmd/build_test_matrix/main.go)" >> $GITHUB_OUTPUT
run: |
output=$(go run cmd/build_test_matrix/main.go)
echo "matrix=$output" >> $GITHUB_OUTPUT
env:
TEST_ENTRYPOINT: '${{ inputs.test-entry-point }}'
TEST_EXCLUSIONS: '${{ inputs.test-exclusions }}'
Expand Down

0 comments on commit 39e6073

Please sign in to comment.