Please provide process for small, targeted fixes in tools/regression
casey.peel at isilon.com
Thu May 1 22:44:35 UTC 2014
> 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?
To be honest, I haven't tested on a stock FreeBSD system, but on our
internal almost-but-not-quite FreeBSD-CURRENT system removing the last
partition of a gmirror fails. To remove the last partition you have to
destroy the gmirror. It should be easy to confirm this is still true on
FreeBSD-CURRENT. That said, I have the gmirror tests entirely moved over
to ATF so I'm not sure its worth messing with.
> These changes look ok. The only thing (just a consistency nit) is that
> I would choose a consistent way of incrementing in loops, i.e.
Good point, will do!
Performance Test Ninja
casey.peel at isilon.com / 206.777.7945
On 4/30/14 10:16 AM, "Garrett Cooper" <yaneurabeya at gmail.com> wrote:
>On Fri, Apr 11, 2014 at 6:59 PM, Peel, Casey <casey.peel at isilon.com>
>> Exellent, thanks Julio.
>> Attached are 3 self-contained diffs, one per directory. Descriptions of
>> * 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
>> * 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?
More information about the freebsd-testing