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

Try to reset console to switch to correct tty #14067

Merged
merged 1 commit into from Jan 21, 2022

Conversation

tinawang123
Copy link
Contributor

@@ -47,6 +47,7 @@ sub run {

record_info('Version', 'VERSION=' . get_var('VERSION'));
reconnect_mgmt_console if is_pvm;
reset_consoles_tty;
Copy link
Contributor

Choose a reason for hiding this comment

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

if is_pvm then
reconnect_mgmt_console
else
reset_consoles_tty

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I checked powerVM, Don't need this condition, it still can work.

@lemon-suse
Copy link
Contributor

Please write the PR's background, introduce the history in the description, it will help people to understand why need this PR.

Copy link
Contributor

@chcao chcao left a comment

Choose a reason for hiding this comment

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

LGTM

@coolgw coolgw merged commit 3886d61 into os-autoinst:master Jan 21, 2022
@tinawang123 tinawang123 deleted the system branch January 25, 2022 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants