Skip to content

Stabilize testJvmRouteLostJoinAtRuntime flaky test#11

Merged
marekkopecky merged 1 commit into
modcluster:mainfrom
honza-kasik:stable-fails
Jun 5, 2026
Merged

Stabilize testJvmRouteLostJoinAtRuntime flaky test#11
marekkopecky merged 1 commit into
modcluster:mainfrom
honza-kasik:stable-fails

Conversation

@honza-kasik

Copy link
Copy Markdown
Contributor

The test was failing for 17 consecutive CI builds because Infinispan state transfer during worker2 join/leave stalls request processing on worker1 beyond the default 10-second HTTP read timeout.

Three changes:

  • Add STATE_TRANSFER_REQUEST timeout constant (30s) to TestTimeouts
  • Add HttpClient.getWithSession overload with custom timeout
  • Use extended timeout + retry for initial requests between cycles, and raise the transient failure budget from 10 to 25

The test was failing for 17 consecutive CI builds because Infinispan
state transfer during worker2 join/leave stalls request processing
on worker1 beyond the default 10-second HTTP read timeout.

Three changes:
- Add STATE_TRANSFER_REQUEST timeout constant (30s) to TestTimeouts
- Add HttpClient.getWithSession overload with custom timeout
- Use extended timeout + retry for initial requests between cycles,
  and raise the transient failure budget from 10 to 25
@honza-kasik honza-kasik requested a review from marekkopecky June 5, 2026 08:59
@marekkopecky

Copy link
Copy Markdown
Contributor

Thank you, PR LGTM, merging.

@marekkopecky marekkopecky merged commit e2dbde4 into modcluster:main Jun 5, 2026
10 checks passed
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.

2 participants