Re: [jenkins-ci PATCH 4/3] guests: Remove ad-hoc package installation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Mar 21, 2018 at 01:19:57PM +0100, Andrea Bolognani wrote:
> There is a small number of packages that we install as part of
> the 'base' task with an ad-hoc call to the package module.
> 
> Since we have generic facilities for installing packages and a
> generic 'base' pseudo-project that we use for all packages that
> are not related to any specific project, we can fold everything
> into it and have a single source of truth.
> 
> The change requires us to delay user creation, though, because
> as part of that we edit the sudoers file, which doesn't exist
> until the sudo package has been installed.
> 
> Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> ---
>  guests/site.yml               |  7 +++++--
>  guests/tasks/base.yml         | 11 -----------
>  guests/vars/mappings.yml      | 15 +++++++++++++++
>  guests/vars/projects/base.yml |  5 +++++
>  4 files changed, 25 insertions(+), 13 deletions(-)
> 
> diff --git a/guests/site.yml b/guests/site.yml
> index 4207d4e..d057116 100644
> --- a/guests/site.yml
> +++ b/guests/site.yml
> @@ -14,16 +14,19 @@
>  
>    tasks:
>  
> -    # Prepare the base environment
> +    # Prepare environment. None of the actions performed here might
> +    # depend on packages being installed
>      - include: tasks/base.yml
>      - include: tasks/compat.yml
> -    - include: tasks/user.yml
>  
>      # Install base packages
>      - include: tasks/packages.yml
>        vars:
>          project: base
>  
> +    # Create users. This needs to happen after installing base packages
> +    - include: tasks/user.yml
> +

This would be solved by moving the package installation into the base
task.

Reviewed-by: Pavel Hrdina <phrdina@xxxxxxxxxx>

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux