Project

General

Profile

action #59394

[functional][u] Overwrite empty ISO variable everywhere where not needed, i.e. `+ISO=`, to prevent useless ISO downloading and storage

Added by okurz 9 months ago. Updated 7 months ago.

Status:
Workable
Priority:
Normal
Assignee:
-
Category:
Enhancement to existing tests
Target version:
-
Start date:
2019-11-13
Due date:
% Done:

0%

Estimated time:
Difficulty:
Duration:

Description

Motivation

By default we pass "ISO" to all jobs even when it is not needed, e.g. in downstream jobs booting HDD. This causes useless downloads to the worker for each job as well as on each cloning. See #33946 for details. This can be prevented with a simple +ISO= in all test suites that do not actually need the ISO.

Suggestions

  • Add +ISO= to all test suites booting HDD images and not relying on the ISO
  • Crosscheck test results

Related issues

Related to openQA Tests - action #38807: [functional][u][medium] Make sure all used assets are accounted for in tests on osd (#38804) but not as assets which people need to download on cloning the job as the ISO is not actually usedWorkable2018-07-25

Related to openQA Project - action #64938: '+ISO=' in test suite breaks a number of testsResolved2020-03-27

Copied from openQA Project - action #33946: Iso image of Parent Job is downloaded even if "--skip-deps" property is specified while cloning Child JobResolved2018-03-282019-11-12

History

#1 Updated by okurz 9 months ago

  • Copied from action #33946: Iso image of Parent Job is downloaded even if "--skip-deps" property is specified while cloning Child Job added

#2 Updated by okurz 8 months ago

  • Related to action #38807: [functional][u][medium] Make sure all used assets are accounted for in tests on osd (#38804) but not as assets which people need to download on cloning the job as the ISO is not actually used added

#3 Updated by okurz 8 months ago

  • Status changed from New to In Progress
  • Assignee set to okurz

doing it on o3

#4 Updated by okurz 8 months ago

  • Status changed from In Progress to Workable
  • Assignee deleted (okurz)

ok, only hadded +ISO= to "boot_to_snapshot" as I have tested this scenario. https://openqa.opensuse.org/admin/test_suites shows 337 test suites mentionting "HDD_1=". Some need the ISO, e.g. "external_iso". For the most I am not sure if the ISO is not enabled as repo and used during install so I will leave this for now. I recommend to check the scenarios if the ISO is actually enabled and then just switch all, e.g. using API or DB update.

#5 Updated by mgriessmeier 8 months ago

  • Subject changed from Overwrite empty ISO variable everywhere where not needed, i.e. `+ISO=`, to prevent useless ISO downloading and storage to [functional][u][y] Overwrite empty ISO variable everywhere where not needed, i.e. `+ISO=`, to prevent useless ISO downloading and storage

#6 Updated by riafarov 7 months ago

  • Subject changed from [functional][u][y] Overwrite empty ISO variable everywhere where not needed, i.e. `+ISO=`, to prevent useless ISO downloading and storage to [functional][u] Overwrite empty ISO variable everywhere where not needed, i.e. `+ISO=`, to prevent useless ISO downloading and storage

I'm not sure if that's the right solution. If iso gets removed, we won't be able to regenerate the image, so having setting will help us. On the worker side, copying definitely can be improved by not doing so if we boot into the image and iso is not used as a repo (like in the case of repoless installer). So we are not going to change test suites in our job group in order to have chance to re-generate images.

#7 Updated by okurz 7 months ago

riafarov wrote:

I'm not sure if that's the right solution. If iso gets removed, we won't be able to regenerate the image, so having setting will help us.

There should still be the ISO variable for all jobs that rely on the ISO, e.g. create_hdd_…. I don't think having the ISO referenced in more tests ensures that the image would stay around longer effectively.

#8 Updated by ggardet_arm 4 months ago

  • Related to action #64938: '+ISO=' in test suite breaks a number of tests added

Also available in: Atom PDF