search

coreos / coreos-assembler

Tooling container to assemble CoreOS-like systems
https://coreos.github.io/coreos-assembler/
Apache License 2.0
347 stars 168 forks source link

Revert "tests/ingition/kdump: add a remote NFS kdump test" #3914

Closed dustymabe closed 1 month ago

dustymabe commented 1 month ago

This reverts commit b10d8dcfe730d2e7c8a5366bfcddc7f3081be203.

The test passes on F40 but not on F41+ [1] and also it is failing on RHCOS so let's just yank it for now and re-apply when it's confirmed to be passing everywhere.

[1] https://github.com/coreos/fedora-coreos-tracker/issues/1820

c4rt0 commented 1 month ago

Thanks for this @dustymabe.

jbtrystram commented 1 month ago

😭

openshift-ci[bot] commented 1 month ago

@dustymabe: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/rhcos 696dbaab4ef002a5a8ec0a2b19196bc97e760d53 link true /test rhcos

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
dustymabe commented 1 month ago

hmm the rhcos.upgrade.from-ocp-rhcos fail in prow/rhcos

but really removing this test couldn't cause that.. merging over red