Skip to content

CI: add nightly blktests #2915

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
igaw merged 1 commit into from
Sep 25, 2025
Merged

CI: add nightly blktests #2915

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
51 changes: 50 additions & 1 deletion .github/workflows/run-nightly-tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,56 @@ jobs:
repository: "linux-blktests/blktests-ci"
#We don't have to build the kernel here, as we have a cron job running on
#the k8s cluster that builds the linus-master target nightly.
- name: Run in VM
- name: Run blktests in VM
uses: ./.github/actions/kubevirt-action
with:
kernel_version: linus-master
vm_artifact_upload_dir: blktests/results/
run_cmds: |
set -e
set -x
uname -a

sudo dnf install -y gcc clang make util-linux llvm gawk fio udev \
kmod coreutils g++ gzip e2fsprogs xfsprogs f2fs-tools btrfs-progs \
device-mapper-multipath blktrace kernel-headers liburing \
liburing-devel nbd device-mapper ktls-utils dosfstools \
bc libnl3-cli cryptsetup sg3_utils pciutils unzip jq git wget \
meson json-c-devel openssl-devel keyutils-libs-devel dbus-devel \
swig xz-devel libcurl-devel curl libarchive libarchive-devel json-c

# mdadm has a buffer overflow in v4.3 and was fixed with this commit:
# 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

@igaw igaw Sep 25, 2025

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

@MaisenbacherD MaisenbacherD Sep 25, 2025

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

@igaw igaw Sep 25, 2025

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.


git clone https://git.kernel.org/pub/scm/utils/mdadm/mdadm.git
cd mdadm
git checkout d764c4829947923142a83251296d04edaee7d2f7
make -j$(nproc)
sudo make install
cd -

git clone https://github.com/${{ github.repository }} nvme-cli
cd nvme-cli
scripts/build.sh -b release -c gcc
sudo meson install -C .build-ci
sudo ldconfig /usr/local/lib64

cd -
git clone https://github.com/linux-blktests/blktests.git blktests

cd blktests
make

# Run blktests nvme and md group
cat > config << EF
TEST_DEVS=(${BDEV0})
NVMET_TRTYPES="loop rdma tcp"
EF
sudo ./check nvme md
- name: Run nvme-cli tests in VM
uses: ./.github/actions/kubevirt-action
with:
kernel_version: linus-master
Expand Down