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

tests: add kdump over NFS #3922

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jbtrystram
Copy link
Contributor

This was previously merged in [1] then reverted in [2] because the nfs
server container was not multi-arch, causing the pipeline to trip on it.

It's also not functionning on systemd256 (so anything f41 and above),
see [3]

This requires #3917 for
the multi-arch container, and
#3921

[1] b10d8dc
[2] af1468c
[3] rhkdump/kdump-utils#52

This way we have good coverage of most-used kdump features.
Some context on the NFS kdump configuration:
coreos/fedora-coreos-tracker#1729

This was previously merged in [1] then reverted in [2] because the nfs
server container was not multi-arch, causing the pipeline to trip on it.

It's also not functionning on systemd256 (so anything f41 and above),
see [3]

This requires coreos#3917 for
the multi-arch container, and
coreos#3921

[1] coreos@b10d8dc
[2] coreos@af1468c
[3] rhkdump/kdump-utils#52
@jbtrystram jbtrystram added the hold-for-external Needs an external dependency to change label Nov 3, 2024
@jbtrystram
Copy link
Contributor Author

/hold
Let's wait for rhkdump/kdump-utils#52 or denylist on f41 and above
Also the quay.io/coreos-asembler/nfs does not exist now, i am hitting coreos/fedora-coreos-pipeline#1053

@jbtrystram jbtrystram changed the title tests/containers: add a NFS server containers tests: add kdump over NFS Nov 3, 2024
@HuijingHei
Copy link
Member

/retest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/hold hold-for-external Needs an external dependency to change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants