os-autoinst-distri-opensuse icon indicating copy to clipboard operation
os-autoinst-distri-opensuse copied to clipboard

Apply workaround to force screen refresh on YaST QT window

Open rakoenig opened this issue 2 years ago • 1 comments

  • Related ticket: https://progress.opensuse.org/issues/117949

  • All workarounds for screen refresh problems that used alt-f10 are now replaced with the shift-f3 workaround.

  • The workaround code is outsourced into a library module and called with the needle(s) that we are looking for. The workaround is only applied when necessary (needle would not match) and then accompanied by a soft failure. So soft failures are only recorded when the screen refresh problem occurs.

  • Version matiching is now is_sle(''>=15-SP4') because problem seems to occur on SLE 15-SP5 too.

  • Verification run: https://openqa.suse.de/tests/9954571 (remaining errors are related to firewall problems)

rakoenig avatar Nov 10 '22 10:11 rakoenig

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • [ ] Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files

github-actions[bot] avatar Nov 10 '22 10:11 github-actions[bot]

This seems to have resulted in https://openqa.opensuse.org/tests/2919731#step/yast2_security/28

Looking at the investigation jobs on that test, we see:

  • New snapshot, New test => fail
  • New snapshot, Old testtest => success (unlikely to be a product issue)
  • Old snapshot, New test => fail
  • Old snapshot, Old test => successful

(nevertheless, I also raised a product bug, as there seems to be yast crash which is not expected - but that seems to exist in 15.4 already too)

DimStar77 avatar Dec 01 '22 14:12 DimStar77