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.

Sign up for GitHub

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

Jump to bottom

[BUG] OperationRoutingTests.testWeightedOperationRoutingWeightUndefinedForOneZone failure #4881

Closed
andrross opened this issue Oct 22, 2022 · 6 comments
Closed

[BUG] OperationRoutingTests.testWeightedOperationRoutingWeightUndefinedForOneZone failure #4881

andrross opened this issue Oct 22, 2022 · 6 comments
Labels
bug Something isn't working flaky-test Random test failure that succeeds on second run good first issue Good for newcomers hacktoberfest Global event that encourages people to contribute to open-source.

Comments

@andrross
Copy link
Member

Failure here: https://build.ci.opensearch.org/job/gradle-check/5293/

This test failure appears to be reproducible with the given seed:

./gradlew ':server:test' --tests "org.opensearch.cluster.routing.OperationRoutingTests.testWeightedOperationRoutingWeightUndefinedForOneZone" -Dtests.seed=B1745FEE14F29C02 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=ar-EG -Dtests.timezone=Asia/Magadan -Druntime.java=17

I haven't investigated at all but should be easily fixable given that it is deterministic.
@andrross andrross added bug Something isn't working good first issue Good for newcomers untriaged hacktoberfest Global event that encourages people to contribute to open-source. labels Oct 22, 2022
@andrross andrross mentioned this issue Oct 22, 2022
Merged
6 tasks
@andrross
Copy link
Member Author

Maybe not a one-off...another failure here: https://build.ci.opensearch.org/job/gradle-check/5299/

@andrross andrross mentioned this issue Oct 22, 2022
Merged
6 tasks
@adnapibar adnapibar mentioned this issue Oct 24, 2022
Closed
@adnapibar adnapibar added the flaky-test Random test failure that succeeds on second run label Oct 24, 2022
@kotwanikunal kotwanikunal mentioned this issue Oct 28, 2022
Merged
6 tasks
@iofit
Copy link

iofit commented Oct 29, 2022

I've submitted #4980 as an attempt to fix this issue. I've observed that there are different behaviors when using java17 specially with those related to math,

@VachaShah VachaShah mentioned this issue Nov 1, 2022
Merged
6 tasks
@dblock
Copy link
Member

dblock commented Nov 15, 2022

#5241 (comment)

This was referenced Nov 15, 2022
Closed
Closed
@dblock
Copy link
Member

dblock commented Nov 15, 2022

@anshu1106 looks like this was introduced in #4241, can you please take a look?

@Bukhtawar Bukhtawar mentioned this issue Nov 15, 2022
Merged
6 tasks
@adnapibar adnapibar mentioned this issue Nov 15, 2022
Merged
6 tasks
@Poojita-Raj Poojita-Raj mentioned this issue Nov 15, 2022
Closed
37 tasks
@anshu1106 anshu1106 mentioned this issue Nov 17, 2022
Merged
6 tasks
@anshu1106
Copy link
Contributor

anshu1106 commented Nov 17, 2022 •
edited
Loading

@anshu1106 looks like this was introduced in #4241, can you please take a look?

Raised PR with the fix

@adnapibar
Copy link
Contributor

Resolved by #5289

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working flaky-test Random test failure that succeeds on second run good first issue Good for newcomers hacktoberfest Global event that encourages people to contribute to open-source.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change number of shards and fix typo. dblock/OpenSearch
6 participants
@iofit @dblock @adnapibar @anshu1106 @andrross @minalsha