Project

General

Profile

Actions

action #20464

closed

[scheduling] Job created with "jobs post" not expand specified MACHINE variable

Added by asmorodskyi almost 7 years ago. Updated almost 5 years ago.

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

100%

Estimated time:

Description

I am creating job with "jobs post" and specifying this set of variables :
{ "HDDSIZEGB=20",
"MACHINE=aarch64",
"VERSION=12-SP2",
"FLAVOR=Server-HPC",
"DISTRI=sle",
"WORKER_CLASS=qemu_aarch64",
"DESKTOP=textmode",
"ARCH=aarch64",
"PACKAGE_REPO=http://download.suse.de/ibs/Devel:/HPC:/SLE12SP2/standard",
"PACKAGETOINSTALL=openqa-ci-test-fail-openqa-tests",
"_GROUP=HPC StagingNext",
"HDD_1=sle-12-SP2-Server-HPC-aarch64-0290-with-hpc.qcow2",
"TEST=single_package_openqa-ci-test-fail-openqa-tests",
"HPC=1",
"BUILD=01"}

Created jobs don't get all variables which assigned to MACHINE='aarch64' in openQA settings.


Related issues 1 (0 open1 closed)

Has duplicate openQA Project - action #51479: Some jobs don't get all the %variables% properly expandedRejectedXiaojing_liu2019-05-15

Actions
Actions #1

Updated by szarate almost 7 years ago

WHile talking to @asmorodskyi about this issue in particular, I was thinking that maybe if %MACHINE is found, it could be expanded by the webUI like we do already with other variables...

Actions #2

Updated by coolo almost 7 years ago

both the expansion and the MACHINE is a feature of the job templates - which is only used by isos post. clone job also posts jobs with MACHINE because the other instance has it - and I don't think the MACHINE on your instance should have any relevance for clone_job.

So I would go with _MACHINE, _TEST_SUITE and _PRODUCT - along with _GROUP

Actions #3

Updated by asmorodskyi almost 7 years ago

usage of _MACHINE variable produce same behavior . Or requirement is in obligatory usage of all 3 ( _MACHINE / _TEST_SUITE and _PRODUCT ) ?

Actions #4

Updated by coolo almost 7 years ago

This is not supported yet - but it sounds useful to support, so we duplicate less infos

Actions #5

Updated by coolo over 6 years ago

  • Subject changed from [tools] Job created with "jobs post" not expand specified MACHINE variable to Job created with "jobs post" not expand specified MACHINE variable
  • Category set to 122
  • Target version set to Ready
Actions #6

Updated by coolo about 5 years ago

  • Assignee set to Xiaojing_liu
  • Target version changed from Ready to Current Sprint
  • Difficulty set to medium
Actions #7

Updated by Xiaojing_liu about 5 years ago

  • Status changed from New to In Progress
Actions #8

Updated by Xiaojing_liu almost 5 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100
Actions #9

Updated by mkittler almost 5 years ago

  • Status changed from Resolved to In Progress

The implementation needs to be revised, see https://github.com/os-autoinst/openQA/pull/2073.

Likely it makes most sense to cherry-pick your first commit https://github.com/os-autoinst/openQA/pull/2057/commits/9091529a8d0e8c6b62ff38a503f1a7502a2ec62c and improve it as needed:

  • The code for the settings expansion should be correct. That is the most important point and it can be verified with the testcases you created today. As @kraih mentioned it makes also sense to add tests with variables used in production.
  • It should be in its own module because the utils have already grown too big. But I still wouldn't use object orientation here. It is simply an overkill in my opinion. Without object orientation the module can still hide the internal helper.
  • Prevent unnecessary copies. (The previous implementation attempt created an unnecessary copy in the constructor.)
Actions #10

Updated by szarate almost 5 years ago

  • Related to action #51479: Some jobs don't get all the %variables% properly expanded added
Actions #11

Updated by szarate almost 5 years ago

  • Related to deleted (action #51479: Some jobs don't get all the %variables% properly expanded)
Actions #12

Updated by szarate almost 5 years ago

  • Has duplicate action #51479: Some jobs don't get all the %variables% properly expanded added
Actions #14

Updated by okurz almost 5 years ago

  • Subject changed from Job created with "jobs post" not expand specified MACHINE variable to [scheduling] Job created with "jobs post" not expand specified MACHINE variable
  • Category changed from 122 to Feature requests
Actions #15

Updated by Xiaojing_liu almost 5 years ago

  • Status changed from In Progress to Resolved
Actions

Also available in: Atom PDF