Skip to content

Commit 06d7fab

Browse files
update
1 parent 76c77df commit 06d7fab

File tree

2 files changed

+17
-21
lines changed

2 files changed

+17
-21
lines changed

.github/workflows/beam_PostCommit_Python.yml

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ name: PostCommit Python
2020
on:
2121
schedule:
2222
- cron: '30 5/6 * * *'
23-
pull_request_target:
23+
pull_request:
2424
paths: ['release/trigger_all_tests.json', '.github/trigger_files/beam_PostCommit_Python.json']
2525
workflow_dispatch:
2626

@@ -53,27 +53,25 @@ env:
5353

5454
jobs:
5555
beam_PostCommit_Python:
56-
name: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }}) [${{ matrix.os }}]
56+
name: 'beam_PostCommit_Python'
5757
runs-on: ${{ matrix.os }}
5858
timeout-minutes: 240
5959
strategy:
6060
fail-fast: false
6161
matrix:
62-
job_name: [beam_PostCommit_Python]
6362
job_phrase: [Run Python PostCommit]
6463
python_version: ['3.9', '3.10', '3.11', '3.12']
6564
# Run on both self-hosted and GitHub-hosted runners.
6665
# Some tests (marked require_docker_in_docker) can't run on Beam's
6766
# self-hosted runners due to Docker-in-Docker environment constraint.
6867
# 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.
68+
# Context: https://github.com/apache/beam/pull/35585.
7169
os: [[self-hosted, ubuntu-20.04, highmem22], [ubuntu-latest]]
72-
if: |
73-
github.event_name == 'workflow_dispatch' ||
74-
github.event_name == 'pull_request_target' ||
75-
(github.event_name == 'schedule' && github.repository == 'apache/beam') ||
76-
startswith(github.event.comment.body, 'Run Python PostCommit 3.')
70+
# if: |
71+
# github.event_name == 'workflow_dispatch' ||
72+
# github.event_name == 'pull_request_target' ||
73+
# (github.event_name == 'schedule' && github.repository == 'apache/beam') ||
74+
# startswith(github.event.comment.body, 'Run Python PostCommit 3.')
7775
steps:
7876
- uses: actions/checkout@v4
7977
- name: Setup repository

.github/workflows/beam_PreCommit_Python_ML.yml

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515

1616
name: PreCommit Python ML tests with ML deps installed
1717
on:
18-
pull_request_target:
18+
pull_request:
1919
branches: [ "master", "release-*" ]
2020
paths: [ "model/**","sdks/python/**","release/**", 'release/trigger_all_tests.json', '.github/trigger_files/beam_PreCommit_Python_ML.json']
2121
issue_comment:
@@ -57,28 +57,26 @@ env:
5757

5858
jobs:
5959
beam_PreCommit_Python_ML:
60-
name: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }}) [${{ matrix.os }}]
60+
name: 'beam_PreCommit_Python_ML'
6161
runs-on: ${{ matrix.os }}
6262
timeout-minutes: 180
6363
strategy:
6464
fail-fast: false
6565
matrix:
66-
job_name: ['beam_PreCommit_Python_ML']
6766
job_phrase: ['Run Python_ML PreCommit']
6867
python_version: ['3.9','3.10','3.11','3.12']
6968
# Run on both self-hosted and GitHub-hosted runners.
7069
# Some tests (marked require_docker_in_docker) can't run on Beam's
7170
# self-hosted runners due to Docker-in-Docker environment constraint.
7271
# 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.
72+
# Context: https://github.com/apache/beam/pull/35585.
7573
os: [[self-hosted, ubuntu-20.04, main], [ubuntu-latest]]
76-
if: |
77-
github.event_name == 'push' ||
78-
github.event_name == 'pull_request_target' ||
79-
(github.event_name == 'schedule' && github.repository == 'apache/beam') ||
80-
github.event_name == 'workflow_dispatch' ||
81-
startsWith(github.event.comment.body, 'Run Python_ML PreCommit')
74+
# if: |
75+
# github.event_name == 'push' ||
76+
# github.event_name == 'pull_request_target' ||
77+
# (github.event_name == 'schedule' && github.repository == 'apache/beam') ||
78+
# github.event_name == 'workflow_dispatch' ||
79+
# startsWith(github.event.comment.body, 'Run Python_ML PreCommit')
8280
steps:
8381
- uses: actions/checkout@v4
8482
- name: Setup repository

0 commit comments

Comments
 (0)