Project

General

Profile

action #169159

Updated by mkittler 18 days ago

## Motivation 
 This would be useful so job settings referring to assets (e.g. `QA_HEAD_REPO=https://download.suse.de/ibs/…`) could be written in a more generic way (e.g. `QA_HEAD_REPO=https://%REPO_MIRROR_HOST%/ibs/…`). This way the download server could be replaced by a mirror more easily. Considering ideas like https://gitlab.suse.de/openqa/salt-pillars-openqa/-/merge_requests/924 it would be useful if worker settings are considered as well. See https://suse.slack.com/archives/C02CANHLANP/p1730383116044599 for the related discussion with @szarate. 

 ## Acceptance criteria 
 * **AC1**: Placeholders in job settings are substituted before the asset download taking settings from the worker config into account. 
 * **AC2**: The variable substitution introduced via AC1 is in-line with https://open.qa/docs/#_variable_expansion (and internally uses the same code as much as possible). 

 ## Suggestions 
 * Check what function is used to implement placeholders. Invoke that function in `isotovideo.pm` as well before the asset handling is done.

Back