QA (public) » openQA Project (public) » openQA Tests (public)

It is failing here: https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/lib/utils.pm#L987 when grep tries to highlight some entries based on a pattern that does not match anymore due to changes in name format of SLE-15 repos.
"Pool" and "Updates" seems to be only elements interesting to be match from this list https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/lib/utils.pm#L1113 that are also Enabled Yes regarding the list of repos displayed in the test https://openqa.suse.de/tests/1193249#step/zypper_lr_validate/3

Included within a condition to only be applied to SLE-15 and also only for Pool and Updates, the following modified grep command would highlight the enabled product channels (that in my opinion seems to be the original goal of those lines, correct me if I am wrong):
my $distri = uc(get_var('DISTRI')); // Capital letter, in this case SLE
grep --color \"$distri\[\[:alnum:\]\[:punct:\]\]*-*$version-$product_channel $distri\[\[:alnum:\]\[:punct:\]\[:space:\]\]*-*$version-$product_channel $enabled_repo $uri\"";

Still appears in the most recent build (303.1). We should also really consider to improve the output of this module and what it tries to do to ease the process of debugging without digging in the code too much (like it is right now).

Also appears in toolchain test:
https://openqa.suse.de/tests/1216241#

This is an autogenerated message for openQA integration by the openqa_review script:

This bug is still referenced in a failing openQA test: toolchain_zypper
https://openqa.suse.de/tests/1245488

@JERiveraMoya, could you please point which line numbers should it be for your suggested modifications? (i.e. my $distri ; grep --color )

PR created: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/3968

Can you please try to find a reference first? I.e. the "Last good" in the description should be found. Therefore I also suggest to verify against that "last good", e.g. SLE 12 SP3 GM.

For SLE12SP3 there was no zypper_lr_validate, but zypper_lr called: https://openqa.suse.de/tests/1058478

When was zypper_lr_validate first used for SLE15, I have no idea. From the failed test in the description for Build264.3 there's no previous results recorded. The previous tests were incomplete. There's no reference.

This change didn't work for SLE12: http://f146.suse.de/tests/1756. Can someone also have a look and give any hint about it?

PR updated.

Comments provided in the PR.

jrivera, just in case, I have a backup of the branches in my repo where you could pull them from: https://github.com/SergioAtSUSE/os-autoinst-distri-opensuse/tree/zypper_lr

PR: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/4144