Skip to content

Commit

Permalink
Merge pull request #1007 from rvykydal/daily-tests-error-on-the-right…
Browse files Browse the repository at this point in the history
…-place

infra: Raise kstest errors in the permian step
  • Loading branch information
rvykydal authored Sep 14, 2023
2 parents 8713353 + ae84fb5 commit 6f668a2
Showing 1 changed file with 8 additions and 11 deletions.
19 changes: 8 additions & 11 deletions .github/workflows/scenarios-permian.yml
Original file line number Diff line number Diff line change
Expand Up @@ -179,6 +179,14 @@ jobs:
}
}'
# Permian hides the exit code of launcher, so error out this step manually based on logs
rc=$( awk '/Runner return code: /{ print $4 }' permian.log)
if [ -n "$rc" ]; then
exit $rc
else
exit 111
fi
- name: Collect anaconda logs
if: always()
uses: actions/upload-artifact@v3
Expand Down Expand Up @@ -214,14 +222,3 @@ jobs:
name: 'results-xunit-${{ matrix.scenario }}'
path: |
permian/xunit-*.xml
# Keep compatibility of the overall results, Permian now hides the exit code of launcher
- name: Pass the launch script exit code
working-directory: ./permian
run: |
rc=$( awk '/Runner return code: /{ print $4 }' permian.log)
if [ -n "$rc" ]; then
exit $rc
else
exit 111
fi

0 comments on commit 6f668a2

Please sign in to comment.