On Wed, Sep 23, 2015 at 02:05:16PM -0700, Mark Fasheh wrote: > Since the last time I sent this test, drop snapshot was broken again with > respect to qgroups. What practical step could I take to get a test for that > in here which I can beat the btrfs developers over the head with the next > time someone handwaves this problem away ;)
I'll merge tests that break a filesystem as a reminder to developers that there is a problem that needs fixing. We do that from time to time for XFS issues that are either really hard to fix or not urgent but require significant amounts of work to correct... > From: Mark Fasheh <[email protected]> > > [PATCH] btrfs: add test for quota groups and drop snapshot .... > +# NOTE: The ability to vary tree height for this test is very useful > +# for debugging problems with drop_snapshot(). As a result we retain > +# that parameter even though the test below always does level 2 trees. > +_explode_fs_tree () { > + local level=$1; > + local loc="$2"; > + local n; > + > + if [ -z $loc ]; then > + echo "specify location for fileset" > + exit 1; > + fi > + > + case $level in > + 1)# this always reproduces level 1 trees > + n=10; > + ;; > + 2)# this always reproduces level 2 trees > + n=1500 Still some minor whitespace issues, but I can fix that on commit as everything else looks fine. Cheers, Dave. -- Dave Chinner [email protected] -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
