Skip to content

blktest liburing package addition and error handling improvement#3144

Open
maramsmurthy wants to merge 1 commit intoavocado-framework-tests:masterfrom
maramsmurthy:smaram_blktest_fixes
Open

blktest liburing package addition and error handling improvement#3144
maramsmurthy wants to merge 1 commit intoavocado-framework-tests:masterfrom
maramsmurthy:smaram_blktest_fixes

Conversation

@maramsmurthy
Copy link
Copy Markdown
Contributor

Added liburing Dependency:
Added liburing-devel (SUSE/RHEL) and liburing-dev to pkg dependencies - Ensures nvme-passthru-admin-uring binary is built
Enables nvme/067 test to execute successfully
Supports io_uring-based NVMe admin command testing

Improved Exit Code Handling:
Modified test failure logic -
Before: Failed on ANY non-zero exit code (including when tests were skipped) After: Only fails when actual test failures are detected ([failed] markers) Added informational warning when exit code is 1 due to skipped tests Eliminates false positive failures

@maramsmurthy
Copy link
Copy Markdown
Contributor Author

debug_rhel.log
debug_sles16.log
RHEL and SUSE log files

Added liburing Dependency:
Added liburing-devel (SUSE/RHEL) and liburing-dev to pkg dependencies -
Ensures nvme-passthru-admin-uring binary is built
Enables nvme/067 test to execute successfully
Supports io_uring-based NVMe admin command testing

Improved Exit Code Handling:
Modified test failure logic -
Before: Failed on ANY non-zero exit code (including when tests were skipped)
After: Only fails when actual test failures are detected ([failed] markers)
Added informational warning when exit code is 1 due to skipped tests
Eliminates false positive failures

Signed-off-by: Maram Srimannarayana Murthy <msmurthy@linux.vnet.ibm.com>
@maramsmurthy maramsmurthy force-pushed the smaram_blktest_fixes branch from 760b2cf to 48c261f Compare April 27, 2026 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant