mbox series

[v2,0/3] fstests: various RAID56 related fixes for btrfs

Message ID cover.1712346845.git.josef@toxicpanda.com (mailing list archive)
Headers show
Series fstests: various RAID56 related fixes for btrfs | expand

Message

Josef Bacik April 5, 2024, 7:56 p.m. UTC
v1: https://lore.kernel.org/linux-btrfs/cover.1710867187.git.josef@toxicpanda.com/

v1->v2:
- Reworked the golden output of btrfs/197 and btrfs/198 to handle running some
  of the RAID profiles.
- Added a new way to require RAID profiles in the tests, updated all the
  different tests.
- Changed btrfs/197, btrfs/198, and btrfs/297 to skip any RAID profiles that
  aren't supported by our RAID profile settings.

--- Original email ---
Hello,

While trying to get CI setup internally I noticed that we were sometimes failing
raid56 tests even though I had specified BTRFS_PROFILE_CONFIGS without raid56 in
them.

This is because tests where we require raid56 to work only check to see if it's
enabled by the kernel, not to check and see if we're configured to use it with
our profile configs.

One test needed to be updated to skip any configurations that weren't in our
profile configs, and then a few tests didn't use the 

_require_btrfs_fs_feature raid56

check in the top of their test.  This series fixes everything up and honors the
user settings which makes my internal CI runs clean where we don't want to test
raid56.  Thanks,

Josef

Josef Bacik (3):
  fstests: change btrfs/197 and btrfs/198 golden output
  fstests: change how we test for supported raid configs
  fstests: update tests to skip unsupported raid profile types

 common/btrfs        | 29 ++++++++++++++++++++++++-----
 tests/btrfs/125     |  2 +-
 tests/btrfs/148     |  3 ++-
 tests/btrfs/157     |  2 +-
 tests/btrfs/158     |  2 +-
 tests/btrfs/197     | 17 +++++++++++++----
 tests/btrfs/197.out | 25 +------------------------
 tests/btrfs/198     | 17 +++++++++++++----
 tests/btrfs/198.out | 25 +------------------------
 tests/btrfs/297     | 10 ++++++++++
 10 files changed, 67 insertions(+), 65 deletions(-)