-
Notifications
You must be signed in to change notification settings - Fork 845
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
Use ratpack 1.4.1 for testing #4748
Conversation
@@ -92,7 +92,10 @@ abstract class NettyAlignmentRule : ComponentMetadataRule { | |||
dependencies { | |||
add(dependencyManagementConf.name, platform(project(":dependencyManagement"))) | |||
|
|||
components.all<NettyAlignmentRule>() | |||
// ratpack tests don't work correctly with 4.1.65.Final |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this a separate flaky issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The problem is that besides the fix that hopefully gets rid of the flakiness there is another change that breaks some tests when run with netty 4.1.65. These tests pass when run with the netty version that ratpack depends on. All tests in RatpackPooledHttpClientTest
fail, from other http client tests only highConcurrencyOnSingleConnection
fails, there are also failures under library
tests.
...main/java/io/opentelemetry/instrumentation/ratpack/client/AbstractRatpackHttpClientTest.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Trask Stalnaker <[email protected]>
// to allow all tests to pass we need to choose a specific netty version | ||
if (!(findProperty("testLatestDeps") as Boolean)) { | ||
configurations.configureEach { | ||
if (!name.contains("muzzle")) { | ||
resolutionStrategy { | ||
eachDependency { | ||
// specifying a fixed version for all libraries with io.netty group | ||
if (requested.group == "io.netty") { | ||
useVersion("4.1.31.Final") | ||
} | ||
} | ||
} | ||
} | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx 👍
* Use ratpack 1.4.1 for testing * don't force netty version in ratpack tests * spotless * Apply suggestions from code review Co-authored-by: Trask Stalnaker <[email protected]> * enable https test Co-authored-by: Trask Stalnaker <[email protected]> Co-authored-by: Lauri Tulmin <[email protected]>
* Use ratpack 1.4.1 for testing * don't force netty version in ratpack tests * spotless * Apply suggestions from code review Co-authored-by: Trask Stalnaker <[email protected]> * enable https test Co-authored-by: Trask Stalnaker <[email protected]>
Resolves #4736
Resolves #4618
I believe that the flakiness of ratpack tests is caused by a bug in ratpack that was fixed in this commit ratpack/ratpack@dde536a
Basically due to using wrong type of queue some threads will end in a busy loop inside queue implementation because internal state of the queue has been corrupted. Here is a sample of such stuck thread