| Message ID | 20250113-jag-bringup_fixes-v1-0-fb28030b1f26@kernel.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | kdevops: Various fixes | expand |
On Mon, Jan 13, 2025 at 12:52:58PM +0100, Joel Granados wrote: > These are fixes that I used when doing a 'make bringup' on a debian > 13 without libvirt. They seem unrelated because I changed things that > prevented me from moving forward. Get back to me if I need to separate > these in some way. > > Summary of changes (more information in commit messages): > * Adjust the heuristic on becomming sudo > * Prefer -K ansible argument for getting the sudo password > * Use full paths when using tasks from other roles > * Call libvirt configuration tasks when doing the bringup > > Signed-off-by: Joel Granados <joel.granados@kernel.org> For the rest of the patches I'm on vacation next week so I'll yield to Daniel and other's feedback. Thanks! LUis
These are fixes that I used when doing a 'make bringup' on a debian 13 without libvirt. They seem unrelated because I changed things that prevented me from moving forward. Get back to me if I need to separate these in some way. Summary of changes (more information in commit messages): * Adjust the heuristic on becomming sudo * Prefer -K ansible argument for getting the sudo password * Use full paths when using tasks from other roles * Call libvirt configuration tasks when doing the bringup Signed-off-by: Joel Granados <joel.granados@kernel.org> --- Joel Granados (12): Kconfig: Add error path in conf_value_to_yaml scripts: Adjust heuristic to see if current user can sudo ansible: libvirt_user: Install additional pkgs for debian Makefile: Add virsh and libvirtd to minimum dependencies ansible: Add build-deps tag ansible: Use full paths when importing tasks ansible: Run sudo by using the '-K' ansible arg ansible: Install libvirt deps and enable user ansible: Reduce long ansible messages for readability bringup: Use full path when creating custom_source file scripts: Return the full default path of the pool path ansible: Set running_user explicitly to the USER var of the script Makefile.min_deps | 2 ++ playbooks/roles/bringup_guestfs/tasks/main.yml | 32 ++++++++++++++++------ .../libvirt_user/tasks/enable-user/debian/main.yml | 7 +++-- .../roles/libvirt_user/tasks/enable-user/main.yml | 12 +++++--- .../tasks/install-deps/debian/main.yml | 2 ++ .../roles/libvirt_user/tasks/install-deps/main.yml | 15 +++++++--- scripts/bringup_guestfs.sh | 3 +- scripts/get_libvirsh_pool_path.sh | 2 +- scripts/guestfs.Makefile | 5 ++-- scripts/kconfig/confdata.c | 16 +++++------ scripts/libvirt_pool.sh | 25 ++++++++++++++--- 11 files changed, 86 insertions(+), 35 deletions(-) --- base-commit: b9e7570e72dac2df62d2ba80b01e3ff534316517 change-id: 20241217-jag-bringup_fixes-24ef1bc4211a Best regards,