action #95362
open
[qe-core] make zypper_call use serial terminal regardless of the architecture
Added by mgrifalconi almost 3 years ago.
Updated about 2 months ago.
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)
- Description updated (diff)
- Related to action #95365: [qe-core] move ncurses out of installation tests added
- 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
- Description updated (diff)
- Status changed from New to Workable
- Target version set to QE-Core: Ready
- Assignee set to mgrifalconi
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.
- Related to action #75319: [qe-core][functional] test fails in apache_ssl - Where possible, switch to serial_console added
- Assignee changed from mgrifalconi to szarate
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
- Category changed from Refactor/Code Improvements to Spike/Research
- Target version deleted (
QE-Core: Ready)
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.
Also available in: Atom
PDF