Skip to content

Commit ea7c886

Browse files
workflows: properly show job names
1 parent 76c77df commit ea7c886

File tree

2 files changed

+12
-14
lines changed

2 files changed

+12
-14
lines changed

.github/workflows/beam_PostCommit_Python.yml

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -53,21 +53,20 @@ env:
5353

5454
jobs:
5555
beam_PostCommit_Python:
56-
name: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }}) [${{ matrix.os }}]
56+
name: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }}) (${{ join(matrix.os, ', ') }})
5757
runs-on: ${{ matrix.os }}
5858
timeout-minutes: 240
5959
strategy:
6060
fail-fast: false
6161
matrix:
62-
job_name: [beam_PostCommit_Python]
63-
job_phrase: [Run Python PostCommit]
62+
job_name: ['beam_PostCommit_Python']
63+
job_phrase: ['Run Python PostCommit']
6464
python_version: ['3.9', '3.10', '3.11', '3.12']
6565
# Run on both self-hosted and GitHub-hosted runners.
6666
# Some tests (marked require_docker_in_docker) can't run on Beam's
6767
# self-hosted runners due to Docker-in-Docker environment constraint.
6868
# These tests will only execute on ubuntu-latest (GitHub-hosted).
69-
# Context: https://github.com/apache/beam/pull/35585
70-
# Temporary removed the ubuntu-latest env till resolving deps issues.
69+
# Context: https://github.com/apache/beam/pull/35585.
7170
os: [[self-hosted, ubuntu-20.04, highmem22], [ubuntu-latest]]
7271
if: |
7372
github.event_name == 'workflow_dispatch' ||
@@ -81,7 +80,7 @@ jobs:
8180
with:
8281
comment_phrase: ${{ matrix.job_phrase }} ${{ matrix.python_version }}
8382
github_token: ${{ secrets.GITHUB_TOKEN }}
84-
github_job: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }})
83+
github_job: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }}) (${{ join(matrix.os, ', ') }})
8584
- name: Setup environment
8685
uses: ./.github/actions/setup-environment-action
8786
with:
@@ -118,7 +117,7 @@ jobs:
118117
uses: actions/upload-artifact@v4
119118
if: failure()
120119
with:
121-
name: Python ${{ matrix.python_version }} Test Results [${{ matrix.os }}]
120+
name: Python ${{ matrix.python_version }} Test Results (${{ join(matrix.os, ', ') }})
122121
path: '**/pytest*.xml'
123122
- name: Publish Python Test Results
124123
uses: EnricoMi/publish-unit-test-result-action@v2
@@ -128,4 +127,4 @@ jobs:
128127
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
129128
files: '**/pytest*.xml'
130129
large_files: true
131-
check_name: 'Python ${{ matrix.python_version }} Test Results [${{ matrix.os }}]'
130+
check_name: "Python ${{ matrix.python_version }} Test Results (${{ join(matrix.os, ', ') }})"

.github/workflows/beam_PreCommit_Python_ML.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ env:
5757

5858
jobs:
5959
beam_PreCommit_Python_ML:
60-
name: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }}) [${{ matrix.os }}]
60+
name: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }}) (${{ join(matrix.os, ', ') }})
6161
runs-on: ${{ matrix.os }}
6262
timeout-minutes: 180
6363
strategy:
@@ -70,8 +70,7 @@ jobs:
7070
# Some tests (marked require_docker_in_docker) can't run on Beam's
7171
# self-hosted runners due to Docker-in-Docker environment constraint.
7272
# These tests will only execute on ubuntu-latest (GitHub-hosted).
73-
# Context: https://github.com/apache/beam/pull/35585
74-
# Temporary removed the ubuntu-latest env till resolving deps issues.
73+
# Context: https://github.com/apache/beam/pull/35585.
7574
os: [[self-hosted, ubuntu-20.04, main], [ubuntu-latest]]
7675
if: |
7776
github.event_name == 'push' ||
@@ -86,7 +85,7 @@ jobs:
8685
with:
8786
comment_phrase: ${{ matrix.job_phrase }} ${{ matrix.python_version }}
8887
github_token: ${{ secrets.GITHUB_TOKEN }}
89-
github_job: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }})
88+
github_job: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }}) (${{ join(matrix.os, ', ') }})
9089
- name: Setup environment
9190
uses: ./.github/actions/setup-environment-action
9291
with:
@@ -113,7 +112,7 @@ jobs:
113112
uses: actions/upload-artifact@v4
114113
if: failure()
115114
with:
116-
name: Python ${{ matrix.python_version }} Test Results [${{ matrix.os }}]
115+
name: Python ${{ matrix.python_version }} Test Results ${{ matrix.os }}
117116
path: '**/pytest*.xml'
118117
- name: Publish Python Test Results
119118
uses: EnricoMi/publish-unit-test-result-action@v2
@@ -123,4 +122,4 @@ jobs:
123122
comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
124123
files: '**/pytest*.xml'
125124
large_files: true
126-
check_name: 'Python ${{ matrix.python_version }} Test Results [${{ matrix.os }}]'
125+
check_name: "Python ${{ matrix.python_version }} Test Results (${{ join(matrix.os, ', ') }})"

0 commit comments

Comments
 (0)