search

fluent / fluentd

Fluentd: Unified Logging Layer (project under CNCF)
https://www.fluentd.org
Apache License 2.0
12.92k stars 1.34k forks source link

test_in_udp: add timeout for message_length_limit test #4676

Closed Watson1978 closed 1 month ago

Watson1978 commented 1 month ago

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:

ashie commented 1 month ago

Memo: On windows, data over message_length_limit will be lost completely: https://github.com/fluent/fluentd/pull/4117#issuecomment-1488673996