[libvirt-ci PATCH 00/12] Introduce a new global YAML config file for lcitool

Erik Skultety posted 12 patches 3 years, 10 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
guests/README.markdown                    |  18 +-
guests/config.yaml                        |  40 +++++
guests/lcitool                            | 192 +++++++++-------------
guests/playbooks/build/main.yml           |   2 +-
guests/playbooks/update/main.yml          |   6 +-
guests/playbooks/update/tasks/gitlab.yml  |   4 +-
guests/playbooks/update/tasks/kludges.yml |   2 +-
guests/playbooks/update/tasks/users.yml   |  42 ++---
guests/requirements.txt                   |   2 +
9 files changed, 159 insertions(+), 149 deletions(-)
create mode 100644 guests/config.yaml
create mode 100644 guests/requirements.txt
[libvirt-ci PATCH 00/12] Introduce a new global YAML config file for lcitool
Posted by Erik Skultety 3 years, 10 months ago
This series is trying to consolidate the number of config files we currently
recognize under ~/.config/lcitool to a single global YAML config file. Thanks
to this effort we can expose more seetings than we previously could which will
come handy in terms of generating cloudinit images for OpenStack.

Patches 1-4 patches are just a little extra - not heavily related to the series
Patches 1-3 (ACKed) only cosmetical changes (like moving requirements.txt)

Since RFC:
- replaced TOML with YAML which simplified some aspects of the code, thanks
Andrea
- instead of hardcoding the default values, the config within the repo is used
as a template and overriden with user-selected options

Erik Skultety (12):
  requirements: Introduce a requirements.txt file
  lcitool: Decrease the indent when creating a tempdir for initrd
    injection
  lcitool: Prefer tempfile's native wrappers over low level primitives
  lcitool: Use a temporary JSON file to pass extra variables
  config: Introduce a new global config.yaml configuration file
  lcitool: Introduce methods to load and validate the YAML config
  lcitool: Update the config values with internal playbook settings
  lcitool: Drop the get_flavor() method
  lcitool: Drop the get_root_password_file() method
  lcitool: Drop the gitlab-related getter methods
  config: Move the virt-install settings from install.yml to the config
  guests: README: Document the existence and usage of config.toml

 guests/README.markdown                    |  18 +-
 guests/config.yaml                        |  40 +++++
 guests/lcitool                            | 192 +++++++++-------------
 guests/playbooks/build/main.yml           |   2 +-
 guests/playbooks/update/main.yml          |   6 +-
 guests/playbooks/update/tasks/gitlab.yml  |   4 +-
 guests/playbooks/update/tasks/kludges.yml |   2 +-
 guests/playbooks/update/tasks/users.yml   |  42 ++---
 guests/requirements.txt                   |   2 +
 9 files changed, 159 insertions(+), 149 deletions(-)
 create mode 100644 guests/config.yaml
 create mode 100644 guests/requirements.txt

-- 
2.25.3