search

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

M 23.1 JDK 21 Q 3.x Warnings #219

Closed Karm closed 7 months ago

Karm commented 8 months ago 
build-and-run.log log should not contain error or warning lines that are not whitelisted. See /home/tester/jenkins/workspace/mandrel-linux-container-integration-tests/96a0edf0/testsuite/target/archived-logs/org.graalvm.tests.integration.RuntimesSmokeTest/quarkusEncodingIssues/build-and-run.log and check these offending lines: 
Warning: The option '-H:ReflectionConfigurationResources=META-INF/native-image/io.netty/netty-transport/reflection-config.json' is experimental and must be enabled via '-H:+UnlockExperimentalVMOptions' in the future.
Warning: Please re-evaluate whether any experimental option is required, and either remove or unlock it. The build output lists all active experimental options, including where they come from and possible alternatives. If you think an experimental option should be considered as stable, please file an issue. ==> expected: <true> but was: <false>

Eventually whitelist...

zakkak commented 8 months ago

There is an upstream netty PR that should fix this. In the meantime I agree that we should whitelist it (I thought we already did so in https://github.com/Karm/mandrel-integration-tests/pull/203).

jerboaa commented 8 months ago

@Karm Why is #203 not sufficient?

Karm commented 8 months ago

@Karm Why is #203 not sufficient?

On my Cosmo Communicator ™ now, might not have the full context pulled, but I think the whitelist was not complete, did not include container tests for other Q apps.

jerboaa commented 8 months ago

It would be good if steps to reproduce would be provided for issues being opened in the future :)

Karm commented 7 months ago

Whitelisted. Closing. https://github.com/Karm/mandrel-integration-tests/blob/master/testsuite/src/it/java/org/graalvm/tests/integration/utils/WhitelistLogLines.java#L121