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

[BUG] SnapshotStatusApisIT.testExceptionOnMissingShardLevelSnapBlob fails with reproducible seed #15584

Closed
mch2 opened this issue Sep 2, 2024 · 5 comments
Assignees
Labels
bug Something isn't working Storage:Snapshots >test-failure Test failure from CI, local build, etc.

Comments

@mch2
Copy link
Member

mch2 commented Sep 2, 2024

Describe the bug

junit.framework.AssertionFailedError: Expected exception SnapshotMissingException but no exception was thrown
	at __randomizedtesting.SeedInfo.seed([5F17849827BA79F3:C1C9BD258865E37A]:0)
	at org.apache.lucene.tests.util.LuceneTestCase.expectThrows(LuceneTestCase.java:2886)
	at org.apache.lucene.tests.util.LuceneTestCase.expectThrows(LuceneTestCase.java:2872)
	at org.opensearch.snapshots.SnapshotStatusApisIT.testExceptionOnMissingShardLevelSnapBlob(SnapshotStatusApisIT.java:211)
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)
REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.snapshots.SnapshotStatusApisIT.testExceptionOnMissingShardLevelSnapBlob" -Dtests.seed=5F17849827BA79F3 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=it-VA -Dtests.timezone=US/Hawaii -Druntime.java=21

Related component

Storage:Snapshots

To Reproduce

Seed above.

Expected behavior

Test to pass

Additional Details

No response

@prudhvigodithi
Copy link
Member

Hey @mch2 the automation flagged the class SnapshotStatusApisIT showing some flaky tests, please check #15815. Can we close this issue?
Thanks
@dblock @rednaksi91 @andrross @getsaurabh02

@mch2
Copy link
Member Author

mch2 commented Sep 7, 2024

@prudhvigodithi this fails with a reproducible seed so imo should be treated as a bug not a flaky test.

@prudhvigodithi
Copy link
Member

Coming from here #15815, the SnapshotStatusApisIT failed at the post merge action which means it passed at one point, the PR was merged and started to see some failures after then. The issue also has the other PR's linked on which the SnapshotStatusApisIT failed at one point and passed after re-try as some of them are in merged state.
Thanks

@ashking94
Copy link
Member

@ltaragi Lets close this issue if it has been fixed.
[Triage - attendees 1 2 3 4 5]

@ltaragi
Copy link
Contributor

ltaragi commented Oct 1, 2024

Auto-cut issue (#15815) was closed by PR (#16146). Closing this one as well.

@ltaragi ltaragi closed this as completed Oct 1, 2024
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in Storage Project Board Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Storage:Snapshots >test-failure Test failure from CI, local build, etc.
Projects
Status: ✅ Done
Development

No branches or pull requests

4 participants