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

Ensure match reboot for autoyast test on yast X86_64 #20872

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hjluo
Copy link
Contributor

@hjluo hjluo commented Dec 26, 2024

Ensure match reboot for autoyast test on yast X86_64

@hjluo hjluo requested a review from lemon-suse December 26, 2024 07:45
@hjluo hjluo added the qe-yam label Dec 26, 2024
@hjluo hjluo force-pushed the ay_reboot branch 2 times, most recently from 0b84403 to 436a9f3 Compare December 26, 2024 08:03
@hjluo hjluo changed the title Ensure match reboot for autoyast test on X86_64 Ensure match reboot for autoyast test on yast X86_64 Dec 27, 2024
@@ -129,7 +129,7 @@ sub run {
# Autoyast reboot automatically without confirmation, usually assert 'bios-boot' that is not existing on zVM
# So push a needle to check upcoming reboot on zVM that is a way to indicate the stage done
# For bsc#1231522, need match reboot popup on powerVM to reboot after installation.
push @needles, 'autoyast-stage1-reboot-upcoming' if is_s390x || (is_pvm && !is_upgrade && is_sle('<15-SP7')) || (check_var('VIRSH_VMM_FAMILY', 'xen') && check_var('VIRSH_VMM_TYPE', 'hvm'));
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add comments for the reason to exclude of hvm.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added comment there. please take a look again. thanks.

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

Successfully merging this pull request may close these issues.

2 participants