action #59394
open[qe-core][functional] Overwrite empty ISO variable everywhere where not needed, i.e. `+ISO=`, to prevent useless ISO downloading and storage
0%
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
Updated by okurz about 5 years ago
- Copied from action #33946: Iso image of Parent Job is downloaded even if "--skip-deps" property is specified while cloning Child Job added
Updated by okurz about 5 years ago
- Related to action #38807: API call that allows to search for test settings (was: Make sure all used assets are accounted for in tests on osd (#38804)) added
Updated by okurz about 5 years ago
- Status changed from New to In Progress
- Assignee set to okurz
doing it on o3
Updated by okurz about 5 years 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.
Updated by mgriessmeier about 5 years 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
Updated by riafarov almost 5 years 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.
Updated by okurz almost 5 years 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.
Updated by ggardet_arm over 4 years ago
- Related to action #64938: '+ISO=' in test suite breaks a number of tests added
Updated by tjyrinki_suse about 4 years ago
- Subject changed from [functional][u] Overwrite empty ISO variable everywhere where not needed, i.e. `+ISO=`, to prevent useless ISO downloading and storage to [qe-core][functional] Overwrite empty ISO variable everywhere where not needed, i.e. `+ISO=`, to prevent useless ISO downloading and storage
Updated by szarate over 3 years ago
- Status changed from Workable to Blocked
Lot of busy work, setting to blocked until api call in #38807 is provided
Updated by slo-gin over 2 years ago
This ticket was set to Normal priority but was not updated within the SLO period. Please consider picking up this ticket or just set the ticket to the next lower priority.
Updated by slo-gin 9 months ago
This ticket was set to Normal priority but was not updated within the SLO period. Please consider picking up this ticket or just set the ticket to the next lower priority.