Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RuntimesSmokeTest.quarkusFullMicroProfile/JFRTest.jfrPerfTest fail due to new Quarkus warnings with quarkus main (3.6) #220

Closed
jerboaa opened this issue Nov 13, 2023 · 6 comments

Comments

@jerboaa
Copy link
Collaborator

jerboaa commented Nov 13, 2023

The RuntimesSmokeTest.quarkusFullMicroProfile test fails in CI with JDK 22 + GraalVM 24.0 with:

2023-11-13 02:28:31,843 WARN  [io.qua.run.con.DeprecatedRuntimePropertiesRecorder] (main) The 'quarkus.http.auth.form.redirect-after-login' config property is deprecated and should not be used anymore ==> expected: <true> but was: <false>
Error:    RuntimesSmokeTest.quarkusFullMicroProfile:192->testRuntime:157 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.RuntimesSmokeTest/quarkusFullMicroProfile/build-and-run.log and check these offending lines: 
2023-11-13 03:03:04,462 WARN  [io.qua.run.con.DeprecatedRuntimePropertiesRecorder] (main) The 'quarkus.http.auth.form.redirect-after-login' config property is deprecated and should not be used anymore ==> expected: <true> but was: <false>

and

Error:  Failures: 
Error:    JFRTest.jfrPerfTest:176->jfrPerfTestRun:219 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/jfrPerfTest/build-and-run.log and check these offending lines: 
2023-11-13 02:28:29,532 WARN  [io.qua.run.con.DeprecatedRuntimePropertiesRecorder] (main) The 'quarkus.http.auth.form.redirect-after-login' config property is deprecated and should not be used anymore

See: https://github.com/graalvm/mandrel/actions/runs/6844550940/job/18609261996#step:11:18840

@jerboaa
Copy link
Collaborator Author

jerboaa commented Nov 13, 2023

I have reproduced this locally with the quarkus microprofile app, but I have no idea where the property is coming from:

$ java -jar target/quarkus-app/quarkus-run.jar 
__  ____  __  _____   ___  __ ____  ______ 
 --/ __ \/ / / / _ | / _ \/ //_/ / / / __/ 
 -/ /_/ / /_/ / __ |/ , _/ ,< / /_/ /\ \   
--\___\_\____/_/ |_/_/|_/_/|_|\____/___/   
2023-11-13 18:08:21,863 WARN  [io.qua.run.con.DeprecatedRuntimePropertiesRecorder] (main) The 'quarkus.http.auth.form.redirect-after-login' config property is deprecated and should not be used anymore
2023-11-13 18:08:22,130 INFO  [io.quarkus] (main) quarkus-full-microprofile 1.0.0-SNAPSHOT on JVM (powered by Quarkus 999-SNAPSHOT) started in 0.917s. Listening on: http://0.0.0.0:8080
2023-11-13 18:08:22,131 INFO  [io.quarkus] (main) Profile prod activated. 
2023-11-13 18:08:22,131 INFO  [io.quarkus] (main) Installed features: [cdi, jaeger, rest-client, resteasy, security, smallrye-context-propagation, smallrye-fault-tolerance, smallrye-health, smallrye-jwt, smallrye-metrics, smallrye-openapi, smallrye-opentracing, vertx]
^C2023-11-13 18:08:29,397 INFO  [io.quarkus] (Shutdown thread) quarkus-full-microprofile stopped in 0.221s

A simple grep doesn't return anything for me....

@michalvavrik
Copy link

Please see quarkusio/quarkus#37072.

@jerboaa jerboaa changed the title RuntimesSmokeTest.quarkusFullMicroProfile/JFRTest.jfrPerfTest fail due to new Quarkus warnings with quarkus main (3.5) RuntimesSmokeTest.quarkusFullMicroProfile/JFRTest.jfrPerfTest fail due to new Quarkus warnings with quarkus main (3.6) Nov 14, 2023
@jerboaa
Copy link
Collaborator Author

jerboaa commented Nov 14, 2023

Thanks!

@jerboaa
Copy link
Collaborator Author

jerboaa commented Nov 15, 2023

I'm proposing to allow-list the deprecation lines in #222 until we have a proper fix in Quarkus so as to reduce test noise and miss issues because of this known problem.

@jerboaa
Copy link
Collaborator Author

jerboaa commented Nov 17, 2023

quarkusio/quarkus#37072 fixed this for us.

@jerboaa jerboaa closed this as completed Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants