Skip to content
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.

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

test: Skip NFS related tests on Fedora 42/43 #2024

Merged
merged 3 commits into from
Feb 8, 2025

Conversation

martinpitt
Copy link
Member

@martinpitt martinpitt commented Feb 7, 2025

Setting up an NFS pool triggers a kernel oops:
https://bugzilla.redhat.com/show_bug.cgi?id=2344326

This cannot be recovered from and breaks all subsequent tests and the VM, so skip the pool tests wholesale.


Avoids this mess: https://artifacts.dev.testing-farm.io/64435bb1-4258-45fc-a3ab-25d967adaee7/

@martinpitt
Copy link
Member Author

at least it ran through the end now, but still 4 failures. What worries me is TestMachinesDisks.testAddDiskSCSI failed first, which doesn't go anywhere near NFS. It's reproducible locally as well, but it doesn't trigger a kernel oops - instead it fails with

virtqemud[866]: failed to load cgroup BPF prog: Operation not permitted

This is independent, so I'll leave that for a separate PR (it'll be a naughty pattern in bots). (not SELinux)

testAddDiskPool is the same.

testAddDiskDirPool is different -- it already fails during early setup in virsh pool-define-as, but passes locally. So might just be a follow-up error.

That leaves testAddDiskNFS , which triggers the kernel oops. So that one has to be skipped as well.

@martinpitt
Copy link
Member Author

This is clearing up now it's down to testAddDisk{Pool,SCSI} which I discussed above. and Fedora 42 has the same kernel, so extending the skips.

Setting up an NFS pool triggers a kernel oops:
https://bugzilla.redhat.com/show_bug.cgi?id=2344326

This cannot be recovered from and breaks all subsequent tests and the
VM, so skip the pool tests wholesale.
@martinpitt martinpitt marked this pull request as ready for review February 7, 2025 12:02
@martinpitt martinpitt changed the title test: Skip NFS related tests on Rawhide test: Skip NFS related tests on Fedora 42/43 Feb 7, 2025
@martinpitt martinpitt requested review from jelly and removed request for jelly February 7, 2025 12:12
@martinpitt
Copy link
Member Author

I reported the BPF related bug to https://bugzilla.redhat.com/show_bug.cgi?id=2344342 and created known issue cockpit-project/bots#7405 .

I'll create a bots naughty thing, block on that.

TF is having a bad time right now anyway, and there's this exciting issue:

+ rpm -e --verbose amazon-ec2-utils
Preparing packages...
amazon-ec2-utils-2.2.0-6.fc42.noarch
+ udevadm trigger '/dev/nvme*'
Failed to open the device '/dev/nvme*': No such device
Shared connection to 3.145.154.28 closed.

I'll fix this along.

Robustify the amazon-ec2-utils workaround: Fedora 42 images don't seem
to have /dev/nvme* any more, so don't fail on

> Failed to open the device '/dev/nvme*': No such device
@martinpitt martinpitt requested a review from jelly February 7, 2025 14:30
@martinpitt
Copy link
Member Author

/packit test

@martinpitt
Copy link
Member Author

Whew, green!

@martinpitt martinpitt merged commit dcdf498 into cockpit-project:main Feb 8, 2025
15 checks passed
@martinpitt martinpitt deleted the nfs-rawhide branch February 8, 2025 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants