Karm / mandrel-integration-tests

Integration tests for GraalVM and its Mandrel distribution. Runs Quarkus, Helidon and Micronaut applications and small targeted reproducers. The focus is solely on native-image utility and compilation of Java applications into native executables.
Apache License 2.0
5 stars 3 forks source link

JFRTest fails with graal master: AllowVMInspection deprecated warning #107

Closed jerboaa closed 1 year ago

jerboaa commented 1 year ago

Discovered here: https://github.com/graalvm/mandrel/runs/7987554026?check_suite_focus=true

Fails with:

Error:  Failures: 
Error:    JFRTest.jfrOptionsSmokeTest:157->jfrOptionsSmoke:288 build-and-run.log log should not contain error or warning lines that are not whitelisted. See /home/runner/work/mandrel/mandrel/mandrel-integration-tests/testsuite/target/archived-logs/org.graalvm.tests.integration.JFRTest/jfrOptionsSmokeTest/build-and-run.log and check these offending lines: 
Warning: Option 'AllowVMInspection' is deprecated and might be removed from future versions: Please use --enable-monitoring ==> expected: <true> but was: <false>
Error:    JFRTest.jfrSmokeTest:83->jfrSmoke:135 build-and-run.log log should not contain error or warning lines that are not whitelisted. See /home/runner/work/mandrel/mandrel/mandrel-integration-tests/testsuite/target/archived-logs/org.graalvm.tests.integration.JFRTest/jfrSmokeTest/build-and-run.log and check these offending lines: 
Warning: Option 'AllowVMInspection' is deprecated and might be removed from future versions: Please use --enable-monitoring ==> expected: <true> but was: <false>

we should be using --enable-monitoring API option in 22.3+

jerboaa commented 1 year ago

@Karm Could you please look at this when you get a chance. Thanks!

jerboaa commented 1 year ago

I'll try to get this one fixed.