stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8284507: GHA: Only check test results if testing was not skipped

Browse Source 
Reviewed-by: shade, ihse
This commit is contained in:
Christoph Langer 2022-04-08 13:29:40 +00:00
parent 6028181071
commit 8eac3427b1
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
.github/workflows/submit.yml vendored
View File

@ -343,6 +343,7 @@ jobs:
echo "imageroot=`dirname ${imageroot}`" >> $GITHUB_ENV echo "imageroot=`dirname ${imageroot}`" >> $GITHUB_ENV
- name: Run tests - name: Run tests
id: run_tests
run: > run: >
JDK_IMAGE_DIR=${{ env.imageroot }} JDK_IMAGE_DIR=${{ env.imageroot }}
TEST_IMAGE_DIR=${HOME}/jdk-linux-x64${{ matrix.artifact }}/jdk-${{ env.JDK_VERSION }}-internal_linux-x64_bin-tests${{ matrix.artifact }} TEST_IMAGE_DIR=${HOME}/jdk-linux-x64${{ matrix.artifact }}/jdk-${{ env.JDK_VERSION }}-internal_linux-x64_bin-tests${{ matrix.artifact }}
@ -358,7 +359,7 @@ jobs:
JTREG="JAVA_OPTIONS=-XX:-CreateCoredumpOnCrash" JTREG="JAVA_OPTIONS=-XX:-CreateCoredumpOnCrash"
- name: Check that all tests executed successfully - name: Check that all tests executed successfully
if: always() if: steps.run_tests.outcome != 'skipped'
run: > run: >
if ! grep --include=test-summary.txt -lqr build/*/test-results -e "TEST SUCCESS" ; then if ! grep --include=test-summary.txt -lqr build/*/test-results -e "TEST SUCCESS" ; then
cat build/*/test-results/*/text/newfailures.txt ; cat build/*/test-results/*/text/newfailures.txt ;
@ -808,6 +809,7 @@ jobs:
echo "imageroot=`dirname ${imageroot}`" >> $GITHUB_ENV echo "imageroot=`dirname ${imageroot}`" >> $GITHUB_ENV
- name: Run tests - name: Run tests
id: run_tests
run: > run: >
JDK_IMAGE_DIR=${{ env.imageroot }} JDK_IMAGE_DIR=${{ env.imageroot }}
TEST_IMAGE_DIR=${HOME}/jdk-linux-x86${{ matrix.artifact }}/jdk-${{ env.JDK_VERSION }}-internal_linux-x86_bin-tests${{ matrix.artifact }} TEST_IMAGE_DIR=${HOME}/jdk-linux-x86${{ matrix.artifact }}/jdk-${{ env.JDK_VERSION }}-internal_linux-x86_bin-tests${{ matrix.artifact }}
@ -823,7 +825,7 @@ jobs:
JTREG="JAVA_OPTIONS=-XX:-CreateCoredumpOnCrash" JTREG="JAVA_OPTIONS=-XX:-CreateCoredumpOnCrash"
- name: Check that all tests executed successfully - name: Check that all tests executed successfully
if: always() if: steps.run_tests.outcome != 'skipped'
run: > run: >
if ! grep --include=test-summary.txt -lqr build/*/test-results -e "TEST SUCCESS" ; then if ! grep --include=test-summary.txt -lqr build/*/test-results -e "TEST SUCCESS" ; then
cat build/*/test-results/*/text/newfailures.txt ; cat build/*/test-results/*/text/newfailures.txt ;
@ -1248,6 +1250,7 @@ jobs:
run: echo ("imageroot=" + (Get-ChildItem -Path $HOME/jdk-windows-x64${{ matrix.artifact }}/jdk-${{ env.JDK_VERSION }}-internal_windows-x64_bin${{ matrix.artifact }} -Filter release -Recurse -ErrorAction SilentlyContinue -Force).DirectoryName) | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8 run: echo ("imageroot=" + (Get-ChildItem -Path $HOME/jdk-windows-x64${{ matrix.artifact }}/jdk-${{ env.JDK_VERSION }}-internal_windows-x64_bin${{ matrix.artifact }} -Filter release -Recurse -ErrorAction SilentlyContinue -Force).DirectoryName) | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8
- name: Run tests - name: Run tests
id: run_tests
run: > run: >
$env:Path = "$HOME\cygwin\cygwin64\bin;$HOME\cygwin\cygwin64\bin;$env:Path" ; $env:Path = "$HOME\cygwin\cygwin64\bin;$HOME\cygwin\cygwin64\bin;$env:Path" ;
$env:Path = $env:Path -split ";" -match "C:\\Windows|PowerShell|cygwin" -join ";" ; $env:Path = $env:Path -split ";" -match "C:\\Windows|PowerShell|cygwin" -join ";" ;
@ -1266,7 +1269,7 @@ jobs:
JTREG="JAVA_OPTIONS=-XX:-CreateCoredumpOnCrash" JTREG="JAVA_OPTIONS=-XX:-CreateCoredumpOnCrash"
- name: Check that all tests executed successfully - name: Check that all tests executed successfully
if: always() if: steps.run_tests.outcome != 'skipped'
run: > run: >
if ((Get-ChildItem -Path build\*\test-results\test-summary.txt -Recurse | Select-String -Pattern "TEST SUCCESS" ).Count -eq 0) { if ((Get-ChildItem -Path build\*\test-results\test-summary.txt -Recurse | Select-String -Pattern "TEST SUCCESS" ).Count -eq 0) {
Get-Content -Path build\*\test-results\*\*\newfailures.txt ; Get-Content -Path build\*\test-results\*\*\newfailures.txt ;
@ -1643,6 +1646,7 @@ jobs:
echo "imageroot=`dirname ${imageroot}`" >> $GITHUB_ENV echo "imageroot=`dirname ${imageroot}`" >> $GITHUB_ENV
- name: Run tests - name: Run tests
id: run_tests
run: > run: >
JDK_IMAGE_DIR=${{ env.imageroot }} JDK_IMAGE_DIR=${{ env.imageroot }}
TEST_IMAGE_DIR=${HOME}/jdk-macos-x64${{ matrix.artifact }}/jdk-${{ env.JDK_VERSION }}-internal_macos-x64_bin-tests${{ matrix.artifact }} TEST_IMAGE_DIR=${HOME}/jdk-macos-x64${{ matrix.artifact }}/jdk-${{ env.JDK_VERSION }}-internal_macos-x64_bin-tests${{ matrix.artifact }}
@ -1658,7 +1662,7 @@ jobs:
JTREG="JAVA_OPTIONS=-XX:-CreateCoredumpOnCrash" JTREG="JAVA_OPTIONS=-XX:-CreateCoredumpOnCrash"
- name: Check that all tests executed successfully - name: Check that all tests executed successfully
if: always() if: steps.run_tests.outcome != 'skipped'
run: > run: >
if ! grep --include=test-summary.txt -lqr build/*/test-results -e "TEST SUCCESS" ; then if ! grep --include=test-summary.txt -lqr build/*/test-results -e "TEST SUCCESS" ; then
cat build/*/test-results/*/text/newfailures.txt ; cat build/*/test-results/*/text/newfailures.txt ;