action #94534
closed[qem][qe-core] test fails in scc_registration
0%
Description
Observation¶
TL;DR Sometimes it needs Yes, sometimes Trust. We should handle both cases.
The issue:
- on 15/06 it required a
Trust/Cancel
response https://openqa.suse.de/tests/6248670#step/scc_registration/129 - from 16/06 it required a
Yes/No
response, so we changed the code https://openqa.suse.de/tests/6277052#step/scc_registration/128 - today it went back to
Trust/Cancel
response https://openqa.suse.de/tests/6310943#step/scc_registration/269
openQA test in scenario sle-15-SP2-Server-DVD-Updates-x86_64-qam-textmode+sle15@64bit fails in
scc_registration
Test suite description¶
Reproducible¶
Fails since (at least) Build 20210623-1 (current job)
Expected result¶
Last good: 20210622-1 (or more recent)
Further details¶
Always latest result in this scenario: latest
Updated by mgrifalconi almost 3 years ago
Chat with lslezak on IRC #yast
mich_: it might be actually caused by changes on the server, one dialog is "import untrusted key" the other one is "unknown key", I guess the GPG key was not published at the second case
mich_: very likely the nvidia repo was broken/incomplete at that time
mich_: as this is a 3rd party server we cannot guarantee anything so I think it would be better to support both cases in the test
Updated by mgrifalconi almost 3 years ago
New PR to go back to alt-t as temporary fix. https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/12779
We should be able to handle both cases without changing the code all the times.
Simple fix: Send both keys (alt-y and alt-t), maybe the second only if the first do not work as intended
Another way: recognize what key is needed by creating a lot of new needles
Of course other ideas are welcome :)
Updated by okurz almost 3 years ago
Sending both keys is what I had in mind as well. On the other hand if you need "a lot of new needles" you would do it wrong, both cases had been in there in before. Maybe only need to add specific tags to distinguish. The complete amount of needles should not exceed 8: ncurses/GUI*yes/trust/no/cancel
Updated by okurz almost 3 years ago
- Status changed from New to Feedback
- Assignee set to okurz
let me give it a try: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/12781
Updated by okurz almost 3 years ago
- Status changed from Feedback to Resolved
PR merged. Hard to verify both cases as that would depend on the state in external repos which we don't have under our control. However we can at least verify one of two cases, e.g. https://openqa.suse.de/tests/6329671#step/scc_registration/38