Skip to content

Commit

Permalink
Always require native libraries for running tests (#2990)
Browse files Browse the repository at this point in the history
Motivation:

In rare occasions tests may fail with ambiguous errors. After hours of
debugging, the root cause may end up an inability to load native
libraries.

Modifications:

- Enforce `io.servicetalk.transport.netty.requireNativeLibs=true` for
running tests.

Result:

Significantly simplifies finding the root cause for hanging tests.
  • Loading branch information
idelpivnitskiy authored Jun 27, 2024
1 parent 4ce7d2c commit 63089ee
Showing 1 changed file with 4 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -229,6 +229,10 @@ final class ServiceTalkLibraryPlugin extends ServiceTalkCorePlugin {

jvmArgs "-server", "-Xms2g", "-Xmx4g", "-dsa", "-da", "-ea:io.servicetalk...",
"-XX:+HeapDumpOnOutOfMemoryError"

// Always require native libraries for running tests. This helps to make sure transport-level state machine
// receives all expected network events from Netty.
systemProperty "io.servicetalk.transport.netty.requireNativeLibs", "true"
}

dependencies {
Expand Down

0 comments on commit 63089ee

Please sign in to comment.