Project

General

Profile

Actions

action #69715

open

improve error feedback from lockapi to not just have "# Test died: mydie at /usr/lib/os-autoinst/lockapi.pm line 41."

Added by okurz over 3 years ago. Updated over 3 years ago.

Status:
Workable
Priority:
Low
Assignee:
-
Category:
Feature requests
Target version:
Start date:
2020-08-07
Due date:
% Done:

0%

Estimated time:

Description

Observation

https://openqa.suse.de/tests/4535890#step/rsync_client/10 shows how a test failed with "# Test died: mydie at /usr/lib/os-autoinst/lockapi.pm line 41." which is not that helpful. https://github.com/os-autoinst/os-autoinst/blob/master/lockapi.pm#L41 says bmwqemu::mydie "$type '$name': lock owner already finished" if $res == 410; but the message does not show up in openQA. The log has:

[2020-08-07T02:54:22.190 UTC] [debug] <<< bmwqemu::mydie(cause_of_death="barrier 'rsync_setup': lock owner already finished")
[2020-08-07T02:54:22.324 UTC] [info] ::: basetest::runtest: # Test died: mydie at /usr/lib/os-autoinst/lockapi.pm line 41.

Acceptance criteria

  • AC1: The cause of death is also shown on the level of openQA job module details, e.g. the thumbnail popup

Suggestions

  • Research why bmwqemu::mydie is used and not just die which should be properly handled already
  • Change implementation to forward the cause_of_death as in case of other failed checks (try first with simple die)

Related issues 1 (0 open1 closed)

Related to openQA Tests - action #80570: [qe-core][qem][sporadic] test fails in smt_client1 auto_review:"mydie.*acquiring barrier 'smt_setup': lock owner already finished":retryRejectedzluo

Actions
Actions #1

Updated by mkittler over 3 years ago

I'm aware that the error handling of the lock API lacks forwarding the message containing what's actually wrong to the user because I've received complaints about that before. E.g. if one makes a simple mistake like using an invalid character for the lock name it is not obvious at all why it doesn't work.

Actions #2

Updated by okurz over 3 years ago

  • Related to action #80570: [qe-core][qem][sporadic] test fails in smt_client1 auto_review:"mydie.*acquiring barrier 'smt_setup': lock owner already finished":retry added
Actions

Also available in: Atom PDF