-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Security annotations don't work with CompletionStage #20997
Conversation
This workflow status is outdated as a new workflow run has been triggered. Failing Jobs - Building ab6cb23
Full information is available in the Build summary check run. Failures⚙️ MicroProfile TCKs Tests #- Failing: tcks/microprofile-fault-tolerance
📦 tcks/microprofile-fault-tolerance✖
|
.../security/runtime/src/main/java/io/quarkus/security/runtime/interceptor/SecurityHandler.java
Show resolved
Hide resolved
Failing Jobs - Building 5990962
Full information is available in the Build summary check run. Failures⚙️ Native Tests - Messaging2 #- Failing: integration-tests/reactive-messaging-rabbitmq
📦 integration-tests/reactive-messaging-rabbitmq✖
|
@cescoffier the There doesn't seem to be anything obviously wrong with the test, so maybe there is an issue in the SR RM RabbitMQ module? |
@geoand The rabbit MQ connector is very new and probably not very stable yet. However, the test suite in reactive messaging seems to be ok. We should ping the contributor. |
@kdubb ^ |
The Quarkus SR RM RabbitMQ module does nothing except add the SR RM RMQ dependency. I am having no trouble running the tests locally but I am usually using the Also, I was seeing intermittent failing of internal ITs that use Dev Services when I was running the module, as if the external docker service wasn't getting started in a timely fashion. In my last two rebases of the PR that issue seemed to have gone away. Is that a previously known issue? If so, how is that generally remedied. (You can see the evidence of this in the PR #20659 test history). |
Fixes #20979