From: Anand Jain <anand.jain@oracle.com>
To: Wang Shilong <wangshilong1991@gmail.com>
Cc: xfs@oss.sgi.com, linux-btrfs@vger.kernel.org
Subject: Re: [PATCH 1/2] xfstests: make btrfs/001.out in line with subvol delete output
Date: Mon, 13 Jan 2014 21:27:50 +0800 [thread overview]
Message-ID: <52D3E9D6.2030104@oracle.com> (raw)
In-Reply-To: <5A6081D2-68E6-47BC-A534-6502B680DDDD@gmail.com>
Hi Wang,
Ah yes. why did I miss that point earlier.
Will fix it.
Thanks, Anand
On 13/01/2014 21:14, Wang Shilong wrote:
> Hello Anand,
>
> I was noticing this problem when using the latest btrfs-progs to test ^_^
>
> I think a better way is to filter message 'Transaction commit: none (default)'
> Rather than add it, otherwise, people will fail this test if they use previous
> btrfs-progs……
>
> Thanks,
> Wang
>> From: Anand Jain <Anand.Jain@oracle.com>
>>
>> btrfs/001 is failing as below btrfs-progs patch changed the
>> output during subvol delete.
>> Patch :
>> btrfs-progs: add options to set commit mode after subvol delete
>>
>> so add it to the btrfs/001.out
>>
>> Signed-off-by: Anand Jain <Anand.Jain@oracle.com>
>> ---
>> tests/btrfs/001.out | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/tests/btrfs/001.out b/tests/btrfs/001.out
>> index c782bde..a255614 100644
>> --- a/tests/btrfs/001.out
>> +++ b/tests/btrfs/001.out
>> @@ -33,6 +33,7 @@ subvol
>> Listing subvolumes
>> snap
>> subvol
>> +Transaction commit: none (default)
>> Delete subvolume 'SCRATCH_MNT/snap'
>> List root dir
>> subvol
>> --
>> 1.8.4.2
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
WARNING: multiple messages have this Message-ID (diff)
From: Anand Jain <anand.jain@oracle.com>
To: Wang Shilong <wangshilong1991@gmail.com>
Cc: linux-btrfs@vger.kernel.org, xfs@oss.sgi.com
Subject: Re: [PATCH 1/2] xfstests: make btrfs/001.out in line with subvol delete output
Date: Mon, 13 Jan 2014 21:27:50 +0800 [thread overview]
Message-ID: <52D3E9D6.2030104@oracle.com> (raw)
In-Reply-To: <5A6081D2-68E6-47BC-A534-6502B680DDDD@gmail.com>
Hi Wang,
Ah yes. why did I miss that point earlier.
Will fix it.
Thanks, Anand
On 13/01/2014 21:14, Wang Shilong wrote:
> Hello Anand,
>
> I was noticing this problem when using the latest btrfs-progs to test ^_^
>
> I think a better way is to filter message 'Transaction commit: none (default)'
> Rather than add it, otherwise, people will fail this test if they use previous
> btrfs-progs……
>
> Thanks,
> Wang
>> From: Anand Jain <Anand.Jain@oracle.com>
>>
>> btrfs/001 is failing as below btrfs-progs patch changed the
>> output during subvol delete.
>> Patch :
>> btrfs-progs: add options to set commit mode after subvol delete
>>
>> so add it to the btrfs/001.out
>>
>> Signed-off-by: Anand Jain <Anand.Jain@oracle.com>
>> ---
>> tests/btrfs/001.out | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/tests/btrfs/001.out b/tests/btrfs/001.out
>> index c782bde..a255614 100644
>> --- a/tests/btrfs/001.out
>> +++ b/tests/btrfs/001.out
>> @@ -33,6 +33,7 @@ subvol
>> Listing subvolumes
>> snap
>> subvol
>> +Transaction commit: none (default)
>> Delete subvolume 'SCRATCH_MNT/snap'
>> List root dir
>> subvol
>> --
>> 1.8.4.2
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next prev parent reply other threads:[~2014-01-13 13:27 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-13 13:16 [PATCH 1/2] xfstests: make btrfs/001.out in line with subvol delete output Anand Jain
2014-01-13 13:16 ` Anand Jain
2014-01-13 13:14 ` Wang Shilong
2014-01-13 13:14 ` Wang Shilong
2014-01-13 13:27 ` Anand Jain [this message]
2014-01-13 13:27 ` Anand Jain
2014-01-13 13:16 ` [PATCH 2/2] xfstest: tests btrfs/006 fails with mixed-mode/small disks Anand Jain
2014-01-13 13:16 ` Anand Jain
2014-01-14 3:23 ` [PATCH 1/2 v2] xfstests: make btrfs/001.out in line with subvol delete output Anand Jain
2014-01-15 14:19 ` David Sterba
2014-01-16 3:03 ` [PATCH 1/2 v3] " Anand Jain
2014-01-16 3:04 ` Anand Jain
2014-01-16 3:04 ` Anand Jain
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52D3E9D6.2030104@oracle.com \
--to=anand.jain@oracle.com \
--cc=linux-btrfs@vger.kernel.org \
--cc=wangshilong1991@gmail.com \
--cc=xfs@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.