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

[Proposal] Automate detection and quarantine of flakey tests #2496

Open
dblock opened this issue Mar 17, 2022 · 4 comments
Open

[Proposal] Automate detection and quarantine of flakey tests #2496

dblock opened this issue Mar 17, 2022 · 4 comments
Labels
enhancement Enhancement or improvement to existing feature or request >test-failure Test failure from CI, local build, etc.

Comments

@dblock
Copy link
Member

dblock commented Mar 17, 2022

Is your feature request related to a problem? Please describe.

The amount of flakey tests (#1715) is growing and gradle checks fail roughly 50% of the time. This impedes velocity, but also reduces the trust in the tests.

Describe the solution you'd like

  1. Label flakey tests, @Ignore them, and link to a GitHub issue.
  2. Automatically detect repeated flakey test failures and link them to a GitHub issue. Automatically keep count of flakey tests.
  3. Remove flakey tests that have been flakey for over a period of time, e.g. 2 weeks, via an auto-PR.

Describe alternatives you've considered

Fix all flakey tests one-by-one.

@dblock dblock added enhancement Enhancement or improvement to existing feature or request untriaged labels Mar 17, 2022
@dblock dblock changed the title [Proposal] Automate detection and quarantine of flakey integration tests [Proposal] Automate detection and quarantine of flakey tests Mar 17, 2022
@reta
Copy link
Collaborator

reta commented Mar 17, 2022

@dblock we could also try to use Gradle's flaky test mitigation tooling, wdyt?

[1] https://blog.gradle.org/gradle-flaky-test-retry-plugin

@saratvemulapalli
Copy link
Member

@dblock love the idea. Are you picking this up or do you need help?

@saratvemulapalli saratvemulapalli added the >test-failure Test failure from CI, local build, etc. label Mar 18, 2022
@dblock
Copy link
Member Author

dblock commented Mar 18, 2022

@dblock love the idea. Are you picking this up or do you need help?

I am not picking it up, I stole the idea from a discussion with another engineer that shall remain nameless for now (will give them credit sometime in the future), so go for it!

@dblock
Copy link
Member Author

dblock commented Mar 21, 2022

@dblock we could also try to use Gradle's flaky test mitigation tooling, wdyt?

[1] https://blog.gradle.org/gradle-flaky-test-retry-plugin

#2547

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement or improvement to existing feature or request >test-failure Test failure from CI, local build, etc.
Projects
None yet
Development

No branches or pull requests

3 participants