svn commit: r343543 - head/sbin/bectl/tests

[Enji Cooper]

> On Jan 29, 2019, at 9:21 AM, Enji Cooper <yaneurabeya at gmail.com> wrote:

…

>> Right- this is the part that makes sense to me. =) The 'default2' BE
>> is clearly a leftover from a previous test, but it should've been
>> unmounted in the process and the zpool cleaned up.
>> 
>> What's not clear is how the cleanup routine is failing; the `zpool get
>> health` shouldn't have any reason to fail (though I can't test it
>> here, apparently =() and `zpool destroy -f` should've unmounted this
>> BE even if that hadn't happened throughout the course of the test, but
>> the failure remains (even if probably incorrectly fixed) following the
>> above commit.
>> 
>> Setting the zpool name globally with a suffix unique to this test
>> would likely be a great idea -- I think there's something else going
>> on as well, though.
> 
> My comment about the zpool name being defined locally to tests as opposed is globally is probably what’s tripping you up. The cleanup function is not run in the same shell context as the test.

I submitted a proposed fix here: https://reviews.freebsd.org/D19024 <https://reviews.freebsd.org/D19024> .

After doing some more thinking, it dawned on me that these tests might not be running in parallel, which (because they weren’t using unique names) could be clashing with parallel executing tests.

-Enji