Abacn commented on code in PR #24204:
URL: https://github.com/apache/beam/pull/24204#discussion_r1058662550
##########
.test-infra/jenkins/job_PreCommit_Python.groovy:
##########
@@ -22,6 +22,9 @@ PrecommitJobBuilder builder = new PrecommitJobBuilder(
scope: this,
nameBase: 'Python',
gradleTask: ':pythonPreCommit',
+ gradleSwitches: [
Review Comment:
Also tested on my draft pr
(https://ci-beam.apache.org/job/beam_PreCommit_Python_Commit/26409/) and found
understand the obstacle of using `--ignore`. Although run_pytest.sh has
pytest_args in addition to posargs, tox only has posargs that is passed from
gradle script. Currently, feed `--ignore` in posargs will get this error:
-----
ERROR: module or package not found: '--ignore=apache_beam/coders (missing
__init__.py?)
ERROR: InvocationError for command
/home/jenkins/jenkins-slave/workspace/beam_PreCommit_Python_Commit/src/sdks/python/test-suites/tox/py37/build/srcs/sdks/python/scripts/run_pytest.sh
py37-cython ''"'"'--ignore=apache_beam/coders --ignore=apache_beam/dataframe
--ignore=apache_beam/examples --ignore=apache_beam/internal
--ignore=apache_beam/io --ignore=apache_beam/metrics
--ignore=apache_beam/ml'"'"'' (exited with code 4)
-----
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
