Skip to content

Commit

Permalink
8307837: [8u] Check step in GHA should also print errors
Browse files Browse the repository at this point in the history
Reviewed-by: andrew
  • Loading branch information
zzambers committed Oct 2, 2023
1 parent f4260a4 commit 27058b2
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions .github/workflows/submit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,7 @@ jobs:
run: >
if cat test-results/testoutput/*/exitcode.txt | grep -q -v '^0$'
|| ! cat test-results/testoutput/*/Stats.txt | grep -q 'fail=0' ; then
cat test-results/testoutput/*/JTreport/text/newfailures.txt ;
cat test-results/testoutput/*/JTreport/text/{newfailures,other_errors}.txt ;
exit 1 ;
fi
Expand Down Expand Up @@ -638,7 +638,7 @@ jobs:
run: >
if cat test-results/testoutput/*/exitcode.txt | grep -q -v '^0$'
|| ! cat test-results/testoutput/*/Stats.txt | grep -q 'fail=0' ; then
cat test-results/testoutput/*/JTreport/text/newfailures.txt ;
cat test-results/testoutput/*/JTreport/text/{newfailures,other_errors}.txt ;
exit 1 ;
fi
Expand Down Expand Up @@ -1143,6 +1143,7 @@ jobs:
run: >
if ((Get-ChildItem -Path test-results\testoutput\*\exitcode.txt -Recurse | Select-String -Pattern '^0$' -NotMatch ).Count -gt 0) {
Get-Content -Path test-results\testoutput\*\JTreport\text\newfailures.txt ;
Get-Content -Path test-results\testoutput\*\JTreport\text\other_errors.txt ;
exit 1
}
Expand Down Expand Up @@ -1299,6 +1300,7 @@ jobs:
run: >
if ((Get-ChildItem -Path test-results\testoutput\*\exitcode.txt -Recurse | Select-String -Pattern '^0$' -NotMatch ).Count -gt 0) {
Get-Content -Path test-results\testoutput\*\JTreport\text\newfailures.txt ;
Get-Content -Path test-results\testoutput\*\JTreport\text\other_errors.txt ;
exit 1
}
Expand Down Expand Up @@ -1529,7 +1531,7 @@ jobs:
run: >
if cat test-results/testoutput/*/exitcode.txt | grep -q -v '^0$'
|| ! cat test-results/testoutput/*/Stats.txt | grep -q 'fail=0' ; then
cat test-results/testoutput/*/JTreport/text/newfailures.txt ;
cat test-results/testoutput/*/JTreport/text/{newfailures,other_errors}.txt ;
exit 1 ;
fi
Expand Down

1 comment on commit 27058b2

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.