Re: [PATCH 3/3 V4] xfstests: btrfs tests for basic informational commands

2013-03-19 Thread Rich Johnston
This has been reverted and recommitted as test 307. Thanks --Rich commit 4f092a2e681c57394a0055a0735b8208bf83ec5f Author: Eric Sandeen sand...@sandeen.net Date: Wed Mar 13 19:01:58 2013 + xfstests: btrfs tests for basic informational commands Run basic btrfs information

Re: [PATCH 3/3 V4] xfstests: btrfs tests for basic informational commands

2013-03-18 Thread Rich Johnston
Dave, Sorry for skipping test numbers, I was going to revert, correct the test number and resubmit this patch. Do you want me to rename btrfs.filter to common.filter.btrfs when I resubmit this patch? Regards --Rich -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in

Re: [PATCH 3/3 V4] xfstests: btrfs tests for basic informational commands

2013-03-15 Thread Dave Chinner
On Wed, Mar 13, 2013 at 02:01:58PM -0500, Eric Sandeen wrote: Run basic btrfs information commands in various ways, performing sanity tests of: filesystem show, label, sync, and device stats (sync is included just because it's simple). These are mostly just smoke tests, although for example

Re: [PATCH 3/3 V4] xfstests: btrfs tests for basic informational commands

2013-03-15 Thread Eric Sandeen
Yep sorry. I figure its always a race in numbers do I don't bother with fixing them up. Should I? On Mar 15, 2013, at 5:16 AM, Dave Chinner da...@fromorbit.com wrote: On Wed, Mar 13, 2013 at 02:01:58PM -0500, Eric Sandeen wrote: Run basic btrfs information commands in various ways,

Re: [PATCH 3/3 V4] xfstests: btrfs tests for basic informational commands

2013-03-15 Thread Rich Johnston
On 03/15/2013 08:46 AM, Eric Sandeen wrote: Yep sorry. I figure its always a race in numbers do I don't bother with fixing them up. Should I? Eric, I was going to revert and resubmit as test 306, is that OK with you? On Mar 15, 2013, at 5:16 AM, Dave Chinner da...@fromorbit.com wrote:

Re: [PATCH 3/3 V4] xfstests: btrfs tests for basic informational commands

2013-03-15 Thread Eric Sandeen
On Mar 15, 2013, at 9:23 AM, Rich Johnston rjohns...@sgi.com wrote: On 03/15/2013 08:46 AM, Eric Sandeen wrote: Yep sorry. I figure its always a race in numbers do I don't bother with fixing them up. Should I? Eric, I was going to revert and resubmit as test 306, is that OK with you?

Re: [PATCH 3/3 V4] xfstests: btrfs tests for basic informational commands

2013-03-14 Thread Rich Johnston
Stefan, Thanks for reviewing this patch, may I put your Reviewed-by: on this version? Thanks --Rich -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 3/3 V4] xfstests: btrfs tests for basic informational commands

2013-03-14 Thread Stefan Behrens
On Thu, 14 Mar 2013 08:01:54 -0500, Rich Johnston wrote: Thanks for reviewing this patch, may I put your Reviewed-by: on this version? Yes. Thanks. Reviewed-by: Stefan Behrens sbehr...@giantdisaster.de -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a

[PATCH 3/3 V4] xfstests: btrfs tests for basic informational commands

2013-03-13 Thread Eric Sandeen
Run basic btrfs information commands in various ways, performing sanity tests of: filesystem show, label, sync, and device stats (sync is included just because it's simple). These are mostly just smoke tests, although for example show by label UUID should verify that the correct fs was shown.