Skip to content
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

[Failing Test]: Tests for Flink runners are failing in "Java PreCommit check" #26785

Closed
2 of 15 tasks
TSultanov opened this issue May 19, 2023 · 3 comments
Closed
2 of 15 tasks

Comments

@TSultanov
Copy link
Contributor

What happened?

I'm noticing that "Java PreCommit check" job is failing in several of my PRs, some of which are not related to any Java code.

The failures are all seem to be related to Flink runner (:runners:flink:...:test jobs). I've tried running these tests locally, but they all seem to pass fine.

PRs with failing "Java PreCommit check":

PR Link Jenkins Link
#26762 https://ci-beam.apache.org/job/beam_PreCommit_Java_Phrase/6094/
#26488 https://ci-beam.apache.org/job/beam_PreCommit_Java_Commit/27220
#26713 https://ci-beam.apache.org/job/beam_PreCommit_Java_Commit/27221/

Issue Failure

Failure: Test is continually failing

Issue Priority

Priority: 1 (unhealthy code / failing or flaky postcommit so we cannot be sure the product is healthy)

Issue Components

  • Component: Python SDK
  • Component: Java SDK
  • Component: Go SDK
  • Component: Typescript SDK
  • Component: IO connector
  • Component: Beam examples
  • Component: Beam playground
  • Component: Beam katas
  • Component: Website
  • Component: Spark Runner
  • Component: Flink Runner
  • Component: Samza Runner
  • Component: Twister2 Runner
  • Component: Hazelcast Jet Runner
  • Component: Google Cloud Dataflow Runner
@Abacn
Copy link
Contributor

Abacn commented May 19, 2023

This is known flake (#21333); this flink tests each setup a flink minicluster, and when jenkins node under high load it then stucks. A recent build system change worsened the issue: #26776 should be partly mitigated after #26784 in.

@Abacn
Copy link
Contributor

Abacn commented May 19, 2023

Also, we could separate flink runner tests of version matrices into another precommit to reduce the noise. That is, he current java precommit runs on a single flink version; while other versions in another precommit that is only triggered when runner code change.

@Abacn
Copy link
Contributor

Abacn commented May 23, 2023

close as fixed by #26784. Recent java precommit cron jobs are pretty green now

@Abacn Abacn closed this as completed May 23, 2023
@github-actions github-actions bot added this to the 2.49.0 Release milestone May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants