Please provide process for small, targeted fixes in tools/regression

Garrett Cooper yaneurabeya at gmail.com
Wed Apr 30 17:16:11 UTC 2014


On Fri, Apr 11, 2014 at 6:59 PM, Peel, Casey <casey.peel at isilon.com> wrote:
> Exellent, thanks Julio.
>
> Attached are 3 self-contained diffs, one per directory. Descriptions of the changes:
> * flock - .c changes to fix compiler errors; .t to wrap flock binary for integrating with prove
> * geom_mirror - gmirror won't let you remove the last partition in a mirror which all of these testcases do and thus fail (note that I've got these testcases plus other gmirror testcases already moved over to ATF and going through internal code reviews now, so maybe we just want to discard this)
> * kqueue - kqtest outputs files in TAP-consumable format, this is a .t file to run it through prove
>
> These were an intial salvo at getting existing tests working for our internal regression. I'm sure we'll be looking at others as well. If someone can just review the changes, I can get someone internally to commit them if necssary (although these will take more time to review than commit given how small they are).

    The test drivers seem ok for TAP, but out of curiosity why are you
removing the geom remove testing in the geom_mirror testcases? Does
the test not work out of the box on vanilla FreeBSD?
Thanks!
-Garrett


More information about the freebsd-testing mailing list