action #25412
closedClone_job: support global variables
Description
Doing clone_job.pl --skip-chained-deps --skip-download --from http://openqa.suse.de 1173559 WORKER_CLASS=local
Creates
Created job #4099: sle-15-Leanos-DVD-x86_64-Build260.4-autoyast-supportserver@64bit -> http://localhost/t4099
Created job #4100: sle-15-Leanos-DVD-x86_64-Build260.4-sles12_autoyast_tftp@64bit -> http://localhost/t4100
While sle-15-Leanos-DVD-x86_64-Build260.4-sles12_autoyast_tftp@64bit ended up with WORKER_CLASS=local it's parent job sle-15-Leanos-DVD-x86_64-Build260.4-autoyast-supportserver@64bit did not (It kept the values from the original job).
Updated by coolo over 6 years ago
- Subject changed from Clone job does not copy settings to chained jobs to Clone_job: support global variables
- Category changed from Regressions/Crashes to Feature requests
- Target version set to Ready
This is a feature - as the variables you give there are for the jobs to be cloned. WORKER_CLASS (and possibly other machine related variables) need to be differed. But perhaps you realize the problem if you consider that I use TEST very often for clone_job to give the clone a different name. Having this in the parents too doesn't make sense.
But even things like QEMUCPU are not necessarly also for the dependencies (imagine you're cloning parallel jobs, not chained ones).
So IMO what we need is a switch in clone_job for variables to be passed to dependent jobs.
Updated by oorlov over 6 years ago
Today I've also noticed that behavior while cloning child job with the custom WORKER_CLASS parameter. The parent did not get the same worker class name and did not start. Would be a nice feature to pass the WORKER_CLASS parameter to parent job.
Updated by mitiao over 6 years ago
- Assignee set to mitiao
- Target version changed from Ready to Current Sprint
Updated by mitiao over 6 years ago
so what other global variables like WORKER_CLASS we are using?
Updated by mitiao over 6 years ago
- Status changed from New to In Progress
Updated by mitiao over 6 years ago
- Status changed from In Progress to Resolved
pr merged with minor refactor and unit test added.
Updated by szarate over 6 years ago
- Target version changed from Current Sprint to Done