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 that send_key doesn't run on virtio console #5814
Conversation
lib/console_yasttest.pm
Outdated
select_console 'log-console'; | ||
save_screenshot; | ||
|
||
show_tasks_in_blocked_state; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does not look like a good idea and I did not see any test runs conducted by you. This way the test would fail whenever we can not log into the logconsole, exactly what we wanted to fix in the first place. AFAICS this code change is defeating the whole purpose.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could alternatively only run show_tasks_in_blocked_state
unless testapi::is_serial_terminal()
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test for old patch (to be replaced in a minute): http://artemis.suse.de/tests/480
Test for new patch: http://artemis.suse.de/tests/483 |
lib/console_yasttest.pm
Outdated
if ($defer_blocked_task_info) { | ||
show_tasks_in_blocked_state; | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about – personal taste:
my $defer_blocked_task_info = testapi::is_serial_terminal() ? 1 : 0;
show_tasks_in_blocked_state unless $defer_blocked_task_info;
…
show_tasks_in_blocked_state if $defer_blocked_task_info;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
If you apply @okurz's suggestions I will merge it.
Call show_tasks_in_blocked_state after selecting non-virtio console if needed.
Call show_tasks_in_blocked_state after selecting
non-virtio console.
https://progress.opensuse.org/issues/39497