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(a)redhat.com>
---
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(a)redhat.com>