Skip to content

Conversation

@MaisenbacherD
Copy link
Contributor

This introduces blktests which require nvme-cli to the nightly tests.

This introduces blktests which require nvme-cli to the nightly tests.

Signed-off-by: Dennis Maisenbacher <[email protected]>
# https://git.kernel.org/pub/scm/utils/mdadm/mdadm.git/commit/?id=7f960c3bd050e76f8bf0a8a0c8fbdcbaa565fc78
# The Fedora packages ship v4.3.
# We are now compiling and installing mdadm from source with the most
# recent known good commit. The v4.4 release does not compile.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just thinking aloud here: it would be possible to package up this version special version and just install it.
Another solution would be to prebuild the container as we currently do in the container-ci project. I'd just mentioning this because the install step failed very often due to $reason. Since we have the containers pre build things got way more stable.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prepackaging this version would be a great idea. I initially tried using a container here, but I bumped into various issues (also due to the way blktests operate). For this reason I put the container solution aside for now. When I have time I will revisit this, as I think the issues I faced are solvable.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No worries, as I said. This is something we can tackle later. It's good to have this as it is.

@igaw igaw merged commit 3bb41bd into linux-nvme:master Sep 25, 2025
15 checks passed
@igaw
Copy link
Collaborator

igaw commented Sep 25, 2025

Thanks!

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.

2 participants