Create read and write timeout tests for netty / chfw by volosied · Pull Request #35680 · OpenLiberty/open-liberty

volosied · 2026-09-02T15:57:46Z

I have considered the risk of behavior change or other zero migration impact (https://github.com/OpenLiberty/open-liberty/wiki/Behavior-Changes).
If this PR fixes an Issue, the description includes "Fixes #FILLMEIN" or "Resolves #FILLMEIN" (verify release bug label if applicable: https://github.com/OpenLiberty/open-liberty/wiki/Open-Liberty-Conventions).
If this PR resolves an external Known Issue (including APARS), the description includes "Fixes #FILLMEIN" or "Resolves #FILLMEIN".

volosied · 2026-09-16T01:26:08Z

!build
(view Open Liberty Personal Build - ❌ completed with errors/failures.)
spawn.fullfat.buckets=io.openliberty.transport.http_fat
extra.fat.ant.options=-Dglobal.jvm.args="-Dcom.ibm.ws.beta.edition=true"

Note: Target locations of links might be accessible only to IBM employees.

LibbyBot · 2026-09-16T01:51:41Z

Code analysis and actions

DO NOT DELETE THIS COMMENT.

3 FAT files were changed, added, or removed.
Check that the build did not break the affected FAT suite(s).
1 test infrastructure code files were changed.
Test failures/errors in the build could be due to these changes.
4 product code files were changed.
Please describe in a separate comment how you tested your changes.
1 messages files were changed and need an L2 review.
@OpenLiberty/message-reviewer Please review.
- dev/com.ibm.ws.channelfw/resources/com/ibm/ws/tcpchannel/internal/resources/TCPChannelMessages.nlsprops
1 NLS files were changed and need an ID review.
@OpenLiberty/message-reviewer Please review.
- dev/com.ibm.ws.channelfw/resources/com/ibm/ws/tcpchannel/internal/resources/TCPChannelMessages.nlsprops

michael-bowler-232

Suggested revised user action for CWWKO1713W:
"Check for network latency or connectivity issues between the client and server. If the connection is valid, increase the readTimeout value in the HTTP endpoint configuration."

Also. The substitution variable {0} is unexplained — what does it contain? A channel name, a connection ID, a timeout value, a URI? The variable should be meaningful to the user.

Suggested revised explanation for CWWKO1714W:
"The server did not complete an HTTP write operation within the configured write timeout."

Similar comment for {0}

Suggested explanation for CWWKO1715W:

The HTTP/2 connection was idle for longer than the configured idle timeout. The server closed the connection.

Similar comment for {0}

volosied · 2026-09-16T13:14:04Z

@michael-bowler-232 I actually commented these messages out. I'll still update them, and decide whether to use them later or not.

They are used only in beta as of now.

co authored by bob 2.0.2 in progress IN PROGERSS Working timeout tests more updates Change warning to trace message to better align netty with chfw Working write timeout test

volosied · 2026-09-16T20:26:08Z

!build
(view Open Liberty Personal Build - ❌ completed with errors/failures.)
spawn.fullfat.buckets=io.openliberty.transport.http_fat
extra.fat.ant.options=-Dglobal.jvm.args="-Dcom.ibm.ws.beta.edition=true"

Note: Target locations of links might be accessible only to IBM employees.

volosied · 2026-09-16T20:26:13Z

!build
spawn.fullfat.buckets=io.openliberty.transport.http_fat

LibbyBot · 2026-09-16T20:41:14Z

Code analysis and actions

DO NOT DELETE THIS COMMENT.

2 FAT files were changed, added, or removed.
Check that the build did not break the affected FAT suite(s).
1 test infrastructure code files were changed.
Test failures/errors in the build could be due to these changes.
5 product code files were changed.
Please describe in a separate comment how you tested your changes.
1 messages files were changed and need an L2 review.
@OpenLiberty/message-reviewer Please review.
- dev/com.ibm.ws.channelfw/resources/com/ibm/ws/tcpchannel/internal/resources/TCPChannelMessages.nlsprops
1 NLS files were changed and need an ID review.
@OpenLiberty/message-reviewer Please review.
- dev/com.ibm.ws.channelfw/resources/com/ibm/ws/tcpchannel/internal/resources/TCPChannelMessages.nlsprops

Skip testWriteTimeout_throwIOEForInboundConnections for now

volosied · 2026-09-17T14:10:40Z

!build
(view Open Liberty Personal Build - ⏳InProgress)
spawn.fullfat.buckets=io.openliberty.transport.http_fat
extra.fat.ant.options=-Dglobal.jvm.args="-Dcom.ibm.ws.beta.edition=true"

Note: Target locations of links might be accessible only to IBM employees.

LibbyBot · 2026-09-17T14:18:55Z

Code analysis and actions

DO NOT DELETE THIS COMMENT.

2 FAT files were changed, added, or removed.
Check that the build did not break the affected FAT suite(s).
1 test infrastructure code files were changed.
Test failures/errors in the build could be due to these changes.
5 product code files were changed.
Please describe in a separate comment how you tested your changes.
1 messages files were changed and need an L2 review.
@OpenLiberty/message-reviewer Please review.
- dev/com.ibm.ws.channelfw/resources/com/ibm/ws/tcpchannel/internal/resources/TCPChannelMessages.nlsprops
1 NLS files were changed and need an ID review.
@OpenLiberty/message-reviewer Please review.
- dev/com.ibm.ws.channelfw/resources/com/ibm/ws/tcpchannel/internal/resources/TCPChannelMessages.nlsprops

volosied · 2026-09-17T16:58:56Z

!build (view Open Liberty Personal Build - ⏳InProgress)

Note: Target locations of links might be accessible only to IBM employees.

LibbyBot · 2026-09-17T17:07:29Z

Code analysis and actions

DO NOT DELETE THIS COMMENT.

2 FAT files were changed, added, or removed.
Check that the build did not break the affected FAT suite(s).
1 test infrastructure code files were changed.
Test failures/errors in the build could be due to these changes.
5 product code files were changed.
Please describe in a separate comment how you tested your changes.
1 messages files were changed and need an L2 review.
@OpenLiberty/message-reviewer Please review.
- dev/com.ibm.ws.channelfw/resources/com/ibm/ws/tcpchannel/internal/resources/TCPChannelMessages.nlsprops
1 NLS files were changed and need an ID review.
@OpenLiberty/message-reviewer Please review.
- dev/com.ibm.ws.channelfw/resources/com/ibm/ws/tcpchannel/internal/resources/TCPChannelMessages.nlsprops

volosied added this to Netty in Open Liberty Sep 2, 2026

pnicolucci assigned volosied Sep 14, 2026

volosied force-pushed the timeout-tests branch from a8d639f to 98810b5 Compare September 16, 2026 01:25

LibbyBot added the CLA Signed label Sep 16, 2026

michael-bowler-232 reviewed Sep 16, 2026

View reviewed changes

Create read and write timeout tests

af5ba35

co authored by bob 2.0.2 in progress IN PROGERSS Working timeout tests more updates Change warning to trace message to better align netty with chfw Working write timeout test

volosied force-pushed the timeout-tests branch 2 times, most recently from a71f0f9 to f720308 Compare September 16, 2026 20:25

Work on testWriteTimeout_throwIOEForInboundConnections

9019765

Skip testWriteTimeout_throwIOEForInboundConnections for now

volosied force-pushed the timeout-tests branch from f720308 to 9019765 Compare September 17, 2026 14:10

Navigation Menu

Create read and write timeout tests for netty / chfw - #35680

Create read and write timeout tests for netty / chfw#35680
volosied wants to merge 2 commits into
OpenLiberty:integrationfrom
volosied:timeout-tests

volosied commented Sep 2, 2026 •

edited by pnicolucci

Loading

Uh oh!

volosied commented Sep 16, 2026 •

edited by LibbyBot

Loading

Uh oh!

LibbyBot commented Sep 16, 2026

Uh oh!

michael-bowler-232 left a comment

Uh oh!

volosied commented Sep 16, 2026

Uh oh!

volosied commented Sep 16, 2026 •

edited by LibbyBot

Loading

Uh oh!

volosied commented Sep 16, 2026

Uh oh!

LibbyBot commented Sep 16, 2026

Uh oh!

volosied commented Sep 17, 2026 •

edited by LibbyBot

Loading

Uh oh!

LibbyBot commented Sep 17, 2026

Uh oh!

volosied commented Sep 17, 2026 •

edited by LibbyBot

Loading

Uh oh!

LibbyBot commented Sep 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

volosied commented Sep 2, 2026 • edited by pnicolucci Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

volosied commented Sep 16, 2026 • edited by LibbyBot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

LibbyBot commented Sep 16, 2026

Code analysis and actions

Uh oh!

michael-bowler-232 left a comment

Choose a reason for hiding this comment

Uh oh!

volosied commented Sep 16, 2026

Uh oh!

volosied commented Sep 16, 2026 • edited by LibbyBot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

volosied commented Sep 16, 2026

Uh oh!

LibbyBot commented Sep 16, 2026

Code analysis and actions

Uh oh!

volosied commented Sep 17, 2026 • edited by LibbyBot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

LibbyBot commented Sep 17, 2026

Code analysis and actions

Uh oh!

volosied commented Sep 17, 2026 • edited by LibbyBot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

LibbyBot commented Sep 17, 2026

Code analysis and actions

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

volosied commented Sep 2, 2026 •

edited by pnicolucci

Loading

volosied commented Sep 16, 2026 •

edited by LibbyBot

Loading

volosied commented Sep 16, 2026 •

edited by LibbyBot

Loading

volosied commented Sep 17, 2026 •

edited by LibbyBot

Loading

volosied commented Sep 17, 2026 •

edited by LibbyBot

Loading