Project

General

Profile

action #39869

Updated by riafarov over 5 years ago

## Observation 

 openQA test in scenario sle-15-SP1-Installer-DVD-ppc64le-toolchain_zypper@ppc64le fails in 
 [kdump_and_crash](https://openqa.suse.de/tests/1946629/modules/kdump_and_crash/steps/10) 

 We are missing sles-server debug repo because of incorrect openQA variable (should be REPO_SLE15_SP1_MODULE_BASESYSTEM_DEBUG). 

 ``` 
 sub get_repo_url_for_kdump_sle { 
     return join('/', $utils::OPENQA_FTP_URL, get_var('REPO_SLE15_MODULE_BASESYSTEM_DEBUG')) 
       if get_var('REPO_SLE15_MODULE_BASESYSTEM_DEBUG') 
       and is_sle('15+'); 
     return join('/', $utils::OPENQA_FTP_URL, get_var('REPO_SLES_DEBUG')) if get_var('REPO_SLES_DEBUG'); 
 } 
 ``` 

 In the main.pm we have code to inject version              
 ``` 
 my $repo_variable_name 
               = is_module($short_name) ? 
               "REPO_SLE${version}_MODULE_${repo_name}" 
               : "REPO_SLE${version}_PRODUCT_${repo_name}"; 
 ``` 

 In general it doesn't make sense to have version in the variable names. For that we should introduce changes to rsync.pl and then adjust tests code all usages of REPO_SLE_xxx 

 This will simplify test code and will work in future, as we already had to introduce fix for SP1 in main.pm to replace - with _ 

 ## Acceptance criteria 
 1. REPO_ variables do not include version in the name 
 


 ## Reproducible 

 Fails since (at least) Build [19.7](https://openqa.suse.de/tests/1916613) 


 ## Expected result 

 Last good: [SLE15GM](https://openqa.suse.de/tests/1771968#step/kdump_and_crash/10) 


 ## Further details 

 Always latest result in this scenario: [latest](https://openqa.suse.de/tests/latest?flavor=Installer-DVD&distri=sle&test=toolchain_zypper&machine=ppc64le&version=15-SP1&arch=ppc64le) 

Back