Skip to content

Commit f0f8f43

Browse files
magicgoodymagicgoody
authored
Publish test results for PreCommit and PostCommit Python workflows (#29089)
* python test results publish * add postcommit python test publish step
1 parent 68fec9e commit f0f8f43

31 files changed

Lines changed: 397 additions & 180 deletions

‎.github/workflows/beam_PerformanceTests_WordCountIT_PythonVersions.yml‎

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -103,9 +103,16 @@ jobs:
103103
--info \
104104
-Ptest=apache_beam/examples/wordcount_it_test.py::WordCountIT::test_wordcount_it \
105105
"-Ptest-pipeline-options=${{ env.beam_PerformanceTests_WordCountIT_PythonVersions_test_arguments_1 }}"
106-
- name: Archive code coverage results
106+
- name: Archive Python Test Results
107107
uses: actions/upload-artifact@v3
108+
if: failure()
109+
with:
110+
name: Python Test Results
111+
path: '**/pytest*.xml'
112+
- name: Publish Python Test Results
113+
uses: EnricoMi/publish-unit-test-result-action@v2
108114
if: always()
109115
with:
110-
name: python-code-coverage-report
111-
path: "**/pytest*.xml"
116+
commit: '${{ env.prsha || env.GITHUB_SHA }}'
117+
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
118+
files: '**/pytest*.xml'

‎.github/workflows/beam_PostCommit_PortableJar_Flink.yml‎

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,9 +79,16 @@ jobs:
7979
gradle-command: :sdks:python:test-suites:portable:py38:testPipelineJarFlinkRunner
8080
arguments: |
8181
-PpythonVersion=3.8 \
82-
- name: Archive code coverage results
82+
- name: Archive Python Test Results
8383
uses: actions/upload-artifact@v3
84+
if: failure()
85+
with:
86+
name: Python Test Results
87+
path: '**/pytest*.xml'
88+
- name: Publish Python Test Results
89+
uses: EnricoMi/publish-unit-test-result-action@v2
8490
if: always()
8591
with:
86-
name: python-code-coverage-report
87-
path: "**/pytest*.xml"
92+
commit: '${{ env.prsha || env.GITHUB_SHA }}'
93+
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
94+
files: '**/pytest*.xml'

‎.github/workflows/beam_PostCommit_PortableJar_Spark.yml‎

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ on:
2323
#Setting explicit permissions for the action to avoid the default permissions which are `write-all` in case of pull_request_target event
2424
permissions:
2525
actions: write
26-
pull-requests: read
27-
checks: read
26+
pull-requests: write
27+
checks: write
2828
contents: read
2929
deployments: read
3030
id-token: none
31-
issues: read
31+
issues: write
3232
discussions: read
3333
packages: read
3434
pages: read
@@ -79,9 +79,16 @@ jobs:
7979
gradle-command: :sdks:python:test-suites:portable:py38:testPipelineJarSparkRunner
8080
arguments: |
8181
-PpythonVersion=3.8 \
82-
- name: Archive code coverage results
82+
- name: Archive Python Test Results
8383
uses: actions/upload-artifact@v3
84+
if: failure()
85+
with:
86+
name: Python Test Results
87+
path: '**/pytest*.xml'
88+
- name: Publish Python Test Results
89+
uses: EnricoMi/publish-unit-test-result-action@v2
8490
if: always()
8591
with:
86-
name: python-code-coverage-report
87-
path: "**/pytest*.xml"
92+
commit: '${{ env.prsha || env.GITHUB_SHA }}'
93+
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
94+
files: '**/pytest*.xml'

‎.github/workflows/beam_PostCommit_Python.yml‎

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,12 @@ concurrency:
3030
#Setting explicit permissions for the action to avoid the default permissions which are `write-all` in case of pull_request_target event
3131
permissions:
3232
actions: write
33-
pull-requests: read
34-
checks: read
33+
pull-requests: write
34+
checks: write
3535
contents: read
3636
deployments: read
3737
id-token: none
38-
issues: read
38+
issues: write
3939
discussions: read
4040
packages: read
4141
pages: read
@@ -94,9 +94,16 @@ jobs:
9494
-PpythonVersion=${{ matrix.python_version }} \
9595
env:
9696
CLOUDSDK_CONFIG: ${{ env.KUBELET_GCLOUD_CONFIG_PATH}}
97-
- name: Archive code coverage results
97+
- name: Archive Python Test Results
9898
uses: actions/upload-artifact@v3
99+
if: failure()
100+
with:
101+
name: Python Test Results
102+
path: '**/pytest*.xml'
103+
- name: Publish Python Test Results
104+
uses: EnricoMi/publish-unit-test-result-action@v2
99105
if: always()
100106
with:
101-
name: python-code-coverage-report
102-
path: "**/pytest*.xml"
107+
commit: '${{ env.prsha || env.GITHUB_SHA }}'
108+
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
109+
files: '**/pytest*.xml'

‎.github/workflows/beam_PostCommit_Python_Arm.yml‎

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,12 @@ concurrency:
3232
#Setting explicit permissions for the action to avoid the default permissions which are `write-all` in case of pull_request_target event
3333
permissions:
3434
actions: write
35-
pull-requests: read
36-
checks: read
35+
pull-requests: write
36+
checks: write
3737
contents: read
3838
deployments: read
3939
id-token: none
40-
issues: read
40+
issues: write
4141
discussions: read
4242
packages: read
4343
pages: read
@@ -110,9 +110,16 @@ jobs:
110110
CLOUDSDK_CONFIG: ${{ env.KUBELET_GCLOUD_CONFIG_PATH}}
111111
MULTIARCH_TAG: ${{ steps.set_tag.outputs.TAG }}
112112
USER: github-actions
113-
- name: Archive code coverage results
113+
- name: Archive Python Test Results
114114
uses: actions/upload-artifact@v3
115+
if: failure()
116+
with:
117+
name: Python Test Results
118+
path: '**/pytest*.xml'
119+
- name: Publish Python Test Results
120+
uses: EnricoMi/publish-unit-test-result-action@v2
115121
if: always()
116122
with:
117-
name: python-code-coverage-report
118-
path: "**/pytest*.xml"
123+
commit: '${{ env.prsha || env.GITHUB_SHA }}'
124+
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
125+
files: '**/pytest*.xml'

‎.github/workflows/beam_PostCommit_Python_Examples_Dataflow.yml‎

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ on:
2323
#Setting explicit permissions for the action to avoid the default permissions which are `write-all` in case of pull_request_target event
2424
permissions:
2525
actions: write
26-
pull-requests: read
27-
checks: read
26+
pull-requests: write
27+
checks: write
2828
contents: read
2929
deployments: read
3030
id-token: none
31-
issues: read
31+
issues: write
3232
discussions: read
3333
packages: read
3434
pages: read
@@ -79,9 +79,16 @@ jobs:
7979
arguments: |
8080
-PuseWheelDistribution \
8181
-PpythonVersion=3.11 \
82-
- name: Archive code coverage results
82+
- name: Archive Python Test Results
8383
uses: actions/upload-artifact@v3
84+
if: failure()
85+
with:
86+
name: Python Test Results
87+
path: '**/pytest*.xml'
88+
- name: Publish Python Test Results
89+
uses: EnricoMi/publish-unit-test-result-action@v2
8490
if: always()
8591
with:
86-
name: python-code-coverage-report
87-
path: "**/pytest*.xml"
92+
commit: '${{ env.prsha || env.GITHUB_SHA }}'
93+
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
94+
files: '**/pytest*.xml'

‎.github/workflows/beam_PostCommit_Python_Examples_Direct.yml‎

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ on:
2323
#Setting explicit permissions for the action to avoid the default permissions which are `write-all` in case of pull_request_target event
2424
permissions:
2525
actions: write
26-
pull-requests: read
27-
checks: read
26+
pull-requests: write
27+
checks: write
2828
contents: read
2929
deployments: read
3030
id-token: none
31-
issues: read
31+
issues: write
3232
discussions: read
3333
packages: read
3434
pages: read
@@ -86,9 +86,16 @@ jobs:
8686
gradle-command: :sdks:python:test-suites:direct:py${{steps.set_py_ver_clean.outputs.py_ver_clean}}:examples
8787
arguments: |
8888
-PpythonVersion=${{ matrix.python_version }} \
89-
- name: Archive code coverage results
89+
- name: Archive Python Test Results
9090
uses: actions/upload-artifact@v3
91+
if: failure()
92+
with:
93+
name: Python Test Results
94+
path: '**/pytest*.xml'
95+
- name: Publish Python Test Results
96+
uses: EnricoMi/publish-unit-test-result-action@v2
9197
if: always()
9298
with:
93-
name: python-code-coverage-report
94-
path: "**/pytest*.xml"
99+
commit: '${{ env.prsha || env.GITHUB_SHA }}'
100+
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
101+
files: '**/pytest*.xml'

‎.github/workflows/beam_PostCommit_Python_Examples_Flink.yml‎

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ on:
2323
#Setting explicit permissions for the action to avoid the default permissions which are `write-all` in case of pull_request_target event
2424
permissions:
2525
actions: write
26-
pull-requests: read
27-
checks: read
26+
pull-requests: write
27+
checks: write
2828
contents: read
2929
deployments: read
3030
id-token: none
31-
issues: read
31+
issues: write
3232
discussions: read
3333
packages: read
3434
pages: read
@@ -86,9 +86,16 @@ jobs:
8686
gradle-command: :sdks:python:test-suites:portable:py${{steps.set_py_ver_clean.outputs.py_ver_clean}}:flinkExamples
8787
arguments: |
8888
-PpythonVersion=${{ matrix.python_version }} \
89-
- name: Archive code coverage results
89+
- name: Archive Python Test Results
9090
uses: actions/upload-artifact@v3
91+
if: failure()
92+
with:
93+
name: Python Test Results
94+
path: '**/pytest*.xml'
95+
- name: Publish Python Test Results
96+
uses: EnricoMi/publish-unit-test-result-action@v2
9197
if: always()
9298
with:
93-
name: python-code-coverage-report
94-
path: "**/pytest*.xml"
99+
commit: '${{ env.prsha || env.GITHUB_SHA }}'
100+
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
101+
files: '**/pytest*.xml'

‎.github/workflows/beam_PostCommit_Python_Examples_Spark.yml‎

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ on:
2323
#Setting explicit permissions for the action to avoid the default permissions which are `write-all` in case of pull_request_target event
2424
permissions:
2525
actions: write
26-
pull-requests: read
27-
checks: read
26+
pull-requests: write
27+
checks: write
2828
contents: read
2929
deployments: read
3030
id-token: none
31-
issues: read
31+
issues: write
3232
discussions: read
3333
packages: read
3434
pages: read
@@ -86,9 +86,16 @@ jobs:
8686
gradle-command: :sdks:python:test-suites:portable:py${{steps.set_py_ver_clean.outputs.py_ver_clean}}:sparkExamples
8787
arguments: |
8888
-PpythonVersion=${{ matrix.python_version }} \
89-
- name: Archive code coverage results
89+
- name: Archive Python Test Results
9090
uses: actions/upload-artifact@v3
91+
if: failure()
92+
with:
93+
name: Python Test Results
94+
path: '**/pytest*.xml'
95+
- name: Publish Python Test Results
96+
uses: EnricoMi/publish-unit-test-result-action@v2
9197
if: always()
9298
with:
93-
name: python-code-coverage-report
94-
path: "**/pytest*.xml"
99+
commit: '${{ env.prsha || env.GITHUB_SHA }}'
100+
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
101+
files: '**/pytest*.xml'

‎.github/workflows/beam_PostCommit_Python_MongoDBIO_IT.yml‎

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ on:
2323
#Setting explicit permissions for the action to avoid the default permissions which are `write-all` in case of pull_request_target event
2424
permissions:
2525
actions: write
26-
pull-requests: read
27-
checks: read
26+
pull-requests: write
27+
checks: write
2828
contents: read
2929
deployments: read
3030
id-token: none
31-
issues: read
31+
issues: write
3232
discussions: read
3333
packages: read
3434
pages: read
@@ -78,9 +78,16 @@ jobs:
7878
gradle-command: :sdks:python:test-suites:direct:py311:mongodbioIT
7979
arguments: |
8080
-PpythonVersion=3.11 \
81-
- name: Archive code coverage results
81+
- name: Archive Python Test Results
8282
uses: actions/upload-artifact@v3
83+
if: failure()
84+
with:
85+
name: Python Test Results
86+
path: '**/pytest*.xml'
87+
- name: Publish Python Test Results
88+
uses: EnricoMi/publish-unit-test-result-action@v2
8389
if: always()
8490
with:
85-
name: python-code-coverage-report
86-
path: "**/pytest*.xml"
91+
commit: '${{ env.prsha || env.GITHUB_SHA }}'
92+
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
93+
files: '**/pytest*.xml'

0 commit comments

Comments
 (0)