Page MenuHomeFreeBSD
Differential D21519

procstat/tests: Fix flakiness by waiting for program to start
ClosedPublic
Actions

Authored by jilles on Sep 4 2019, 2:00 PM.
Tags
None
Referenced Files
F149535717: D21519.id61650.diff
Wed, Mar 25, 2:30 AM
Unknown Object (File)
Tue, Mar 24, 1:25 AM
Unknown Object (File)
Fri, Mar 20, 12:23 AM
Unknown Object (File)
Fri, Mar 20, 12:23 AM
Unknown Object (File)
Thu, Mar 19, 7:23 AM
Unknown Object (File)
Tue, Mar 17, 5:57 AM
Unknown Object (File)
Tue, Mar 17, 5:57 AM
Unknown Object (File)
Sun, Mar 15, 11:40 AM
View All Files
Subscribers
imp

Details

Reviewers
ngie
Commits
rS351819: procstat/tests: Fix flakiness by waiting for program to start
Summary

Some of the procstat tests start a program "while1" and examine the process
using procstat, but did not wait properly for it to start (kill -0 will
succeed immediately after the child process has been created).

Instead, have "while1" write something when it starts, and use a fifo to
wait for that.

PR: 233587, 233588

Test Plan

sh -c 'while kyua test; do :; done'
and let it continue for a while

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jilles created this revision.Sep 4 2019, 2:00 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 4 2019, 2:00 PM
Harbormaster completed remote builds in B26272: Diff 61645.Sep 4 2019, 2:00 PM
jilles updated this revision to Diff 61646.Sep 4 2019, 2:06 PM

Remove no longer used variable MAX_TRIES

Harbormaster completed remote builds in B26273: Diff 61646.Sep 4 2019, 2:06 PM
ngie accepted this revision.Sep 4 2019, 2:20 PM

Thank you for fixing my flaky test (and for teaching me a new tidbit about shell support)!

This revision is now accepted and ready to land.Sep 4 2019, 2:20 PM
Closed by commit rS351819: procstat/tests: Fix flakiness by waiting for program to start (authored by jilles). · Explain WhySep 4 2019, 4:26 PM
This revision was automatically updated to reflect the committed changes.
jilles added a commit: rS351819: procstat/tests: Fix flakiness by waiting for program to start.

Revision Contents
Changeset List

PathSize
head/
usr.bin/
procstat/
tests/
18 lines
5 lines

Diff 61650

View Options

head/usr.bin/procstat/tests/procstat_test.sh

Loading...
View Options

head/usr.bin/procstat/tests/while1.c

Loading...