Project

General

Profile

Actions

action #127499

closed

coordination #121876: [epic] Handle openQA review failures in Yam squad - SLE 15 SP5

Fix code for apply_workaround_poo124652

Added by rainerkoenig over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
High
Assignee:
Target version:
Start date:
2023-04-11
Due date:
% Done:

0%

Estimated time:

Description

Motivation

That method in lib/YaST/workarounds.pm tries to overcome the scren refresh issue by opening the style popup with shift-f3 and then closing it by sending esc.

Now we observe situations like this:

openQA test in scenario sle-15-SP5-Online-x86_64-iscsi_server_normal_auth_backstore_fileio@64bit fails in
iscsi_server

The problem is that the screen seems to change in the background, but the shift-f3 is not yet processed. So the code goes on sending esc
before the popup appears, so this leads to the situation that the popup will not be closed because the esc key will get lost.

Actions to mitigate

  • Create a needle with tag style-sheet-selection-popup that matches the popup window from the screenshot above.
  • insert a check_screen('style-sheet-selection-popup', 10): after the sending of shift-f3 so we have a grace period of 10 seconds for the popup to appear.

Acceptance criteria

  • AC1: workaround checks now if style-selection-popup appears
  • AC2: verify that we don't brake anything in maintenance with this change

Related issues 1 (0 open1 closed)

Related to qe-yam - action #126788: Change soft-failure to record_info if still needed in iscsi_serverResolvedcoolgw2023-03-28

Actions
Actions #1

Updated by rainerkoenig over 1 year ago

  • Project changed from openQA Tests to qe-yam
  • Category deleted (Bugs in existing tests)
Actions #2

Updated by rainerkoenig over 1 year ago

  • Status changed from New to In Progress
  • Assignee set to rainerkoenig
Actions #3

Updated by coolgw over 1 year ago

  • Related to action #126788: Change soft-failure to record_info if still needed in iscsi_server added
Actions #4

Updated by JERiveraMoya over 1 year ago

  • Target version set to Current
  • Parent task set to #121876

why we need check_screen, we can just send assert_screen, we shouldn't be waiting 10 seconds, that sounds like an sleep.

Actions #5

Updated by JERiveraMoya over 1 year ago

  • Priority changed from Normal to High
Actions #9

Updated by rainerkoenig over 1 year ago

  • Status changed from In Progress to Resolved
Actions #10

Updated by openqa_review over 1 year ago

  • Status changed from Resolved to Feedback

This is an autogenerated message for openQA integration by the openqa_review script:

This bug is still referenced in a failing openQA test: yast2_gui
https://openqa.suse.de/tests/11050062#step/yast2_lan_restart_bond/1

To prevent further reminder comments one of the following options should be followed:

  1. The test scenario is fixed by applying the bug fix to the tested product or the test is adjusted
  2. The openQA job group is moved to "Released" or "EOL" (End-of-Life)
  3. The bugref in the openQA scenario is removed or replaced, e.g. label:wontfix:boo1234

Expect the next reminder at the earliest in 28 days if nothing changes in this ticket.

Actions #11

Updated by rainerkoenig over 1 year ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF