action #127349
closedcoordination #123583: [epic] Agama auto-installation
Add code for handling agama.auto boot parameter
Description
Motivation¶
The action [#126857] made a proof of concept for automating the ALP installation by using the dinst.auto
boot parameter, that soon will be renamed to agama.auto
.
To avoid the hassle of providing the file for download on the worker in this PoC the parameter just points to a branch on GitHub, but that is not the final destination. We should create a parameter handler similar to prepare_profile
in autoyast scenarios. That means read a openQA setting that we need to specify, e.g. AGAMA_AUTO
and process the file so that we create a temporary upload on the worker and submit this path as the agama.auto
parameter to the bootloader.
Acceptance criteria¶
- AC1: New openQA setting
AGAMA_AUTO
is defined. - AC2: Library modules are adapted to provide routines to upload the config file and provide a temporary URL.
- AC3: Bootloader modules are able to handle the
agama.auto
parameter similar to handling theautoyast
boot parameter - AC4: A testmodule is created that comes before the bootloader and prepares the
agama.auto
profile for the worker.
Additional information¶
Do not mix AutoYaST logic and libraries with this new development, just mimic what is there.
Updated by JERiveraMoya over 1 year ago
- Description updated (diff)
- Status changed from New to Workable
- Priority changed from Normal to High
- Target version set to Current
- Parent task set to #123583
Updated by JERiveraMoya over 1 year ago
- Related to action #126857: Automate auto-installation of SUSE ALP Bedrock added
Updated by leli over 1 year ago
- Status changed from Workable to In Progress
- Assignee set to leli
Updated by rainerkoenig over 1 year ago
I had a look at your code for url_prepare.pm.
The idea of the AGAMA_AUTO
parameter was not that it should contain JSON or a shell script. The parameter should just contain a filename, e.g. test_data/agama/alp.sh
. This file should be put to be used.
Also, as long as you don't want to touch the bootloader code to make use of AGAMA_AUTO then EXTRABOOTPARAMS
should be set to dinst.auto=\"$url\"
.
When the name transition is made from dinstaller to agama this parameter will be renamed to agama.auto
.
Updated by JERiveraMoya over 1 year ago
Looks like this handling was implemented in #126857 so let's use this ticket to polish those details in that ticket, those PR's comments.
Updated by leli over 1 year ago
We need update for alp auto-installation following comments on PR https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/16927.
PR submitted, https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/16929
Updated by JERiveraMoya over 1 year ago
- Status changed from In Progress to Resolved