Project

General

Profile

Actions

action #25412

closed

Clone_job: support global variables

Added by zluo about 7 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2017-09-19
Due date:
% Done:

0%

Estimated time:

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).

Actions #1

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.

Actions #2

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.

Actions #3

Updated by mitiao over 6 years ago

  • Assignee set to mitiao
  • Target version changed from Ready to Current Sprint
Actions #4

Updated by mitiao over 6 years ago

so what other global variables like WORKER_CLASS we are using?

Actions #5

Updated by mitiao over 6 years ago

  • Status changed from New to In Progress
Actions #6

Updated by mitiao over 6 years ago

  • Status changed from In Progress to Resolved

pr merged with minor refactor and unit test added.

Actions #7

Updated by szarate over 6 years ago

  • Target version changed from Current Sprint to Done
Actions

Also available in: Atom PDF