Skip to content

Commit 1c2d0d0

Browse files
committed
revert integration test changes
1 parent 3d52cb7 commit 1c2d0d0

4 files changed

Lines changed: 12 additions & 43 deletions

File tree

.github/workflows/beam_PostCommit_Yaml_Xlang_Direct.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ jobs:
8080
- name: run PostCommit Yaml Xlang Direct script
8181
uses: ./.github/actions/gradle-command-self-hosted-action
8282
with:
83-
gradle-command: :sdks:python:postCommitYamlIntegrationTests -PyamlTestSet=${{ matrix.test_set }}
83+
gradle-command: :sdks:python:postCommitYamlIntegrationTests -PyamlTestSet=${{ matrix.test_set }} -PbeamPythonExtra=ml_test,yaml,transformers
8484
- name: Archive Python Test Results
8585
uses: actions/upload-artifact@v7
8686
if: failure()

.github/workflows/beam_PreCommit_Yaml_Xlang_Direct.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ jobs:
9191
- name: run PreCommit Yaml Xlang Direct script
9292
uses: ./.github/actions/gradle-command-self-hosted-action
9393
with:
94-
gradle-command: :sdks:python:yamlIntegrationTests
94+
gradle-command: :sdks:python:yamlIntegrationTests -PbeamPythonExtra=ml_test,yaml,transformers
9595
- name: Archive Python Test Results
9696
uses: actions/upload-artifact@v7
9797
if: failure()

sdks/python/build.gradle

Lines changed: 2 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -124,35 +124,10 @@ tasks.register("generateYamlDocs") {
124124
outputs.file "${buildDir}/yaml-examples.html"
125125
}
126126

