Skip to content

Conversation

@LevKats
Copy link
Contributor

@LevKats LevKats commented Nov 4, 2025

Add new ci workflow running all files changed in the PR commits multiple
times in parallel by default. Can be disabled by setting the noflaky
PR label.

Co-authored-by: Nikita Zheleztsov n.zheleztsov@proton.me

Fixes #531

NO_DOC=ci
NO_TEST=ci

@LevKats LevKats self-assigned this Nov 4, 2025
@LevKats LevKats added the full-ci label Nov 4, 2025
@LevKats LevKats force-pushed the bump-test-run branch 4 times, most recently from c51d15a to 353cfab Compare November 4, 2025 20:38
@LevKats LevKats force-pushed the bump-test-run branch 2 times, most recently from d8bcd69 to 116956d Compare November 12, 2025 07:22
@LevKats LevKats changed the title test: bump test-run to aaba736 ci: add flaky test catching workflow Nov 12, 2025
@LevKats LevKats added the do not merge Not ready to be merged label Nov 12, 2025
@LevKats LevKats marked this pull request as ready for review November 12, 2025 07:25
@LevKats LevKats requested a review from Serpentian November 12, 2025 07:39
Copy link
Collaborator

@Serpentian Serpentian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you a lot!

@Serpentian Serpentian removed their assignment Nov 12, 2025
Bump test-run so we can use the `--repeat` flag.

Part of tarantool#531

NO_DOC=bump
NO_TEST=bump
Add new ci workflow running all files changed in the PR commits multiple
times in parallel by default. Can be disabled by setting the noflaky
PR label.

Co-authored-by: Nikita Zheleztsov <n.zheleztsov@proton.me>

Fixes tarantool#531

NO_DOC=ci
NO_TEST=ci
Copy link
Collaborator

@Serpentian Serpentian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! That's great, that we finally can catch flaky tests, in vshard they're very frequent.

P.S. @Gerold103, please, don't merge until we drop the last 2 commits, they're for testing the CI and should not be merged

@Serpentian Serpentian assigned Gerold103 and unassigned Serpentian Nov 18, 2025
@Serpentian Serpentian requested a review from Gerold103 November 18, 2025 10:06
Copy link
Collaborator

@Gerold103 Gerold103 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@Serpentian Serpentian merged commit 1f4c2cc into tarantool:master Nov 19, 2025
10 checks passed
@LevKats LevKats removed the do not merge Not ready to be merged label Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Introduce workflow for checking the test flakiness

3 participants