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] TranslogTransferManagerTests consistently fail on Windows CI. #5923

Closed
Tracked by #5716
mch2 opened this issue Jan 18, 2023 · 14 comments · Fixed by #7173
Closed
Tracked by #5716

[BUG] TranslogTransferManagerTests consistently fail on Windows CI. #5923

mch2 opened this issue Jan 18, 2023 · 14 comments · Fixed by #7173
Assignees
Labels
bug Something isn't working distributed framework Storage:Durability Issues and PRs related to the durability framework v2.8.0 'Issues and PRs related to version v2.8.0'

Comments

@mch2
Copy link
Member

mch2 commented Jan 18, 2023

This test consistently fails in nightly windows runs.
Example: https://build.ci.opensearch.org/job/gradle-check/9541/.

@mch2
Copy link
Member Author

mch2 commented Jan 18, 2023

@sachinpkale Can you take a look here?

@ashking94 ashking94 added Storage:Durability Issues and PRs related to the durability framework v2.6.0 'Issues and PRs related to version v2.6.0' labels Jan 19, 2023
@ryanbogan ryanbogan assigned andrross and unassigned ryanbogan and andrross Jan 31, 2023
@sachinpkale
Copy link
Member

Moving to 2.7.0

@sachinpkale sachinpkale added v2.7.0 and removed v2.6.0 'Issues and PRs related to version v2.6.0' labels Feb 21, 2023
@minalsha
Copy link
Contributor

@vibrantvarun what is the status on progress for this issue?

@owaiskazi19
Copy link
Member

owaiskazi19 commented Mar 15, 2023

Ran the test 400 times on Windows. Wasn't able to reproduce the issue, @sachinpkale can you verify and unmute the tests if everythink looks good:

PS C:\Users\opensearch\OpenSearch> ./gradlew ':server:test' --tests "org.opensearch.index.translog.transfer.TranslogTransferManagerTests" "-Dtests.seed=6FF35C90E65A864A" "-Dtests.iters=400"
>>

> Configure project :plugins:repository-hdfs
hdfsFixture unsupported, please set HADOOP_HOME and put HADOOP_HOME\bin in PATH

> Configure project :qa:os
Cannot add task 'destructiveDistroTest.docker' as a task with that name already exists.
=======================================
OpenSearch Build Hamster says Hello!
  Gradle Version        : 8.0.2
  OS Info               : Windows Server 2022 10.0 (amd64)
  JDK Version           : 19 (Eclipse Temurin JDK)
  JAVA_HOME             : C:\Program Files\Eclipse Adoptium\jdk-19.0.2.7-hotspot
  Random Testing Seed   : 6FF35C90E65A864A
  In FIPS 140 mode      : false
=======================================

> Task :server:test
WARNING: A terminally deprecated method in java.lang.System has been called
WARNING: System::setSecurityManager has been called by org.opensearch.bootstrap.BootstrapForTesting (file:/C:/Users/opensearch/OpenSearch/test/framework/build/distributions/framework-3.0.0-SNAPSHOT.jar)
WARNING: Please consider reporting this to the maintainers of org.opensearch.bootstrap.BootstrapForTesting
WARNING: System::setSecurityManager will be removed in a future release
WARNING: A terminally deprecated method in java.lang.System has been called
WARNING: System::setSecurityManager has been called by org.gradle.api.internal.tasks.testing.worker.TestWorker (file:/C:/Users/opensearch/.gradle/wrapper/dists/gradle-8.0.2-all/25ipb77ce0ypy3f9xdton1ae6/gradle-8.0.2/lib/plugins/gradle-testing-base-8.0.2.jar)
WARNING: Please consider reporting this to the maintainers of org.gradle.api.internal.tasks.testing.worker.TestWorker
WARNING: System::setSecurityManager will be removed in a future release

BUILD SUCCESSFUL in 1m 56s
38 actionable tasks: 1 executed, 37 up-to-date

@minalsha
Copy link
Contributor

@sachinpkale could you please address @owaiskazi19 comments above? Thanks

@minalsha
Copy link
Contributor

@sachinpkale can you please address?

@sachinpkale
Copy link
Member

@sachinpkale can you please address?

Will check and unmute the test.

@minalsha
Copy link
Contributor

minalsha commented Apr 5, 2023

@sachinpkale can you please share an update on this test?

@sachinpkale
Copy link
Member

@minalsha I could not verify the test as I don't have the windows setup. Based on the test run by @owaiskazi19, if the test is running fine after multiple iterations, we can unmute the test.

@minalsha
Copy link
Contributor

minalsha commented Apr 5, 2023

@sachinpkale please unmute the test then. Thanks

@DarshitChanpura
Copy link
Member

Hey @sachinpkale, Is the bug-fix still on track to be completed by v2.7.0 code-freeze on Apr-17. If not, it should be labeled for v2.8.0 release

@DarshitChanpura
Copy link
Member

Tagging it for next release: v2.8.0

@DarshitChanpura DarshitChanpura added v2.8.0 'Issues and PRs related to version v2.8.0' and removed v2.7.0 labels Apr 17, 2023
@owaiskazi19
Copy link
Member

Tagging it for next release: v2.8.0

@DarshitChanpura I have a PR ready. Can we still get it in 2.7?

@DarshitChanpura
Copy link
Member

DarshitChanpura commented Apr 17, 2023

@owaiskazi19 I have added backport 2.x label. If the backport gets merged by EOD today then I'll change the release tag to 2.7.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working distributed framework Storage:Durability Issues and PRs related to the durability framework v2.8.0 'Issues and PRs related to version v2.8.0'
Projects
None yet
Development

Successfully merging a pull request may close this issue.