-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Bump smallrye-open-api.version from 4.0.5 to 4.0.6 #45424
base: main
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something is odd. The JDK tests didn’t run. I need to figure out what’s wrong with CI and get another run.
There was some strange behavior in some other repos today as well and I think something mentioned about it on Github status. |
@MikeEdgar thanks for this. Just a note, it does not (yet) fix Fixes #45266, there needs some config to enable this in Quarkus. I'll make a PR for it once this is merged |
It's an issue on our side. I think #45437 might fix it. I will rebase this PR as soon as I can get it in. |
Signed-off-by: Michael Edgar <[email protected]>
2f943bc
to
62adee1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Matrix issue is solved. Let's wait for CI and merge.
Status for workflow
|
Status | Name | Step | Failures | Logs | Raw logs | Build scan |
---|---|---|---|---|---|---|
✖ | JVM Tests - JDK 17 | Build |
Failures | Logs | Raw logs | 🚧 |
✖ | JVM Tests - JDK 21 | Build |
Failures | Logs | Raw logs | 🚧 |
✖ | JVM Tests - JDK 17 Windows | Build |
Failures | Logs | Raw logs | 🚧 |
Full information is available in the Build summary check run.
You can consult the Develocity build scans.
Failures
⚙️ JVM Tests - JDK 17 #
📦 extensions/smallrye-fault-tolerance/deployment
✖ io.quarkus.smallrye.faulttolerance.test.asynchronous.additional.BlockingTest.threadOffloadAndFallback
line 27
- History - More details - Source on GitHub
java.util.concurrent.ExecutionException: java.lang.NullPointerException: Cannot invoke "io.vertx.core.Context.isEventLoopContext()" because "this.vertxContext" is null
at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2073)
at io.quarkus.smallrye.faulttolerance.test.asynchronous.additional.BlockingTest.threadOffloadAndFallback(BlockingTest.java:27)
at java.base/java.lang.reflect.Method.invoke(Method.java:569)
at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
at io.quarkus.test.QuarkusUnitTest.interceptTestMethod(QuarkusUnitTest.java:427)
at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
⚙️ JVM Tests - JDK 21 #
📦 extensions/smallrye-fault-tolerance/deployment
✖ io.quarkus.smallrye.faulttolerance.test.asynchronous.additional.BlockingTest.threadOffloadAndFallback
line 27
- History - More details - Source on GitHub
java.util.concurrent.ExecutionException: java.lang.NullPointerException: Cannot invoke "io.vertx.core.Context.isEventLoopContext()" because "this.vertxContext" is null
at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2073)
at io.quarkus.smallrye.faulttolerance.test.asynchronous.additional.BlockingTest.threadOffloadAndFallback(BlockingTest.java:27)
at java.base/java.lang.reflect.Method.invoke(Method.java:580)
at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
at io.quarkus.test.QuarkusUnitTest.interceptTestMethod(QuarkusUnitTest.java:427)
at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
⚙️ JVM Tests - JDK 17 Windows #
📦 extensions/smallrye-fault-tolerance/deployment
✖ io.quarkus.smallrye.faulttolerance.test.asynchronous.additional.BlockingTest.threadOffloadAndFallback
line 27
- History - More details - Source on GitHub
java.util.concurrent.ExecutionException: java.lang.NullPointerException: Cannot invoke "io.vertx.core.Context.isEventLoopContext()" because "this.vertxContext" is null
at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2073)
at io.quarkus.smallrye.faulttolerance.test.asynchronous.additional.BlockingTest.threadOffloadAndFallback(BlockingTest.java:27)
at java.base/java.lang.reflect.Method.invoke(Method.java:569)
at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:513)
at io.quarkus.test.QuarkusUnitTest.interceptTestMethod(QuarkusUnitTest.java:427)
at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
Flaky tests - Develocity
⚙️ Maven Tests - JDK 17
📦 integration-tests/devmode
✖ io.quarkus.test.devui.DevUIGrpcSmokeTest.testTestService
- History
Too many recursions, message not returned for id [717719135]
-java.lang.RuntimeException
java.lang.RuntimeException: Too many recursions, message not returned for id [717719135]
at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:164)
at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:167)
at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:167)
at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:167)
at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:167)
at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:167)
at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:167)
@Ladicek could you have a look at the error above? My guess is that it's not due to this update but it looks suspicious anyway. Also the problem got the build completely stuck:
|
Fixes #44773
Fixes #45150
Required for #45266