action #63766
closed
Handle HDD* vars in generalhw backend
Added by ggardet_arm about 4 years ago.
Updated about 4 years ago.
Description
Currently, vars in /etc/openqa/workers.ini are not expanded, so, for example, you cannot use %HDD_1% in other variables.
My use case is for generalhw backend used with System-Under-Test (SUT), here Raspberry Pi 3 boards.
If you have an openQA MACHINE (e.g. RPi3) matching a single openQA worker, you can define all the required vars in MACHINE.
But to scale, we need an openQA MACHINE which matches multiple openQA workers, with each worker limited to a single SUT (worker 1 => Raspberry Pi 3 board number 1, etc.) and you need to define things in the worker config (main vars are IP related).
So, for now, we cannot use %HDD_1%, FLASHER_IP or SUT_IP variables in /etc/openqa/workers.ini and cannot have something like GENERAL_HW_FLASH_ARGS = /var/lib/openqa/factory/hdd/%HDD_1% %FLASHER_IP%:/tmp/
For the IPs, it could be workarounded with copy/paste as it will not be changed from one run to another, but %HDD_1% will change with each test.
This could be handled in generalhw.pm, but it does not sound very good.
The expansion should probably be run on worker.ini file with openQA.
We could report the variables from workers.ini on registering it. Expanding in the worker I consider a no-go
- Subject changed from vars in /etc/openqa/workers.ini are not expanded to support for variable expansion in /etc/openqa/workers.ini
- Category set to Feature requests
With %HDD_1% expected to come from the test triggering point. Why not handle any dynamic logic in os-autoinst:backend/generalhw.pm ? At this point we have read out all information from openQA as well as the openQA worker and IMHO the command can be put together in os-autoinst scope from all necessary information in vars.json. I would not use the "%VAR%" format from openQA for this though. It's basically the same as we do for other backends, the qemu command line that we put together from all different variables.
this drops all connection to the asset though - you only get a connection if openQA (and not os-autoinst) knows the value of HDD_1
- Category deleted (
Feature requests)
- Assignee set to ggardet_arm
PR has been merged. Can we agree that "variable expansion in /etc/openqa/workers.ini" as phrased here is therefore not the way we want to go?
- Subject changed from support for variable expansion in /etc/openqa/workers.ini to Handle HDD* vars in generalhw backend
- Status changed from New to Resolved
okurz wrote:
PR has been merged. Can we agree that "variable expansion in /etc/openqa/workers.ini" as phrased here is therefore not the way we want to go?
That's right. I changed it to: "Handle HDD* vars in generalhw backend"
Also available in: Atom
PDF