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

[FEATURE] Add retry for BWC tests #778

Open
chishui opened this issue Jun 6, 2024 · 2 comments
Open

[FEATURE] Add retry for BWC tests #778

chishui opened this issue Jun 6, 2024 · 2 comments
Labels

Comments

@chishui
Copy link
Contributor

chishui commented Jun 6, 2024

Is your feature request related to a problem?

BWC tests are quite flaky, here is one failed test example:

We can introduce retry for BWC tests to reduce the chance of random failures.

What solution would you like?

OpenSearch core repo uses test-retry plugin to retry flaky tests: https://github.com/opensearch-project/OpenSearch/blob/main/build.gradle#L469

What alternatives have you considered?

N/A

Do you have any additional context?

N/A

@dblock
Copy link
Member

dblock commented Jul 1, 2024

[Catch All Triage - Attendees 1, 2, 3, 4, 5]

@dblock dblock removed the untriaged label Jul 1, 2024
@dblock
Copy link
Member

dblock commented Jul 1, 2024

Double check that the errors are truly transient. Retrying often just hides eventual consistency errors and is not recommended.

[Catch All Triage - Attendees 1, 2, 3, 4, 5]

@naveentatikonda naveentatikonda added the good first issue Good for newcomers label Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants