search

quarkusio / quarkus

Quarkus: Supersonic Subatomic Java.
https://quarkus.io
Apache License 2.0
13.83k stars 2.7k forks source link

Do not generate a class to remove drivers #44604

Closed dmlloyd closed 18 hours ago

dmlloyd commented 1 day ago

We can redirect caller-sensitive methods by using privateLookupIn instead.

This is just something I happened to notice while evaluating the class loader code.

quarkus-bot[bot] commented 1 day ago

Status for workflow Quarkus CI

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

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
✖ JVM Tests - JDK 17 Build :warning: Check → Logs Raw logs :construction:
✖ JVM Tests - JDK 21 Build Failures Logs Raw logs :construction:
✖ JVM Tests - JDK 17 Windows Build :warning: Check → Logs Raw logs :construction:

Full information is available in the Build summary check run. You can consult the Develocity build scans.

Failures

:gear: JVM Tests - JDK 21 #

:package: extensions/opentelemetry/deployment

✖ io.quarkus.opentelemetry.deployment.traces.NonAppEndpointsDisabledTest. - 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:289) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Suppressed: java.lang.reflect.InvocationTargetException ```

✖ io.quarkus.opentelemetry.deployment.traces.NonAppEndpointsDisabledWithRootPathTest. - History - More details - Source on GitHub

``` java.lang.RuntimeException: Failed to start quarkus at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source) at io.quarkus.runtime.Application.start(Application.java:101) at java.base/java.lang.reflect.Method.invoke(Method.java:580) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:305) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.RuntimeException: Unable to start HTTP server ```

✖ io.quarkus.opentelemetry.deployment.traces.NonAppEndpointsEnabledLegacyConfigurationTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.NonAppEndpointsEnabledTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.OpenTelemetryCustomSamplerBeanTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.OpenTelemetryHttpCDITest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.OpenTelemetryIdGeneratorTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.OpenTelemetryReactiveRoutesTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.OpenTelemetrySamplerBeanTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.OpenTelemetrySamplerConfigTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.OpenTelemetrySpanSecurityEventsTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.OpenTelemetrySuppressAppUrisTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.OpenTelemetryTracelessTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

✖ io.quarkus.opentelemetry.deployment.traces.OpenTelemetryTracelessWithRootPathTest. - History - More details - Source on GitHub

``` java.lang.IllegalArgumentException: Self-suppression not permitted at java.base/java.lang.Throwable.addSuppressed(Throwable.java:1096) at io.quarkus.runner.bootstrap.StartupActionImpl.run(StartupActionImpl.java:298) at io.quarkus.test.QuarkusUnitTest.beforeAll(QuarkusUnitTest.java:689) at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.lang.OutOfMemoryError: Metaspace ```

Flaky tests - Develocity

:gear: JVM Tests - JDK 21

:package: extensions/smallrye-reactive-messaging/deployment

✖ io.quarkus.smallrye.reactivemessaging.hotreload.ConnectorChangeTest.testUpdatingConnector - History

``` java.lang.AssertionError: Expecting actual: ["-4","-5","-6","-7","-8","-9","-10","-11"] to start with: ["-3", "-4", "-5", "-6"] at io.quarkus.smallrye.reactivemessaging.hotreload.ConnectorChangeTest.testUpdatingConnector(ConnectorChangeTest.java:36) ```
``` java.lang.AssertionError: Expecting actual: ["-6","-8","-9","-10","-11","-12","-13","-14"] to start with: ["-6", "-7", "-8", "-9"] at io.quarkus.smallrye.reactivemessaging.hotreload.ConnectorChangeTest.testUpdatingConnector(ConnectorChangeTest.java:41) ```
dmlloyd commented 19 hours ago

I suppose it's not a coincidence that it's failing due to metaspace.

dmlloyd commented 18 hours ago

Well, I don't have even a theoretical reason why this would cause metaspace exhaustion, yet here we are. This is not an important change though (it was meant as a minor opportunistic improvement) so I'll drop it for now. If it turns out that something else is causing the metaspace problem, I'll re-open and rebase and go from there.