From: Andrew Hamilton <adhamilt@gmail.com>
To: grub-devel@gnu.org
Cc: daniel.kiper@oracle.com, lsandova@redhat.com,
development@efficientek.com, Andrew Hamilton <adhamilt@gmail.com>
Subject: [PATCH v7 0/3] Fix Several Test Failures
Date: Fri, 14 Nov 2025 17:40:07 -0600 [thread overview]
Message-ID: <20251114234012.32925-1-adhamilt@gmail.com> (raw)
Correct several test failures observed while running GRUB 'make check'
on Debian 13. The test failures fixed here are:
1. ZFS ZSTD - Test was failing zfs_test due to lack of support in
zfs-fuse for zstd compression. This specific test case was split
into a new test file allowing failing early if zstd is
not supported to avoid masking other possible (future) zfs test failures.
2. Correct test failure in erofs_test in newer Linux distros caused
by a change in mkfs.erofs to limit the maximum file system label
length to 15 characters. Reduce the label used for testing to work
within this limit.
3. Correct test failure in ext234_test in newer Linux distros caused
by a change in e2fsprogs that removed the ability to set "version 0"
ext2 file systems using the "-r" flag. Instead a new flag is added
to support this. Update the test to support the old method in older
distros and the new method in newer distros.
Changes since v6:
- Add '-q' option to grep commands to avoid output to stdout.
- Combine multiple piped greps into one in zfs_zstd_test
Changes since v5:
- Change return code from 77 to 99 in zstd test
Changes since v4:
- Change return code from 99 to 77 in zstd test
Changes since v3:
- Rebase changes against current master branch.
Changes since v2:
- Change return code from 77 to 99 in zstd test
Changes since v1:
- Remove unrelated changes to .gitignore
Andrew Hamilton (3):
tests: Split ZFS ZSTD Test Into New File
tests: Avoid Test Failure in erofs for label length
tests: Support Changed mkfs.ext2 Behavior for -r Flag
.gitignore | 1 +
Makefile.util.def | 6 ++++++
tests/util/grub-fs-tester.in | 13 ++++++++++---
tests/zfs_test.in | 1 -
tests/zfs_zstd_test.in | 30 ++++++++++++++++++++++++++++++
5 files changed, 47 insertions(+), 4 deletions(-)
create mode 100644 tests/zfs_zstd_test.in
--
2.43.0
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel
next reply other threads:[~2025-11-14 23:43 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-14 23:40 Andrew Hamilton [this message]
2025-11-14 23:40 ` [PATCH v7 1/3] tests: Split ZFS ZSTD Test Into New File Andrew Hamilton
2025-11-14 23:40 ` [PATCH v7 2/3] tests: Avoid Test Failure in erofs for label length Andrew Hamilton
2025-11-14 23:40 ` [PATCH v7 3/3] tests: Support Changed mkfs.ext2 Behavior for -r Flag Andrew Hamilton
2025-11-16 1:46 ` [PATCH v7 0/3] Fix Several Test Failures Glenn Washburn
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=20251114234012.32925-1-adhamilt@gmail.com \
--to=adhamilt@gmail.com \
--cc=daniel.kiper@oracle.com \
--cc=development@efficientek.com \
--cc=grub-devel@gnu.org \
--cc=lsandova@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox