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

Quarkus 2.13 test extension tests fail with the 22.3 mandrel builder image #32152

Closed
jerboaa opened this issue Mar 27, 2023 · 7 comments
Closed
Assignees
Labels

Comments

@jerboaa
Copy link
Contributor

jerboaa commented Mar 27, 2023

Describe the bug

The integration tests fail with the 22.3 mandrel builder image. Filing this for tracking purposes:

[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running io.quarkus.it.extension.ClasspathInGraalITCase
Executing "/home/runner/work/mandrel/mandrel/quarkus/integration-tests/test-extension/tests/target/quarkus-integration-test-test-extension-tests-999-SNAPSHOT-runner -Dquarkus.http.port=8081 -Dquarkus.http.ssl-port=8444 -Dtest.url=http://localhost:8081 -Dquarkus.log.file.path=/home/runner/work/mandrel/mandrel/quarkus/integration-tests/test-extension/tests/target/quarkus.log -Dquarkus.log.file.enable=true"
Mar 25, 2023 2:19:24 AM io.quarkus.runtime.ApplicationLifecycleManager run
ERROR: Failed to start application (with profile prod)
java.lang.ClassNotFoundException: io.quarkus.extest.runtime.config.RunTimeConfigBuilder
	at [email protected]/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:52)
	at [email protected]/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
	at [email protected]/java.lang.ClassLoader.loadClass(ClassLoader.java:132)
	at io.quarkus.runtime.configuration.ConfigUtils.configBuilder(ConfigUtils.java:189)
	at io.quarkus.runtime.generated.Config.readConfig(Unknown Source)
	at io.quarkus.deployment.steps.RuntimeConfigSetup.deploy(Unknown Source)
	at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source)
	at io.quarkus.runtime.Application.start(Application.java:101)
	at io.quarkus.runtime.ApplicationLifecycleManager.run(ApplicationLifecycleManager.java:110)
	at io.quarkus.runtime.Quarkus.run(Quarkus.java:70)
	at io.quarkus.runtime.Quarkus.run(Quarkus.java:43)
	at io.quarkus.runtime.Quarkus.run(Quarkus.java:123)
	at io.quarkus.runner.GeneratedMain.main(Unknown Source)

Expected behavior

Test extension tests pass with the builder image.

Actual behavior

Test throws ClassNotFoundException at boot.

How to Reproduce?

Run integration tests extension tests with the 22.3 builder image.

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Quarkus version or git rev

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

See https://github.com/graalvm/mandrel/actions/runs/4516801805/jobs/7955690955#step:11:1120

@jerboaa jerboaa added the kind/bug Something isn't working label Mar 27, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Mar 27, 2023

/cc @Karm (mandrel), @galderz (mandrel), @zakkak (mandrel)

@jerboaa
Copy link
Contributor Author

jerboaa commented Mar 27, 2023

I'm not sure if this is an expected failure or not.

@geoand
Copy link
Contributor

geoand commented Mar 29, 2023

cc @radcortez

@jerboaa jerboaa changed the title Quarkus 2.13 test extension tests fail with the 22.3 mandrel builder image and Quarkus 2.13 test extension tests fail with the 22.3 mandrel builder image Mar 29, 2023
@radcortez
Copy link
Member

This requires #30220.

It should have been backported with #29842.

@zakkak
Copy link
Contributor

zakkak commented May 3, 2023

Hello @radcortez , are there any updates on this? Should we label the two PRs for backport?

@radcortez
Copy link
Member

I did open a backport here: #32278.

But the CI is having a few issues (not related to this particular backport). It requires additional backports to fix the CI:
#32278 (comment)

@radcortez
Copy link
Member

Should be fixed with the merge of #32278

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants