Skip to content

Conversation

holly-cummins
Copy link
Contributor

@holly-cummins holly-cummins commented Jun 5, 2025

Sort-of fixes #47656

We can't properly fix the issue in Quarkus without moving away from the model where we want test classes get loaded by the 'right' classloader in the discovery phase. The underlying problem is actually an Eclipse problem, which I've submitted an Eclipse PR for.

@holly-cummins holly-cummins requested a review from geoand June 5, 2025 20:45
@holly-cummins holly-cummins added the area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins label Jun 5, 2025

This comment has been minimized.

Copy link
Contributor

@geoand geoand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏽

Copy link

quarkus-bot bot commented Jun 6, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 39b563d.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.


Flaky tests - Develocity

⚙️ JVM Integration Tests - JDK 17

📦 integration-tests/opentelemetry

io.quarkus.it.opentelemetry.LoggingResourceTest.testException - History

  • Condition with Lambda expression in io.quarkus.it.opentelemetry.LoggingResourceTest was not fulfilled within 2 minutes. - org.awaitility.core.ConditionTimeoutException
org.awaitility.core.ConditionTimeoutException: Condition with Lambda expression in io.quarkus.it.opentelemetry.LoggingResourceTest was not fulfilled within 2 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:26)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1160)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1129)
	at io.quarkus.it.opentelemetry.LoggingResourceTest.testException(LoggingResourceTest.java:113)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)

@geoand geoand merged commit 7792f8a into quarkusio:main Jun 6, 2025
110 of 112 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.24 - main milestone Jun 6, 2025
@gsmet gsmet modified the milestones: 3.24 - main, 3.23.3 Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/testing kind/bugfix triage/flaky-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to run QuarkusTests in Eclipse since Quarkus 3.22

3 participants