Add-on installation: review zypper_lr testcase
After the installation the pool channels should be added and tested but it seems that the testcase zypper_lr does not do that properly, no needles are visible,
#1 Updated by okurz almost 7 years ago
- Target version changed from 168 to Milestone 3
#2 Updated by michalnowak almost 7 years ago
- Status changed from New to In Progress
So, I guess the task is to make sure that
zypper_lr verifies that
zypper lr produces something sensible apart from verifying the command exits with return code
#3 Updated by okurz almost 7 years ago
I think the idea Marita had referencing the specific example is that the "zypper_lr" test case does not check that the sdk ftp repo is actually present. I fear needling the expected output will be quite error prone and tedious to support all different cases we have. I am thinking of wait_serial checking for what you expect, which might be tricky as we would need to know the repo paths for the corresponding addon. Can you investigate how the output of zypper_lr should look like per addon?
Also attending https://progress.opensuse.org/projects/openqatests/wiki#Definition-of-READY-for-new-tests might help as a checklist to get it covered. Ask marita for details if you are unsure.
#4 Updated by maritawerner almost 7 years ago
you can find a Table of all Channels. But I think that update channels are only visible with real SCC and not with proxy SCC. I have to double check that next week.
#5 Updated by michalnowak almost 7 years ago
- Assignee set to michalnowak
#6 Updated by michalnowak almost 7 years ago
Did not get you messages as I failed to assign this PR to me. But I believe I came up with a similar approach: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/1495. To test all those addons combinations I extended
addons_product_sle.pm: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/1496. Plus quite a few needles: https://gitlab.suse.de/openqa/os-autoinst-needles-sles/merge_requests/160/.
#7 Updated by dgutu almost 7 years ago
- Related to action #11460: Feature 320494: Disable installation source after installation if we register system added
#8 Updated by michalnowak almost 7 years ago
Looking at Acceptance test groups at openqa.s.de the test is being executed and is not failing anywhere, I believe this PR should be closed as resolved.
#9 Updated by okurz almost 7 years ago
- Status changed from In Progress to Resolved
yes, I agree. Feel free to set the status to RESOLVED yourself next time, especially if you backed your decision by hard facts, i.e. as you stated, no failures in recent builds :-)