action #50432
closedaction #45008: [functional][u] Further improvements on splitting backend code
[sle][functional][u] move specific architectures Utils::Architectures
0%
Description
create Utils::Architectures
move specific architectures to it and let opensusebasetest inheriting from it, to avoid excess of imports
Suggestions:
check like following:
# defined in lib/version_utils.pm
# sub is_s390x {
# return check_var('ARCH', 's390x');
# }
# sub is_x86_64 {
# return check_var('ARCH', 'x86_64');
# }
- Grep for check_var('ARCH' (might use single or double quotes) and move these to an architecture specific utils package
Acceptance criteria
create Utils::Architectures and move specfici architectures to it.
all related tests work as before.
Updated by zluo about 5 years ago
found in version_utils.pm:
sub is_s390x {
return check_var('ARCH', 's390x');
}
sub is_x86_64 {
return check_var('ARCH', 'x86_64');
}
sub is_aarch64 {
return check_var('ARCH', 'aarch64');
}
sub is_ppc64le {
return check_var('ARCH', 'ppc64le');
}
Create Utils::Architectures for these.
Updated by zluo almost 5 years ago
created at first:
# Copyright (C) 2019 SUSE LLC
#
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation; either version 2 of the License, or
# (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License along
# with this program; if not, see <http://www.gnu.org/licenses/>.
package Utils::Architectures;
use strict;
use warnings;
use base qw(Exporter);
use Exporter;
use testapi qw(:DEFAULT);
use constant {
ARCH => [
qw(
is_s390x
is_x86_64
is_aarch64
is_ppc64le
)
]
};
our @EXPORT = @{(+ARCH)};
our %EXPORT_TAGS = (
ARCH => (ARCH),
);
# specific architectures
sub is_s390x {
return check_var('ARCH', 's390x');
}
sub is_x86_64 {
return check_var('ARCH', 'x86_64');
}
sub is_aarch64 {
return check_var('ARCH', 'aarch64');
}
sub is_ppc64le {
return check_var('ARCH', 'ppc64le');
}
1;
Updated by zluo almost 5 years ago
modified: lib/main_common.pm
modified: lib/version_utils.pm
modified: tests/installation/welcome.pm
test runs locally:
http://f40.suse.de/tests/3371 (s390x)
http://f40.suse.de/tests/3372 (aarch64)
http://f40.suse.de/tests/3370 (x86_64)
http://f40.suse.de/tests/3379 (ppc64le)
Updated by zluo almost 5 years ago
modified: tests/console/validate_lvm_encrypt.pm
modified: tests/sles4sap/saptune.pm
Updated by zluo almost 5 years ago
test runs:
http://f40.suse.de/tests/3386 (YaST lvm-full-encrypt@aarch64)
http://f40.suse.de/tests/3387 (sles4sap saptune)
Updated by zluo almost 5 years ago
Updated by zluo almost 5 years ago
need to check:
http://openqa.suse.de/tests/2845045#live (lvm-full-encrypt@aarch64) on osd by using PR
https://openqa.opensuse.org/tests/921396 (TW Gnome) on 3o
https://openqa.opensuse.org/tests/921397 (Leap lvm-full-encrypt) on 3o
Updated by SLindoMansilla almost 5 years ago
- Status changed from Feedback to Workable
PR reviewed. Changes requested.
https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/7357
Updated by SLindoMansilla almost 5 years ago
- Status changed from Feedback to Workable
Please, resolve merge conflicts.
Updated by SLindoMansilla almost 5 years ago
PR merged: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/7357
Waiting for verification in OSD
Updated by zluo almost 5 years ago
- Status changed from Feedback to Resolved
some test runs on osd 3 days ago:
http://openqa.suse.de/tests/2880803
http://openqa.suse.de/tests/2880801
http://openqa.suse.de/tests/2880366
some test runs on 3o:
https://openqa.opensuse.org/tests/931845
https://openqa.opensuse.org/tests/932138
No issue found and all related tests are successful.