Skip to content

Commit

Permalink
Fix itests on machines with lots of processors (openhab#17263)
Browse files Browse the repository at this point in the history
When this JVM variable is not set Jetty creates selectors based on the number of available processors (`Runtime.getRuntime().availableProcessors()`).
Jetty will also throw an exception when there is a mismatch between the max thread pool size and the number of selectors.
This causes issues with the default thread pool size of 10 on machines with more than 10 available processors.

See also [`SelectorManager.defaultSelectors(Executor)`](https://github.com/jetty/jetty.project/blob/70015831e5195ba74da6383c4ab7c90c56bb08af/jetty-io/src/main/java/org/eclipse/jetty/io/SelectorManager.java#L71-L80).

Signed-off-by: Wouter Born <[email protected]>
  • Loading branch information
wborn authored Aug 15, 2024
1 parent e62f3af commit fcef463
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion itests/itest-common.bndrun
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,8 @@ Test-Cases: ${classes;CONCRETE;PUBLIC;NAMED;*Test}
# This property is set by the build-helper-maven-plugin in the itests pom.xml
-runvm: \
-Djdk.util.zip.disableZip64ExtraFieldValidation=true,\
-Dorg.osgi.service.http.port=${org.osgi.service.http.port}
-Dorg.osgi.service.http.port=${org.osgi.service.http.port},\
-DJETTY_AVAILABLE_PROCESSORS=4

# The integration test itself does not export anything.
Export-Package:
Expand Down

0 comments on commit fcef463

Please sign in to comment.