Page MenuHomeFreeBSD

kyua: Add flaky metadata
Needs RevisionPublic

Authored by igoro on Dec 6 2025, 6:40 PM.
Tags
None
Referenced Files
F144828914: D54105.diff
Fri, Feb 13, 4:32 AM
Unknown Object (File)
Sat, Jan 17, 8:35 PM
Unknown Object (File)
Sat, Jan 17, 5:21 PM
Unknown Object (File)
Thu, Jan 15, 5:06 PM
Unknown Object (File)
Jan 14 2026, 2:57 PM
Unknown Object (File)
Jan 14 2026, 12:03 PM
Unknown Object (File)
Jan 12 2026, 11:15 PM
Unknown Object (File)
Jan 10 2026, 4:45 PM
Subscribers

Details

Reviewers
kp
ngie
Group Reviewers
tests

Diff Detail

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

Event Timeline

igoro requested review of this revision.Dec 6 2025, 6:40 PM
contrib/kyua/engine/flaky/tracker.cpp
52

Dumb question: isn't oneofn() just a specific version of xofy()?

contrib/kyua/engine/flaky/tracker.cpp
52

I expected you would catch it :) Yes, it's just 1/n. I've got the same idea after adding xofy. It's just too clumsy to add/remove a class or something in kyua, I would drop a thing only after some discussion and common decision made. Anyway, I would keep the general mechanism so that it's much easier to add a new tracker -- it follows kyua's design and really makes someone's life easier.

I guess the only pros to keep oneofn is the simplest specification with a single integer, but, probably, explicit "1/<integer>" specification is self-explainable. Because of that I would vote to eventually drop oneofn.

ngie requested changes to this revision.EditedDec 26 2025, 6:34 PM

Please provide an example (spec) of what you're implementing in the Testplan section.

This revision now requires changes to proceed.Dec 26 2025, 6:34 PM