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
Unknown Object (File)
Fri, Sep 11, 10:14 AM
Unknown Object (File)
Thu, Sep 10, 9:34 PM
Unknown Object (File)
Wed, Sep 9, 10:35 AM
Unknown Object (File)
Tue, Sep 8, 1:37 AM
Unknown Object (File)
Sun, Sep 6, 9:09 PM
Unknown Object (File)
Fri, Sep 4, 3:30 PM
Unknown Object (File)
Wed, Sep 2, 11:32 AM
Unknown Object (File)
Tue, Sep 1, 10:22 PM
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...