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

Fix workaround bsc#1161421 #14887

Merged
merged 1 commit into from
May 17, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion lib/kdump_utils.pm
Original file line number Diff line number Diff line change
Expand Up @@ -141,8 +141,12 @@ sub handle_warning_install_os_prober {

# use yast2 kdump to enable the kdump service
sub activate_kdump {
my (%args) = @_;
# increase kdump memory when bsc#1161421 applies
my $increase_kdump_memory = $args{increase_kdump_memory} // 1;
# restart info will appear only when change has been done
my $expect_restart_info = 0;

# get kdump memory size bsc#1161421
my $memory_total = script_output('kdumptool calibrate | awk \'/Total:/ {print $2}\'');
my $memory_kdump = $memory_total >= 2048 ? 1024 : 320;
Expand All @@ -167,7 +171,7 @@ sub activate_kdump {
}
# ppcl64e and aarch64 needs increased kdump memory bsc#1161421
if (is_ppc64le || is_aarch64) {
if (is_sle('<15-sp4')) {
if ($increase_kdump_memory) {
send_key('alt-y');
type_string $memory_kdump;
wait_screen_change(sub { send_key 'ret' }, 10) for (1 .. 2);
Expand Down
2 changes: 1 addition & 1 deletion tests/console/yast2_kdump.pm
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ sub run {
zypper_call('in yast2-kdump');

# Kdump configuration with YaST module
kdump_utils::activate_kdump;
kdump_utils::activate_kdump(increase_kdump_memory => 0);

# check service (without restarting)
systemctl('is-enabled kdump');
Expand Down