logstash-plugins / logstash-input-snmp

Apache License 2.0
17 stars 22 forks source link

CI test failures with 8.x versions. #114

Closed mashhurs closed 1 year ago

mashhurs commented 1 year ago

Description

1) When we moved to JDK 17, we introduced adding JVM options to access to Java modules. This plugin was missing to apply these options.

JRUBY_OPTS=-J--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
-J--add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
-J--add-exports=jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED
-J--add-exports=jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED
-J--add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
-J--add-opens=java.base/java.security=ALL-UNNAMED -J--add-opens=java.base/java.io=ALL-UNNAMED
-J--add-opens=java.base/java.nio.channels=ALL-UNNAMED -J--add-opens=java.base/sun.nio.ch=ALL-UNNAMED
-J--add-opens=java.management/sun.management=ALL-UNNAMED
-Xregexp.interruptible=true -Xcompile.invokedynamic=true -Xjit.threshold=0 -J-XX:+PrintCommandLineFlags -v -W1

2) When we released 8.x, ECS compatibility is by default enabled. This affected to some test cases.