-
Notifications
You must be signed in to change notification settings - Fork 169
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
kola/testiso: enable 4k images tests for s390x #3574
base: main
Are you sure you want to change the base?
Conversation
7101de4
to
e05f837
Compare
e05f837
to
7a15b48
Compare
7a15b48
to
075c52a
Compare
075c52a
to
0dad896
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
any reason we have the |
Probably because of:
|
0dad896
to
d14b97d
Compare
"iso-live-login.4k.s390fw", | ||
"iso-offline-install.s390fw", | ||
// https://github.com/coreos/fedora-coreos-tracker/issues/1434 | ||
// "iso-offline-install.mpath.s390fw", | ||
// https://github.com/coreos/fedora-coreos-tracker/issues/1261 | ||
// "iso-offline-install.4k.s390fw", | ||
"iso-offline-install.4k.s390fw", | ||
"iso-offline-install.mpath.s390fw", | ||
"pxe-online-install.s390fw", | ||
"pxe-online-install.4k.s390fw", | ||
"pxe-offline-install.s390fw", | ||
"miniso-install.s390fw", | ||
"pxe-offline-install.4k.s390fw", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should enable so many 4k tests. The goal here is we get a strategic covering of the possible scenarios without trying to cover them all. i.e. let's add maybe 2 or 3 versus 5. Or maybe we can convert some other tests that are running into 4k ones?
This issue seems to be obsolete, but we still have to wait for a new
coreos-installer.rpm
Depends on: ibm-s390-linux/s390-tools#154