Re: [GIT PULL] fstests: btrfs changes for for-next staged-20250328

[Anand Jain <anand.jain@oracle.com>]

On 28/3/25 10:03, Zorro Lang wrote:
> On Fri, Mar 28, 2025 at 09:26:24AM +0800, Anand Jain wrote:
>> Zorro,
>>
>> Please pull this branch, which includes test cases for sysfs syntax
>> verification of btrfs read policy and chunk size. v4 has been on the
>> mailing list for a month now, along with fix from Filipe and Zoned
>> testcase from Johannes.
>>
>> Please note that the commit:
>>   "fstests: btrfs: zoned: verify RAID conversion with write pointer mismatch"
>>
>> has the changes discussed with Naohiro, including his review-by tag,
>> (which is missing in your patches-in-queue branch).
>>
>> Test case number for above commit is changed to btrfs/335 following
>> the integration of the sysfs patches.
>>
>> Thank you.
>>
>> The following changes since commit d71157da4ef4cfdbf39e2c4a07f8013633e6bcbe:
>>
>>    common/rc: explicitly test for engine availability in _require_fio (2025-03-17 00:43:12 +0800)
>>
>> are available in the Git repository at:
>>
>>    https://github.com/asj/fstests/tree/staged-20250328
>>
>> for you to fetch changes up to 208a7f874df38bf873137d00634783422965a7ab:
>>
>>    fstests: btrfs: zoned: verify RAID conversion with write pointer mismatch (2025-03-28 08:25:55 +0800)
>>
>> ----------------------------------------------------------------
>> Anand Jain (5):
>>        fstests: common/rc: set_fs_sysfs_attr: redirect errors to stdout
>>        fstests: filter: helper for sysfs error filtering
>>        fstests: common/rc: add sysfs argument verification helpers
>>        fstests: btrfs: testcase for sysfs policy syntax verification
>>        fstests: btrfs: testcase for sysfs chunk_size attribute validation
> 
> Hi Anand, these 5 patches don't have any RVBs or ACKs. Do you miss that?
> Although you can ack patches by yourself, but these patches are from you,
> maintainers would better not push their own patches directly without any
> RVBs. So please let someone review and ack them at first.

Dave Chinner provided comments on the generic patches, leading to v4.

No, there hasn’t been any RB received. V4 has been on the ML for almost 
a month now. Let me try again.

>>
>> Filipe Manana (1):
>>        btrfs/058: fix test to actually have an open tmpfile during the send operation
>>
>> Johannes Thumshirn (1):
>>        fstests: btrfs: zoned: verify RAID conversion with write pointer mismatch
> 
> I'm going to merge these two patches to the release of this week, and give them
> regression test at first.
> 

Thanks!.
Anand


> Thanks,
> Zorro
> 
>>
>>   common/filter       |   9 ++++
>>   common/rc           |   3 +-
>>   common/sysfs        | 142 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>>   tests/btrfs/058     |  28 +++++++++--
>>   tests/btrfs/329     |  19 +++++++
>>   tests/btrfs/329.out |  19 +++++++
>>   tests/btrfs/334     |  19 +++++++
>>   tests/btrfs/334.out |  14 ++++++
>>   tests/btrfs/335     |  62 +++++++++++++++++++++++
>>   tests/btrfs/335.out |   7 +++
>>   10 files changed, 317 insertions(+), 5 deletions(-)
>>   create mode 100644 common/sysfs
>>   create mode 100755 tests/btrfs/329
>>   create mode 100644 tests/btrfs/329.out
>>   create mode 100755 tests/btrfs/334
>>   create mode 100644 tests/btrfs/334.out
>>   create mode 100755 tests/btrfs/335
>>   create mode 100644 tests/btrfs/335.out
>>
>