fluentd icon indicating copy to clipboard operation
fluentd copied to clipboard

test_in_udp: add timeout for message_length_limit test

Open Watson1978 opened this issue 1 year ago • 0 comments

Which issue(s) this PR fixes: Fixes #

What this PR does / why we need it: On Windows, message_length_limit test always take 300s for execution.

UdpInputTest:
  test:  configure w/o parse section:			.: (0.002351)
  test: configure[ipv4]:				.: (0.002723)
  test: configure[ipv6]:				.: (0.002602)
  test: message size with format[none]:			.: (1.029781)
  test: message size with format[json]:			.: (1.113799)
  test: message size with format[regexp]:		.: (1.110006)
  test: message_length_limit:				.: (300.538596)

The 300 sec comes from https://github.com/fluent/fluentd/blob/a2b935ae2bc4b4d43e5adddbec01092ea4228b9e/lib/fluent/test/driver/base.rb#L36, and it always times out in Windows.

This patch set a short timeout to reduce test execution time on Windows. Docs Changes:

Release Note:

Watson1978 avatar Oct 20 '24 08:10 Watson1978