action #36397
closed[sle][functional][u] zypper_info failed in SLED12SP4
0%
Description
Observation¶
openQA test in scenario sle-12-SP4-Desktop-DVD-x86_64-allpatterns@64bit fails in
zypper_info
Only the source repo of SLES12SP4 was defined in this case
line 54: elsif (sle_version_at_least('12-SP4') and get_var('REPO_SLES_SOURCE')) {
For SLED it was defined as
line 68: $cmd = "ar --type plaindir cd:///?devices=/dev/sr1 repo-source";
which caused the failure.
Updated by okurz over 6 years ago
This is an autogenerated message for openQA integration by the openqa_review script:
This bug is still referenced in a failing openQA test: cryptlvm
https://openqa.suse.de/tests/1744303
Updated by okurz over 6 years ago
- Subject changed from [sle][functional] zypper_info failed in SLED12SP4 to [sle][functional][u] zypper_info failed in SLED12SP4
- Target version set to Milestone 19
Updated by okurz over 6 years ago
- Target version changed from Milestone 19 to Milestone 19
Updated by zcjia over 6 years ago
The test "zypper_info" is successful for SLES 12 SP4 but failed for SLED 12 SP4, because there are no online source repo (defined by 'REPO_SLES_SOURCE') for SLED.
There is ftp://openqa.suse.de/SLE-12-SP4-SERVER-POOL-x86_64-Build0263-Media2/ available, but there are no SLED 12 SP4 source repo such as ftp://openqa.suse.de/SLE-12-SP4-DESKTOP-POOL-x86_64-Build0236-Media2/ . Is that intentional or an oversight?
Updated by okurz over 6 years ago
I guess "zypper_info" was never working for SLED since it learned to handle source repos about 1 year ago. I recommend to just skip the source package test part for SLED. No need to test this for SLED as well IMHO
Updated by yfjiang over 6 years ago
- Assignee set to zcjia
It seems only affecting SLED, and removal the case in Desktop sounds reasonable. Assign to Zhaocong for a finishing :-) Thanks!
Updated by okurz over 6 years ago
I am interested, can you share how it was fixed? I assume a pull request but I do not remember from my head. Could you provide the URL to the PR in this case, please?
Updated by zcjia over 6 years ago
The PR was merged by rwx788: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/5291