Skip to content

Re-enable Http2ClientReconnectTest - Need assistance from contributors #2958

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

velo
Copy link
Member

@velo velo commented Jul 20, 2025

Summary

This PR reverts the temporary disabling of Http2ClientReconnectTest in both feign-vertx and feign-vertx4-test modules.

The test was temporarily disabled in commit 196f282 to prevent build failures, but it should be properly fixed rather than left disabled.

Need for Assistance

@aklenin (Alexei KLENIN) - Could you please help investigate and fix the issues with the HTTP/2 reconnection tests? Based on the git history, you both have significant experience with this test code:

Test Failures

The Http2ClientReconnectTest is currently experiencing issues that cause build failures. The test extends AbstractClientReconnectTest and specifically tests HTTP/2 reconnection scenarios with:

  • Vertx WebClient with HTTP/2 protocol
  • Connection pool size limits
  • Jackson encoder/decoder integration

Action Items

  • Investigate root cause of test failures
  • Fix underlying issues with HTTP/2 reconnection logic
  • Ensure tests pass consistently in CI
  • Re-enable the tests

Any assistance in debugging and fixing these tests would be greatly appreciated!

🤖 Generated with Claude Code

@velo velo force-pushed the revert-http2-test-disable branch from 870072e to 5f0a4d9 Compare July 21, 2025 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant