action #104499
closeds390 tests fail with backend died: unable to extract assets: Too many arguments for subroutine 'backend::baseclass::do_extract_assets
Added by livdywan almost 3 years ago. Updated almost 2 years ago.
Description
Observation¶
https://openqa.suse.de/tests/7917723 started failing. This looks to be a regression.
Suggestions¶
Updated by livdywan almost 3 years ago
- Status changed from Workable to In Progress
cdywan wrote:
https://github.com/os-autoinst/os-autoinst/pull/1892 merged, next steps fixing the signature and looking into the gap in our test coverage based on the code path exercised by mru-install-minimal-with-addons@s390x-kvm-sle12
Updated by livdywan almost 3 years ago
cdywan wrote:
cdywan wrote:
https://github.com/os-autoinst/os-autoinst/pull/1892 merged, next steps fixing the signature and looking into the gap in our test coverage based on the code path exercised by
mru-install-minimal-with-addons@s390x-kvm-sle12
Quite simply svirt doesn't implement do_extract_assets: https://github.com/os-autoinst/os-autoinst/pull/1895 and we don't require all backends to either - maybe that simply needs to change before we can drop the unless QEMU?
Updated by tinita almost 3 years ago
Here's a PR to add coverage for the 'unimplemented' methods: https://github.com/os-autoinst/os-autoinst/pull/1894 (merged)
Updated by livdywan almost 3 years ago
cdywan wrote:
Quite simply svirt doesn't implement
do_extract_assets: https://github.com/os-autoinst/os-autoinst/pull/1895 and we don't require all backends to either - maybe that simply needs to change before we can drop theunless QEMU?
Alternatively we could return based on having assets or not: https://github.com/os-autoinst/os-autoinst/pull/1897
Updated by openqa_review almost 3 years ago
- Due date set to 2022-01-13
Setting due date based on mean cycle time of SUSE QE Tools
Updated by livdywan almost 3 years ago
cdywan wrote:
Alternatively we could return based on having assets or not: https://github.com/os-autoinst/os-autoinst/pull/1897
Discussed it briefly with Tina. With the fix for the 'unimplemented' methods we actually have a pretty clear error message visible in the reason, and Tina will check with Marcelo if it's clear enough.
So the remaining point would be to 1) drop the unless QEMU 2) add a unit test hitting the "not implemented" error if PUBLISH_HDD_1 was set but do_extract_assets isn't implemented.
Updated by tinita almost 3 years ago
I had a closer look at the test code, and it's clear now why the test had set PUBLISH_HDD_1.
The svirt_upload_assets implements extract_assets itself, and it does this using the same special variable name as os-autoinst:
https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/tests/shutdown/svirt_upload_assets.pm
This only worked because PUBLISH_HDD_ was simply ignored for anthing but qemu so far.
The code above should be moved to backend::svirt, and after that we can hopefully remove the check for qemu.
Updated by tinita almost 3 years ago
- Copied to action #104520: Move svirt extract_asset code from os-autoinst-distri-opensuse to os-autoinst/backend/svirt.pm size:M auto_review:"unable to extract assets: Can't call method.+name.+on an undefined value":retry added
Updated by livdywan almost 3 years ago
- Status changed from In Progress to Feedback
I'm thinking we can consider this done, since tests are passing again and #104520 covers the backend changes needed to solve the problems with os-autoinst/os-autoinst/pull/1741
Updated by livdywan almost 3 years ago
- Copied to action #104619: openqa_from_containers needs adapted post_fail_hook (was: tests fails due to openSUSE mirror problems with can't locate Mojo/Base.pm in INC in script/client) added
Updated by livdywan almost 3 years ago
- Copied to deleted (action #104619: openqa_from_containers needs adapted post_fail_hook (was: tests fails due to openSUSE mirror problems with can't locate Mojo/Base.pm in INC in script/client))
Updated by livdywan almost 3 years ago
- Status changed from Feedback to Resolved
Updated by pvorel almost 2 years ago
FYI fix https://github.com/os-autoinst/os-autoinst/pull/2282 for code which is IMHO from this ticket.