> On Jan 29, 2019, at 9:21 AM, Enji Cooper <yaneurab...@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 _______________________________________________ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
