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

Ensure @ApplicationScoped doesn't break default constructor generation #39146

Closed
wants to merge 1 commit into from

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Mar 4, 2024

Copy link

quarkus-bot bot commented Mar 4, 2024

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 5c2c46f.

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
JVM Tests - JDK 17 Build Failures Logs Raw logs 🚧
JVM Tests - JDK 17 Windows Build Failures Logs Raw logs 🚧
JVM Tests - JDK 21 Build Failures Logs Raw logs 🚧

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 17 #

- Failing: extensions/opentelemetry/deployment 
! Skipped: integration-tests/opentelemetry integration-tests/opentelemetry-spi 

📦 extensions/opentelemetry/deployment

io.quarkus.opentelemetry.deployment.propagation.OpenTelemetryMpContextPropagationTest. - History - More details - Source on GitHub

java.lang.ExceptionInInitializerError
	at java.base/java.lang.Class.forName0(Native Method)
	at java.base/java.lang.Class.forName(Class.java:467)
	at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:269)
	at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:662)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
Caused by: java.lang.RuntimeException: Failed to start quarkus
	at io.quarkus.runner.ApplicationImpl.<clinit>(Unknown Source)

⚙️ JVM Tests - JDK 17 Windows #

- Failing: extensions/opentelemetry/deployment 
! Skipped: integration-tests/opentelemetry integration-tests/opentelemetry-spi 

📦 extensions/opentelemetry/deployment

io.quarkus.opentelemetry.deployment.propagation.OpenTelemetryMpContextPropagationTest. - History - More details - Source on GitHub

java.lang.ExceptionInInitializerError
	at java.base/java.lang.Class.forName0(Native Method)
	at java.base/java.lang.Class.forName(Class.java:467)
	at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:269)
	at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:662)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
Caused by: java.lang.RuntimeException: Failed to start quarkus
	at io.quarkus.runner.ApplicationImpl.<clinit>(Unknown Source)

⚙️ JVM Tests - JDK 21 #

- Failing: extensions/opentelemetry/deployment 
! Skipped: integration-tests/opentelemetry integration-tests/opentelemetry-spi 

📦 extensions/opentelemetry/deployment

io.quarkus.opentelemetry.deployment.propagation.OpenTelemetryMpContextPropagationTest. - History - More details - Source on GitHub

java.lang.ExceptionInInitializerError
	at java.base/java.lang.Class.forName0(Native Method)
	at java.base/java.lang.Class.forName(Class.java:534)
	at java.base/java.lang.Class.forName(Class.java:513)
	at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:269)
	at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:662)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
Caused by: java.lang.RuntimeException: Failed to start quarkus

Flaky tests - Develocity

⚙️ JVM Tests - JDK 21

📦 extensions/smallrye-reactive-messaging-kafka/deployment

io.quarkus.smallrye.reactivemessaging.kafka.deployment.dev.KafkaDevServicesDevModeTestCase.sseStream - History

  • Assertion condition Expecting size of: [] to be greater than or equal to 2 but was 0 within 10 seconds. - org.awaitility.core.ConditionTimeoutException
org.awaitility.core.ConditionTimeoutException: 
Assertion condition 
Expecting size of:
  []
to be greater than or equal to 2 but was 0 within 10 seconds.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.AssertionCondition.await(AssertionCondition.java:119)
	at org.awaitility.core.AssertionCondition.await(AssertionCondition.java:31)

@geoand
Copy link
Contributor Author

geoand commented Mar 4, 2024

The test failure is definitely related

@geoand
Copy link
Contributor Author

geoand commented Mar 4, 2024

This fix is actually wrong as Arc is supposed to perform this transformation, so I need to figure out why it does not in the reproducer

@geoand geoand closed this Mar 4, 2024
@quarkus-bot quarkus-bot bot added triage/invalid This doesn't seem right and removed triage/backport-3.8 labels Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RESTEasy resource with constructor or field injection failing with RESTEASY003190
2 participants