Skip to content

Commit

Permalink
Add softfailure for bug: 1221812
Browse files Browse the repository at this point in the history
  • Loading branch information
RoyCai7 committed Jul 15, 2024
1 parent d0739f4 commit 025aad7
Show file tree
Hide file tree
Showing 3 changed files with 29 additions and 0 deletions.
23 changes: 23 additions & 0 deletions lib/parallel_guest_migration_base.pm
Original file line number Diff line number Diff line change
Expand Up @@ -697,6 +697,8 @@ sub initialize_test_result {

$self->set_test_run_progress;
my @_guest_migration_test = split(/,/, get_var('GUEST_MIGRATION_TEST', ''));
# Remove 'virsh_live_native_p2p_manual_postcopy' if present and record a soft failure
@_guest_migration_test = $self->filter_migration_tests(migration_tests => \@_guest_migration_test) if (is_sle('=15-sp6'));
my $_full_test_matrix = is_kvm_host ? $parallel_guest_migration_base::guest_migration_matrix{kvm} : $parallel_guest_migration_base::guest_migration_matrix{xen};
@_guest_migration_test = keys(%$_full_test_matrix) if (scalar @_guest_migration_test == 0);
my $_localip = get_required_var('LOCAL_IPADDR');
Expand Down Expand Up @@ -1750,6 +1752,27 @@ sub check_peer_test_run {
return $_peer_test_run_result;
}

=head2 filter_migration_tests
Filters migration tests based on specified criteria.
=cut

sub filter_migration_tests {
my ($self, %args) = @_;
my @migration_tests = @{ $args{migration_tests} };
my @guest_migration_test;

foreach (@migration_tests) {
if ($_ eq 'virsh_live_native_p2p_manual_postcopy') {
record_soft_failure('bsc#1221812 - qemu coredump when virsh migrate postcopy + virsh migrate-postcopy on sle15sp6 kvm host.');
} else {
push @guest_migration_test, $_;
}
}
return @guest_migration_test;
}

=head2 post_fail_hook
Set TEST_RUN_RESULT to FAILED, create junit log and collect logs.
Expand Down
3 changes: 3 additions & 0 deletions tests/virt_autotest/parallel_guest_migration_destination.pm
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ use mmapi;
use virt_autotest::utils qw(is_kvm_host is_xen_host check_host_health check_guest_health is_fv_guest is_pv_guest add_guest_to_hosts);
use virt_utils qw(collect_host_and_guest_logs cleanup_host_and_guest_logs enable_debug_logging);
use virt_autotest::domain_management_utils qw(construct_uri create_guest remove_guest shutdown_guest show_guest check_guest_state);
use version_utils qw(is_sle);

=head2 run_test
Expand Down Expand Up @@ -133,6 +134,8 @@ sub guest_migration_test {

$self->set_test_run_progress;
my @guest_migration_test = split(/,/, get_var('GUEST_MIGRATION_TEST'));
# Remove 'virsh_live_native_p2p_manual_postcopy' if present and record a soft failure
@guest_migration_test = $self->filter_migration_tests(migration_tests => \@guest_migration_test) if (is_sle('=15-sp6'));
my $full_test_matrix = is_kvm_host ? $parallel_guest_migration_base::guest_migration_matrix{kvm} : $parallel_guest_migration_base::guest_migration_matrix{xen};
@guest_migration_test = keys(%$full_test_matrix) if (scalar @guest_migration_test == 0);
my $localip = get_required_var('LOCAL_IPADDR');
Expand Down
3 changes: 3 additions & 0 deletions tests/virt_autotest/parallel_guest_migration_source.pm
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ use virt_autotest::utils qw(is_kvm_host is_xen_host check_host_health check_gues
use virt_utils qw(collect_host_and_guest_logs cleanup_host_and_guest_logs enable_debug_logging);
use utils qw(script_retry);
use virt_autotest::domain_management_utils qw(construct_uri create_guest remove_guest shutdown_guest show_guest check_guest_state);
use version_utils qw(is_sle);

=head2 run_test
Expand Down Expand Up @@ -163,6 +164,8 @@ sub guest_migration_test {

$self->set_test_run_progress;
my @guest_migration_test = split(/,/, get_var('GUEST_MIGRATION_TEST', ''));
# Remove 'virsh_live_native_p2p_manual_postcopy' if present and record a soft failure
@guest_migration_test = $self->filter_migration_tests(migration_tests => \@guest_migration_test) if (is_sle('=15-sp6'));
my $full_test_matrix = is_kvm_host ? $parallel_guest_migration_base::guest_migration_matrix{kvm} : $parallel_guest_migration_base::guest_migration_matrix{xen};
@guest_migration_test = keys(%$full_test_matrix) if (scalar @guest_migration_test == 0);
my $localip = get_required_var('LOCAL_IPADDR');
Expand Down

0 comments on commit 025aad7

Please sign in to comment.