[sle][functional][medium] test fails in zypper_lr_validate
Adapt the test to SLE15
openQA test in scenario sle-15-Leanos-DVD-x86_64-extra_tests_in_textmode@64bit fails in
Fails since (at least) Build 264.3 (current job)
Last good: https://openqa.suse.de/tests/1359280 (or more recent)
Always latest result in this scenario: latest
#2 Updated by JERiveraMoya over 4 years ago
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\"";
#8 Updated by nicksinger over 4 years ago
- Subject changed from [sle][functional] test fails in zypper_lr_validate - adapt for sle15 to [sle][functional][medium] test fails in zypper_lr_validate - adapt for sle15
- Status changed from New to In Progress
Looks like that there is just some adjustment in the regex needed. I've adjusted the topic to make this item workable and will try to tackle it.
#10 Updated by Anonymous over 4 years ago
- Priority changed from High to Low
Previously there was a zypper_lr test module, which did basic validation. Since zypper_lr_validate validates the output of certain command, and based on the fact that the test module didn't register as failed but only the command output is empty because of syntax issue, I am lowering the priority here.
#12 Updated by Anonymous over 4 years ago
#18 Updated by Anonymous over 4 years ago
- Subject changed from [sle][functional][medium] test fails in zypper_lr_validate - adapt for sle15 to [sle][functional][medium] test fails in zypper_lr_validate
For SLE15 the grep command greps the repos with update or pool, however for SLS12 this command won't find anything from repo with name like SLES12-SP3-12.3-0.
The approach: we can leave the PR only for SLE15: grep --color \"$distri[[:alnum:][:punct:]]-$version-$product_channel $distri[[:alnum:][:punct:][:space:]]-$version-$product_channel $enabled_repo
And write a new line for SLE12.
Or re-enable zypper_lr.pm for SLE12 and only call zypper_lr_validate for SLE15 and newer products.
#24 Updated by SLindoMansilla over 4 years ago
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