Skip to content

Commit cc9f74c

Browse files
committed
test-suites/direct/common.gradle: remove milvus postcommit gradle hook
1 parent 1ff0f56 commit cc9f74c

1 file changed

Lines changed: 0 additions & 28 deletions

File tree

sdks/python/test-suites/direct/common.gradle

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -419,33 +419,6 @@ task feastIntegrationTest {
419419
}
420420
}
421421

422-
// Integration tests that require Docker-in-Docker capabilities.
423-
// These tests are marked with the `require_docker_in_docker` pytest marker
424-
// because they rely on Docker-in-Docker configurations that are not supported
425-
// on Beam's self-hosted GitHub Actions runners. Docker-in-Docker works on
426-
// ubuntu-latest GitHub-hosted runners but not on self-hosted environments due
427-
// to containerization architecture differences.
428-
// Context: https://github.com/apache/beam/pull/35585
429-
task dockerInDockerIntegrationTest {
430-
dependsOn 'installGcpTest'
431-
dependsOn ':sdks:python:sdist'
432-
433-
doLast {
434-
def testOpts = basicTestOpts
435-
def argMap = [
436-
"test_opts": testOpts,
437-
"suite": "postCommitIT-direct-py${pythonVersionSuffix}",
438-
"collect": "require_docker_in_docker",
439-
"runner": "TestDirectRunner",
440-
]
441-
def cmdArgs = mapToArgString(argMap)
442-
exec {
443-
executable 'sh'
444-
args '-c', ". ${envdir}/bin/activate && ${runScriptsDir}/run_integration_test.sh $cmdArgs"
445-
}
446-
}
447-
}
448-
449422
// Add all the RunInference framework IT tests to this gradle task that runs on Direct Runner Post commit suite.
450423
project.tasks.register("inferencePostCommitIT") {
451424
dependsOn = [
@@ -456,7 +429,6 @@ project.tasks.register("inferencePostCommitIT") {
456429
'transformersInferenceTest',
457430
'testcontainersTest',
458431
'feastIntegrationTest',
459-
'dockerInDockerIntegrationTest',
460432
// (TODO) https://github.com/apache/beam/issues/25799
461433
// uncomment tfx bsl tests once tfx supports protobuf 4.x
462434
// 'tfxInferenceTest',

0 commit comments

Comments
 (0)