Skip to content

Conversation

mkouba
Copy link
Contributor

@mkouba mkouba commented Jun 5, 2025

This comment was marked as off-topic.

@mkouba mkouba changed the title WebSockets Next: perform connection cleanup during app shutdown WebSockets Next: perform client connection cleanup during app shutdown Jun 6, 2025
@mkouba mkouba force-pushed the wsnext-client-conn-cleanup branch from 4944cad to f5ed6c3 Compare June 13, 2025 15:05
@mkouba mkouba marked this pull request as ready for review June 13, 2025 15:06
@mkouba
Copy link
Contributor Author

mkouba commented Jun 13, 2025

This PR does not solve the problem mentioned in the zulip discussion; the eclipse-vertx/vert.x#5609 should, but I think that it's correct and the quarkus.websockets-next.client.connection-idle-timeout might be useful.

@mkouba mkouba requested a review from geoand June 13, 2025 15:08

This comment has been minimized.

Copy link

quarkus-bot bot commented Jun 13, 2025

Status for workflow Quarkus CI

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

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
JVM Tests - JDK 17 Clean Gradle temp directory ⚠️ Check → Logs Raw logs 🔍
JVM Tests - JDK 21 Clean Gradle temp directory ⚠️ Check → Logs Raw logs 🔍
JVM Tests - JDK 17 Windows Clean Gradle temp directory ⚠️ Check → Logs Raw logs 🔍
✔️ JVM Integration Tests - JDK 17 Logs Raw logs 🔍
✔️ JVM Integration Tests - JDK 17 Windows Logs Raw logs 🔍
JVM Integration Tests - JDK 21 Clean Gradle temp directory ⚠️ Check → Logs Raw logs 🔍

You can consult the Develocity build scans.

Copy link

quarkus-bot bot commented Jun 13, 2025

Status for workflow Quarkus Documentation CI

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

✅ 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.

Copy link

🎊 PR Preview 4e66327 has been successfully built and deployed to https://quarkus-pr-main-48254-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

@gsmet gsmet merged commit 6b76a56 into quarkusio:main Jun 14, 2025
41 of 45 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.25 - main milestone Jun 14, 2025
@gsmet gsmet modified the milestones: 3.25 - main, 3.24.0 Jun 14, 2025
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.

3 participants