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

Add check for kernel/kernel-devel version mismatch #13683

Merged
merged 1 commit into from Nov 18, 2021

Conversation

BillAnastasiadis
Copy link
Contributor

@BillAnastasiadis BillAnastasiadis commented Nov 15, 2021

Checks for kernel-default-devel installation and matches it to the kernel-default installation. The appropriate message is displayed in each case, should any of those checks fail.

TEST FAILURE RUNS: (test slightly altered to fail) (15sp2)
diffrent kernel-default version: http://10.161.229.209/tests/332#step/systemtap/374
no kernel-default-devel: http://10.161.229.209/tests/333#step/systemtap/371

NORMAL RUNS:
15GA:
x86_64: https://openqa.suse.de/tests/7694449#
s390: https://openqa.suse.de/tests/7694450#
aarch64: FAILED IN PREVIOUS MODULE (mdadm) https://openqa.suse.de/tests/7694451
15sp1:
x86_64: https://openqa.suse.de/tests/7694373#
s390: https://openqa.suse.de/tests/7694429#
aarch64: https://openqa.suse.de/tests/7694448#
15sp2:
x86_64: https://openqa.suse.de/tests/7694291#
15sp3:
x86_64: https://openqa.suse.de/tests/7694290#
12sp5:
x86_64: https://openqa.suse.de/tests/7694452#
s390: https://openqa.suse.de/tests/7694453#
aarch64: https://openqa.suse.de/tests/7694454#
12sp4:
x86_64: https://openqa.suse.de/tests/7694456
s390: https://openqa.suse.de/tests/7694454#
12sp3: http://10.161.229.209/tests/309
x86_64: https://openqa.suse.de/tests/7694458#
12sp2: http://10.161.229.209/tests/310
x86_64: https://openqa.suse.de/tests/7694459#

Test is disabled on opensuse - there is another ticket to fix and enable it.

Copy link
Contributor

@punkioudi punkioudi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general lgtm, just a small comment in the VRs. It's better to run all the mau-extratests2 job, in order to check that the change you've made in systemtap, doesn't cause any issue to the other tests.

@czerw
Copy link
Contributor

czerw commented Nov 16, 2021

I think we already have test for it https://openqa.suse.de/tests/7657091/modules/module_build/steps/1/src . It verifies that devel files are really for particular kernel by compilation.

Copy link
Contributor

@tjyrinki tjyrinki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this would be a fine addition, given that dzedro ran into the problem in the first place. It's a bit different and self-explaining approach to the test as the kernel test is assuming a compilation would fail with different version while this checks actual packages.

@punkioudi punkioudi merged commit f46b872 into os-autoinst:master Nov 18, 2021
@@ -22,6 +22,17 @@ sub run {
$self->select_serial_terminal;
prepare_for_kdump();
zypper_call("in systemtap systemtap-docs kernel-devel systemtap-server");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't it be better to simply specify the kernel-devel version explicitly here? That's what update_kernel module does for kernel incident tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants