Skip to content

Commit e8d4395

Browse files
authored
Partially revert #35598 (#35600)
1 parent 6a4ff71 commit e8d4395

2 files changed

Lines changed: 3 additions & 24 deletions

File tree

.github/workflows/beam_PostCommit_Python.yml

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -54,20 +54,14 @@ env:
5454
jobs:
5555
beam_PostCommit_Python:
5656
name: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }})
57-
runs-on: ${{ matrix.os }}
57+
runs-on: [self-hosted, ubuntu-20.04, highmem22]
5858
timeout-minutes: 240
5959
strategy:
6060
fail-fast: false
6161
matrix:
6262
job_name: [beam_PostCommit_Python]
6363
job_phrase: [Run Python PostCommit]
6464
python_version: ['3.9', '3.10', '3.11', '3.12']
65-
# Run on both self-hosted and GitHub-hosted runners.
66-
# Some tests (marked require_docker_in_docker) can't run on Beam's
67-
# self-hosted runners due to Docker-in-Docker environment constraint.
68-
# These tests will only execute on ubuntu-latest (GitHub-hosted).
69-
# Context: https://github.com/apache/beam/pull/35585
70-
os: [[self-hosted, ubuntu-20.04, highmem22], [ubuntu-latest]]
7165
if: |
7266
github.event_name == 'workflow_dispatch' ||
7367
github.event_name == 'pull_request_target' ||
@@ -105,11 +99,6 @@ jobs:
10599
arguments: |
106100
-Pjava21Home=$JAVA_HOME_21_X64 \
107101
-PuseWheelDistribution \
108-
-Pposargs="${{
109-
contains(matrix.os, 'self-hosted') &&
110-
'-m ''not require_docker_in_docker''' ||
111-
'-m ''require_docker_in_docker'''
112-
}}" \
113102
-PpythonVersion=${{ matrix.python_version }} \
114103
env:
115104
CLOUDSDK_CONFIG: ${{ env.KUBELET_GCLOUD_CONFIG_PATH}}

.github/workflows/beam_PreCommit_Python_ML.yml

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -58,20 +58,14 @@ env:
5858
jobs:
5959
beam_PreCommit_Python_ML:
6060
name: ${{ matrix.job_name }} (${{ matrix.job_phrase }} ${{ matrix.python_version }})
61-
runs-on: ${{ matrix.os }}
61+
runs-on: [self-hosted, ubuntu-20.04, main]
6262
timeout-minutes: 180
6363
strategy:
6464
fail-fast: false
6565
matrix:
6666
job_name: ['beam_PreCommit_Python_ML']
6767
job_phrase: ['Run Python_ML PreCommit']
6868
python_version: ['3.9','3.10','3.11','3.12']
69-
# Run on both self-hosted and GitHub-hosted runners.
70-
# Some tests (marked require_docker_in_docker) can't run on Beam's
71-
# self-hosted runners due to Docker-in-Docker environment constraint.
72-
# These tests will only execute on ubuntu-latest (GitHub-hosted).
73-
# Context: https://github.com/apache/beam/pull/35585
74-
os: [[self-hosted, ubuntu-20.04, main], [ubuntu-latest]]
7569
if: |
7670
github.event_name == 'push' ||
7771
github.event_name == 'pull_request_target' ||
@@ -102,11 +96,7 @@ jobs:
10296
with:
10397
gradle-command: :sdks:python:test-suites:tox:py${{steps.set_py_ver_clean.outputs.py_ver_clean}}:testPy${{steps.set_py_ver_clean.outputs.py_ver_clean}}ML
10498
arguments: |
105-
-Pposargs="${{
106-
contains(matrix.os, 'self-hosted') &&
107-
'apache_beam/ml/ -m ''not require_docker_in_docker''' ||
108-
'apache_beam/ml/ -m ''require_docker_in_docker'''
109-
}}" \
99+
-Pposargs=apache_beam/ml/ \
110100
-PpythonVersion=${{ matrix.python_version }}
111101
- name: Archive Python Test Results
112102
uses: actions/upload-artifact@v4

0 commit comments

Comments
 (0)