Project

General

Profile

Actions

action #95362

open

[qe-core] make zypper_call use serial terminal regardless of the architecture

Added by mgrifalconi almost 3 years ago. Updated 2 months ago.

Status:
Workable
Priority:
Normal
Assignee:
Category:
Spike/Research
Target version:
-
Start date:
2021-07-12
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

Observation

Anything that has to do with aarch64, or any architecture for that matter in terms of calling things like zypper_call, can be shifted to serial terminal.

Suggestions

  • Modify zypper_call to always use serial terminal, then switch back to what it was using originally (testapi has current_console to get, the current console :))
  • Add a parameter to support backwards compatibility, to disable such feature in case of need (such as no_console_switch=1), assume that if the parameter is not set and VIRTO_CONSOLE=1 is set, the switch is enabled by default)
  • Advertise change in proper channels via RFC in os-autoinst-distri-opensuse (openqa, qa-sle ML, link in #testing in RC/Slack)

Related issues 3 (1 open2 closed)

Related to openQA Tests - action #95365: [qe-core] move ncurses out of installation testsResolvedzluo2021-10-12

Actions
Related to openQA Tests - action #90008: [qe-core][typing] test fails in ncurses with typing issues - Improve by using serial_terminal and executing tests via bashResolvedmgrifalconi

Actions
Related to openQA Tests - action #75319: [qe-core][functional] test fails in apache_ssl - Where possible, switch to serial_consoleNew

Actions
Actions #1

Updated by mgrifalconi almost 3 years ago

  • Description updated (diff)
Actions #2

Updated by szarate almost 3 years ago

  • Related to action #95365: [qe-core] move ncurses out of installation tests added
Actions #3

Updated by szarate almost 3 years ago

  • Related to action #90008: [qe-core][typing] test fails in ncurses with typing issues - Improve by using serial_terminal and executing tests via bash added
Actions #4

Updated by szarate almost 3 years ago

  • Description updated (diff)
  • Status changed from New to Workable
  • Target version set to QE-Core: Ready
Actions #5

Updated by mgrifalconi almost 3 years ago

  • Assignee set to mgrifalconi
Actions #6

Updated by favogt almost 3 years ago

What's the benefit of this? If it's performance, are there measurements?

What could be useful is that the entire zypper output is visible in the logs that way.

Actions #8

Updated by favogt over 2 years ago

Looks promising!

Actions #9

Updated by apappas over 2 years ago

  • Related to action #75319: [qe-core][functional] test fails in apache_ssl - Where possible, switch to serial_console added
Actions #10

Updated by mgrifalconi about 2 years ago

  • Assignee changed from mgrifalconi to szarate
Actions #11

Updated by szarate about 2 years ago

https://suse.slack.com/archives/C02CSAZLAR4/p1646983603914099

 I'd rather go the other way... and push for the change to be implemented in a transparent way (so that it works for everybody)... but we can take an RFC approach before changing that
 problem with shared libraries, is that if nobody messes up with them, then nobody will also care for their development/improvement

Agree on the "someone should care about libraries" but, wouldn't make sense to have the lib only do its thing, without changing the terminal while doing it? And than start enforcing everywhere the usage of serial, not just for zypper calls?
Problem we have seen with many library changes so far, that many thing break on day1 and need fix, because we have no real staging setup (needles more sensitive, script_run timeout behavior etc.) and even if you announce it, nobody will be able to test it everywhere I fear.
star-struck
+1::skin-tone-5
thisisfine-flames

as it is mainly a peformance change than a fix or an improvvement we can take our time finding a common ground

Actions #12

Updated by szarate about 2 years ago

  • Category changed from Refactor/Code Improvements to Spike/Research
  • Target version deleted (QE-Core: Ready)
Actions #13

Updated by slo-gin 2 months ago

This ticket was set to Normal priority but was not updated within the SLO period. Please consider picking up this ticket or just set the ticket to the next lower priority.

Actions

Also available in: Atom PDF