Project

General

Profile

Actions

action #65244

closed

[functional][y] test fails in partitioning_warnings

Added by JRivrain about 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
High
Assignee:
-
Category:
Bugs in existing tests
Target version:
-
Start date:
2020-04-03
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

Observation

A recent change broke that test: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/9899/files
The change could be reverted, but actually here it would make more sense to find out why we are using DESKTOP=testmode in that test suite (some ancient main.pm wisdom ?) which does not seem to make much sense. I'd suggest to find where it's set to texmode and set it to gnome, for example.

openQA test in scenario sle-15-SP2-Online-x86_64-btrfs+warnings@64bit fails in
partitioning_warnings

Test suite description

Maintainer: okurz

partitioning_warnings: https://progress.opensuse.org/issues/16570

Reproducible

Fails since (at least) Build 170.1 (current job)

Expected result

Last good: 168.1 (or more recent)

Further details

Always latest result in this scenario: latest

Actions #1

Updated by JRivrain about 4 years ago

I requested a revert for this, as DESKTOP should not be used for installer. https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/9969

Actions #2

Updated by oorlov about 4 years ago

So, what I briefly investigated, "Expert partitioner" button has "alt-e" shortcut when Registration is skipped (this is probably expected in PR, when filter by FLAVOR=Online was added), and 'alt-x' when registration is performed. It is surprising, but this is how our shortcuts auto-generation works. The second filter should be VIDEOMODE=text (instead of DESKTOP=textmode, as this indicates the mode which will be used on the installed system). So, most likely the solution should look like:

if (check_var('VIDEOMODE', 'text') && check_var('SCC_REGISTER', 'installation')) {
        $testapi::cmd{expertpartitioner} = "alt-x";
}
Actions #3

Updated by JRivrain about 4 years ago

  • Status changed from New to Resolved
Actions

Also available in: Atom PDF