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

ci: Add /retest option to rerun only failed test #32425

Closed
learnitall opened this issue May 8, 2024 · 2 comments
Closed

ci: Add /retest option to rerun only failed test #32425

learnitall opened this issue May 8, 2024 · 2 comments
Assignees
Labels
area/CI Continuous Integration testing issue or flake area/CI-improvement Topic or proposal to improve the Continuous Integration workflow

Comments

@learnitall
Copy link
Contributor

To help address cases where flaky test are blocking a PR, we should add a /retest command which only reruns failed tests. This is a better alternative to consistently using /test which reruns all tests, introducing the probability to come across new flakes.

@learnitall learnitall added area/CI Continuous Integration testing issue or flake area/CI-improvement Topic or proposal to improve the Continuous Integration workflow labels May 8, 2024
@brlbil
Copy link
Contributor

brlbil commented May 10, 2024

I am working on an implementation that runs tests as normal with the /test command and re-runs only the failed test if the /test command is issued again. If the commit SHA changes the /test runs all the tests again.

@brlbil
Copy link
Contributor

brlbil commented May 22, 2024

This future is implemented and in use now.

@brlbil brlbil closed this as completed May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake area/CI-improvement Topic or proposal to improve the Continuous Integration workflow
Projects
None yet
Development

No branches or pull requests

2 participants