127-
tasks.register("installYamlIntegrationTestDeps") {
128-
dependsOn installGcpTest
129-
doLast {
130-
exec {
131-
executable 'sh'
132-
args '-c', ". ${envdir}/bin/activate && " +
133-
"py_ver=\$(python -c 'import sys; print(f\"{sys.version_info.major}{sys.version_info.minor}\")') && " +
134-
"ml_extra=\"p\${py_ver}_ml_test\" && " +
135-
"constraint_file=\"\" && " +
136-
"if [ -f \"container/ml/py\${py_ver}/base_image_requirements.txt\" ]; then " +
137-
" constraint_file=\"container/ml/py\${py_ver}/base_image_requirements.txt\"; " +
138-
"elif [ -f \"container/py\${py_ver}/base_image_requirements.txt\" ]; then " +
139-
" constraint_file=\"container/py\${py_ver}/base_image_requirements.txt\"; " +
140-
"fi && " +
141-
"if [ -n \"\$constraint_file\" ]; then " +
142-
" echo \"Installing with constraint file: \$constraint_file\" && " +
143-
" pip install --pre --retries 10 -c \"\$constraint_file\" ${buildDir}/apache-beam.tar.gz[\$ml_extra,yaml,transformers] --extra-index-url https://download.pytorch.org/whl/cpu; " +
144-
"else " +
145-
" echo \"No constraint file found, installing without constraints\" && " +
146-
" pip install --pre --retries 10 ${buildDir}/apache-beam.tar.gz[\$ml_extra,yaml,transformers] --extra-index-url https://download.pytorch.org/whl/cpu; " +
147-
"fi"
148-
}
149-
}
150-
}
151-
152127
tasks.register("yamlIntegrationTests") {
153128
description "Runs precommit integration tests for yaml pipelines."
154129

155-
dependsOn installYamlIntegrationTestDeps
130+
dependsOn installGcpTest
156131
// Need to build all expansion services referenced in apache_beam/yaml/*.*
157132
// grep -oh 'sdk.*Jar' sdks/python/apache_beam/yaml/*.yaml | sort | uniq
158133
dependsOn ":sdks:java:extensions:schemaio-expansion-service:shadowJar"
@@ -171,7 +146,7 @@ tasks.register("yamlIntegrationTests") {
171146
tasks.register("postCommitYamlIntegrationTests") {
172147
description "Runs postcommit integration tests for yaml pipelines - parameterized by yamlTestSet."
173148

174-
dependsOn installYamlIntegrationTestDeps
149+
dependsOn installGcpTest
175150
// Need to build all expansion services referenced in apache_beam/yaml/*.*
176151
// grep -oh 'sdk.*Jar' sdks/python/apache_beam/yaml/*.yaml | sort | uniq
177152
dependsOn ":sdks:java:extensions:schemaio-expansion-service:shadowJar"

sdks/python/setup.py

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,6 @@ def cythonize(*args, **kwargs):
181181
'sentence-transformers>=2.2.2',
182182
'skl2onnx',
183183
'pyod>=0.7.6', # 0.7.5 crashes setuptools
184-
'contourpy<1.3.0; python_version < "3.11"',
185184
'tensorflow',
186185
# tensorflow transitive dep, lower versions not compatible with Python3.10+
187186
'absl-py>=0.12.0',
@@ -208,15 +207,13 @@ def cythonize(*args, **kwargs):
208207
]
209208

210209
ml_adk_dependency = [
211-
# google-adk is excluded here because it requires google-genai<2.0.0,
212-
# which conflicts with google-genai>=2.0.0 (e.g. 2.6.0) pinned in containers.
213-
# 'google-adk==1.28.1',
210+
'google-adk==1.28.1',
214211
# proto-plus<1.24 caps protobuf<5; opentelemetry-proto (via ADK) needs
215212
# protobuf>=5. Scoped here so the main dependency list stays broader.
216213
'proto-plus>=1.26.1,<2',
217-
'opentelemetry-api>=1.37.0,<2',
218-
'opentelemetry-sdk>=1.37.0,<2',
219-
'opentelemetry-exporter-otlp-proto-http>=1.37.0,<2',
214+
'opentelemetry-api==1.37.0',
215+
'opentelemetry-sdk==1.37.0',
216+
'opentelemetry-exporter-otlp-proto-http==1.37.0',
220217
# protobuf>=5 (ADK/OTel); tf2onnx 1.16.x pins protobuf~=3.20 only.
221218
'tf2onnx>=1.17.0,<1.18',
222219
]
@@ -420,7 +417,6 @@ def get_portability_package_data():
420417
ext_modules=extensions,
421418
install_requires=[
422419
'cryptography>=39.0.0,<48.0.0',
423-
'aiohttp<4.0.0',
424420
'envoy-data-plane>=1.0.3,<2; python_version >= "3.11"',
425421
# Newer version only work on Python 3.11. Versions 0.3 <= ver < 1.x
426422
# conflict with other GCP dependencies.
@@ -612,11 +608,10 @@ def get_portability_package_data():
612608
] + ml_base_core,
613609
'p310_ml_test': [
614610
'datatable',
615-
'dill',
616-
'tensorflow_transform>=1.14.0,<1.15.0',
617611
] + ml_base,
618-
'p311_ml_test': ml_base,
619-
'p312_ml_test': ml_base,
612+
'p312_ml_test': [
613+
'datatable',
614+
] + ml_base,
620615
# maintainer: milvus tests only run with this extension. Make sure it
621616
# is covered by docker-in-docker test when changing py version
622617
'p313_ml_test': ml_base + milvus_dependency,
@@ -690,8 +685,7 @@ def get_portability_package_data():
690685
'xgboost': ['xgboost>=1.6.0,<2.1.3', 'datatable==1.0.0'],
691686
'tensorflow-hub': ['tensorflow-hub>=0.14.0,<0.16.0'],
692687
'milvus': milvus_dependency,
693-
'vllm': ['openai==1.107.1', 'vllm==0.10.1.1', 'triton==3.3.1'],
694-
'adk': ['google-adk==1.28.1']
688+
'vllm': ['openai==1.107.1', 'vllm==0.10.1.1', 'triton==3.3.1']
695689
},
696690
zip_safe=False,
697691
# PyPI package information.

0 commit comments

Comments
 (0)