New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BEAM-7598] Do not build Python tar file in run_integration_test.sh #8916
Conversation
Run Python PostCommit |
16bfba3
to
6262a6f
Compare
PTAL @udim |
Run Python PostCommit |
1 similar comment
Run Python PostCommit |
PTAL @udim |
+R: @tvalentyn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, Mark!
6262a6f
to
2d28efe
Compare
PTAL @tvalentyn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot, @markflyhigh !
The failure job |
Run script
run_integration_test.sh
is used by Gradle to execute Python test suite, which are running in parallel in Post/Pre-Commit. Building tar ball depends on shared temporary resources which can cause race condition, so we can't include it in this script. Since we have another Gradle tasksdist
to build the tar ball, only checkSDK_LOCATION
existence should be sufficient.This can fix PostCommit flaky in https://issues.apache.org/jira/browse/BEAM-7598.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
R: @username
).[BEAM-XXX] Fixes bug in ApproximateQuantiles
, where you replaceBEAM-XXX
with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.Post-Commit Tests Status (on master branch)
Pre-Commit Tests Status (on master branch)
See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.