What | Removed | Added |
---|---|---|
CC | xlai@suse.com | |
Flags | needinfo?(xlai@suse.com) |
Nothing I changed, had been introduced in tests originally in 1630d35cb for some PXE based tests. Are you arguing that xvideo does not exist or is wrong? I took the opportunity to "fix" that and created https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/7703 for the code change with the patch ``` --- a/lib/bootloader_setup.pm +++ b/lib/bootloader_setup.pm @@ -385,11 +385,9 @@ sub bootmenu_default_params { # gfxpayload variable replaced vga option in grub2 if (!is_jeos && !is_caasp && (check_var('ARCH', 'i586') || check_var('ARCH', 'x86_64'))) { push @params, "vga=791"; - if (check_var("INSTALL_TO_OTHERS", 1) || !$args{pxe}) { - push @params, "video=1024x768-16"; - } else { - push @params, "xvideo=1024x768"; - } + my $video = 'video=1024x768'; + $video .= '-16' if check_var('QEMUVGA', 'cirrus'); + push @params, $video; } } ``` so it should always be `video=1024x768` unless on "cirrus" where we add the "-16". @xlai as you initially introduced the "xvideo=" parameter which apparently does not even exist for Linux could you please crosscheck the above change and/or the PR?