Skip to content

Commit

Permalink
Fix hdfs integration test (#30458)
Browse files Browse the repository at this point in the history
* Fix hdfs integration test

debug only

skip yaml test

* Add back azureIntegrationTest get removed when teardown Py37 support
  • Loading branch information
Abacn authored Mar 6, 2024
1 parent f0f1cd3 commit 8d7adba
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 4 deletions.
Empty file.
3 changes: 1 addition & 2 deletions build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -504,6 +504,7 @@ tasks.register("python38PostCommit") {
dependsOn(":sdks:python:test-suites:dataflow:py38:postCommitIT")
dependsOn(":sdks:python:test-suites:direct:py38:postCommitIT")
dependsOn(":sdks:python:test-suites:direct:py38:hdfsIntegrationTest")
dependsOn(":sdks:python:test-suites:direct:py38:azureIntegrationTest")
dependsOn(":sdks:python:test-suites:portable:py38:postCommitPy38")
// TODO: https://github.com/apache/beam/issues/22651
// The default container uses Python 3.8. The goal here is to
Expand All @@ -516,7 +517,6 @@ tasks.register("python38PostCommit") {
tasks.register("python39PostCommit") {
dependsOn(":sdks:python:test-suites:dataflow:py39:postCommitIT")
dependsOn(":sdks:python:test-suites:direct:py39:postCommitIT")
dependsOn(":sdks:python:test-suites:direct:py39:hdfsIntegrationTest")
dependsOn(":sdks:python:test-suites:portable:py39:postCommitPy39")
// TODO (https://github.com/apache/beam/issues/23966)
// Move this to Python 3.10 test suite once tfx-bsl has python 3.10 wheel.
Expand All @@ -526,7 +526,6 @@ tasks.register("python39PostCommit") {
tasks.register("python310PostCommit") {
dependsOn(":sdks:python:test-suites:dataflow:py310:postCommitIT")
dependsOn(":sdks:python:test-suites:direct:py310:postCommitIT")
dependsOn(":sdks:python:test-suites:direct:py310:hdfsIntegrationTest")
dependsOn(":sdks:python:test-suites:portable:py310:postCommitPy310")
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ version: '3'
services:
# HDFS namenode.
namenode:
image: gcr.io/apache-beam-testing/uhopper-hadoop/hadoop-namenode:2.8.1
image: gcr.io/apache-beam-testing/uhopper-hadoop/hadoop-namenode:2.10.2
hostname: namenode
environment:
- CLUSTER_NAME=test
Expand All @@ -30,7 +30,7 @@ services:

# HDFS datanode.
datanode:
image: gcr.io/apache-beam-testing/uhopper-hadoop/hadoop-datanode:2.8.1
image: gcr.io/apache-beam-testing/uhopper-hadoop/hadoop-datanode:2.10.2
hostname: datanode
environment:
- CLUSTER_NAME=test
Expand Down
6 changes: 6 additions & 0 deletions sdks/python/test-suites/direct/common.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -182,6 +182,9 @@ tasks.register("directRunnerIT") {
}

tasks.register("hdfsIntegrationTest") {
// needs generated srcs in sdks/python
dependsOn ':sdks:python:sdist'

doLast {
exec {
executable 'sh'
Expand All @@ -191,6 +194,9 @@ tasks.register("hdfsIntegrationTest") {
}

tasks.register("azureIntegrationTest") {
// needs generated srcs in sdks/python
dependsOn ':sdks:python:sdist'

doLast {
exec {
executable 'sh'
Expand Down

0 comments on commit 8d7adba

Please sign in to comment.