Page MenuHomeFreeBSD

syslogd/tests: Fix flakiness in forwarding tests
AcceptedPublic

Authored by markj on Tue, Jan 20, 5:10 PM.
Tags
None
Referenced Files
F142577805: D54800.id170139.diff
Wed, Jan 21, 5:19 AM
F142576348: D54800.id.diff
Wed, Jan 21, 4:56 AM
F142566703: D54800.diff
Wed, Jan 21, 2:18 AM
F142560111: D54800.id.diff
Wed, Jan 21, 12:25 AM
F142559633: D54800.id170147.diff
Wed, Jan 21, 12:18 AM
F142558132: D54800.id170139.diff
Tue, Jan 20, 11:53 PM
Unknown Object (File)
Tue, Jan 20, 8:11 PM
Unknown Object (File)
Tue, Jan 20, 5:50 PM
Subscribers

Details

Reviewers
jlduran
jfree
Summary

syslogd_start() waits for the local log socket to appear before
returning, to ensure that the daemon is ready to handle log messages.
Some tests start two daemons, so by default the socket already exists
when the second daemon is started, so syslogd_start() returns early.
The test subsequently sends a message to this second daemon, which
sometimes isn't ready.

Define a separate log socket for the second daemon. Add a check to
syslogd_start() to help catch this type of bug.

Diff Detail

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

Event Timeline

I've seen that when using ATF, ${PWD} is preferred over $(pwd) (efficiency, I guess).

This revision is now accepted and ready to land.Tue, Jan 20, 5:58 PM
This revision now requires review to proceed.Tue, Jan 20, 7:28 PM
This revision is now accepted and ready to land.Tue, Jan 20, 7:52 PM