Re: [PATCH v2 0/4] Remove the bootlinux-local role

[Daniel Gomez <da.gomez@kernel.org>]

On 20/05/2025 20.28, Daniel Gomez wrote:
> 
> 
> On 19/05/2025 18.21, cel@kernel.org wrote:
>> From: Chuck Lever <chuck.lever@oracle.com>
>>
>> The bootlinux-local role simply installs the bootlinux dependencies
>> on the Ansible controller when bootlinux_9p is set. There doesn't
>> seem to be a good reason that cannot be done via an extra step in
>> bootlinux.
>>
>> This version needs some testing, which I can get to later today.
> 
> 
> I've also tested this series here:
> 
> https://github.com/linux-kdevops/kdevops/actions/runs/15135610179/ 
> job/42546562244

I've also tested this series manually on my setup and looks good.

Tested-by: Daniel Gomez <da.gomez@samsung.com>

> 
> 
>>
>> Changes since v1:
>> - Address Daniel's review comments and clarify patch descriptions
>>
>> Chuck Lever (4):
>>    bootlinux: Remove the 'linux' tag
>>    bootlinux: Modernize install-deps/main.yml
>>    bootlinux: Switch to import_tasks for install-deps
>>    bootlinux: Fold bootlinux-local into the bootlinux role
>>
>>   playbooks/bootlinux-local.yml                 |  5 -
>>   .../roles/bootlinux-local/defaults/main.yml   |  3 -
>>   .../roles/bootlinux-local/tasks/install-deps  |  1 -
>>   .../roles/bootlinux-local/tasks/main.yml      | 19 ----
>>   .../tasks/install-deps/debian/main.yml        |  2 -
>>   .../bootlinux/tasks/install-deps/main.yml     | 21 +++--
>>   playbooks/roles/bootlinux/tasks/main.yml      | 92 ++++++++++---------
>>   .../bootlinux/tasks/update-grub/debian.yml    |  2 +-
>>   .../bootlinux/tasks/update-grub/install.yml   | 12 +--
>>   .../bootlinux/tasks/update-grub/redhat.yml    |  2 +-
>>   .../bootlinux/tasks/update-grub/suse.yml      |  2 +-
>>   workflows/linux/Makefile                      |  3 -
>>   12 files changed, 72 insertions(+), 92 deletions(-)
>>   delete mode 100644 playbooks/bootlinux-local.yml
>>   delete mode 100644 playbooks/roles/bootlinux-local/defaults/main.yml
>>   delete mode 120000 playbooks/roles/bootlinux-local/tasks/install-deps
>>   delete mode 100644 playbooks/roles/bootlinux-local/tasks/main.yml
>>
>