Page MenuHomeFreeBSD
Differential D50499

kyua: increase cleanup_timeout to 300s
ClosedPublic
Actions

Authored by ivy on May 23 2025, 8:11 PM.
Tags
None
Referenced Files
F138015884: D50499.diff
Fri, Nov 28, 4:32 AM
Unknown Object (File)
Wed, Nov 26, 2:54 AM
Unknown Object (File)
Mon, Nov 3, 12:07 AM
Unknown Object (File)
Sun, Nov 2, 1:31 AM
Unknown Object (File)
Oct 29 2025, 6:07 AM
Unknown Object (File)
Oct 29 2025, 4:56 AM
Unknown Object (File)
Oct 27 2025, 2:30 AM
Unknown Object (File)
Oct 26 2025, 10:26 PM
View All 57 Files
Subscribers
imp
ngie

Details

Reviewers
des
kevans
kp
igoro
Group Reviewers
tests
Commits
rGbc6ec9642b5a: kyua: increase cleanup_timeout to 300s
rG0e33211f911c: kyua: increase cleanup_timeout to 300s
Summary

Test cleanup times out after 60 seconds, causing the test to fail if the
cleanup hasn't finished in time. This timeout should be configurable,
but it currently is not.

Several tests (e.g. bridge, pf) are now reaching this limit; as a
workaround, increase the timeout to 300 seconds.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 64393
Build 61277: arc lint + arc unit

Event Timeline

ivy created this revision.May 23 2025, 8:11 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 23 2025, 8:11 PM
ivy requested review of this revision.May 23 2025, 8:11 PM
Harbormaster completed remote builds in B64393: Diff 155945.May 23 2025, 8:12 PM
ivy added a child revision: D50476: tests/sys/net: run if_bridge_test in a jail.May 23 2025, 8:12 PM
des accepted this revision.May 23 2025, 9:02 PM
des added a subscriber: ngie.

kinda ironic that you had five minutes to list your tests, but only 60 seconds to clean up after running one...

lgtm, but please give @ngie or someone else from tests a reasonable chance to object.

This revision is now accepted and ready to land.May 23 2025, 9:02 PM
ivy mentioned this in D50476: tests/sys/net: run if_bridge_test in a jail.May 23 2025, 9:44 PM
ivy added a reviewer: igoro.
igoro accepted this revision as: igoro.May 24 2025, 11:28 AM
kevans accepted this revision as: kevans.Jun 4 2025, 4:00 AM

Please upstream this to github:freebsd/kyua as well, though I think it's fine to land it here first.

Closed by commit rG0e33211f911c: kyua: increase cleanup_timeout to 300s (authored by ivy). · Explain WhyJun 4 2025, 6:23 AM
This revision was automatically updated to reflect the committed changes.
ivy added a commit: rG0e33211f911c: kyua: increase cleanup_timeout to 300s.
ivy added a commit: rGbc6ec9642b5a: kyua: increase cleanup_timeout to 300s.Jun 29 2025, 9:00 PM

Revision Contents
Changeset List

PathSize
contrib/
kyua/
engine/
4 lines

Diff 155945

View Options

contrib/kyua/engine/scheduler.cpp

Loading...