Skip to content

Commit 0b2126a

Browse files
kenhysWatson1978
andauthoredJan 29, 2025··
Backport(v1.16) test_out_forward: remove unnecessary ack_response_timeout setting (#4685) (#4791)
**Which issue(s) this PR fixes**: Backport #4685 **What this PR does / why we need it**: Seems that timeout setting is short in ack_response_timeout. Seems that It may take some time to receive a ACK response so the process in ack handler has expired and the node is disabled. This PR will remove unnecessary ack_response_timeout settings for the test **Docs Changes**: **Release Note**: Signed-off-by: Watson <fujita@clear-code.com> Signed-off-by: Shizuo Fujita <fujita@clear-code.com> Co-authored-by: Watson <fujita@clear-code.com>
1 parent 60c969a commit 0b2126a

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed
 

‎test/plugin/test_out_forward.rb

-2
Original file line numberDiff line numberDiff line change
@@ -611,7 +611,6 @@ def try_write(chunk)
611611

612612
@d = d = create_driver(config + %[
613613
require_ack_response true
614-
ack_response_timeout 1s
615614
<buffer tag>
616615
flush_mode immediate
617616
retry_type periodic
@@ -659,7 +658,6 @@ def try_write(chunk)
659658

660659
@d = d = create_driver(config + %[
661660
require_ack_response true
662-
ack_response_timeout 10s
663661
<buffer tag>
664662
flush_mode immediate
665663
retry_type periodic

0 commit comments

Comments
 (0)
Please sign in to comment.