-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[BUG] org.opensearch.remotestore.RemoteStoreRefreshListenerIT.testRemoteRefreshRetryOnFailure is flaky #7703
Comments
Signed-off-by: Ashish Singh <[email protected]>
There are other exception stack traces as well that I can see -
|
Signed-off-by: Ashish Singh <[email protected]>
Signed-off-by: Ashish Singh <[email protected]> (cherry picked from commit 5d5e8ad) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Still flaky in #7969 (comment) |
…earch-project#7879) (opensearch-project#7881) (cherry picked from commit 5d5e8ad) Signed-off-by: Ashish Singh <[email protected]>
I will be prioritising this issue, will try to get this closed before 2.9 release. |
@ashking94 I think the only issue is that the I tried adding a force refresh operation here and the test seemed to succeed for 500 times without any issues. |
@kotwanikunal Thanks for the investigation. The tests are failing due to segments have not been uploaded to remote store completely yet. In this case, we are intentionally failing the uploads and have retry mechanism which will ultimately get the remote store in sync with local segments. Doing a force refresh is not recommended as it triggers refresh internally and hence it would make the upload go through ultimately. For now, I am muting the test. I should be able to get the fix real soon. |
…earch-project#7879) Signed-off-by: Ashish Singh <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
Describe the bug
org.opensearch.remotestore.RemoteStoreRefreshListenerIT.testRemoteRefreshRetryOnFailure is flaky
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The test should pass reliably
Plugins
Standard
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):
Additional context
https://build.ci.opensearch.org/job/gradle-check/16080/testReport/junit/org.opensearch.remotestore/RemoteStoreRefreshListenerIT/testRemoteRefreshRetryOnFailure/
The text was updated successfully, but these errors were encountered: