From 0111661102ebb211ce8e1a08df57fb51cb4671fb Mon Sep 17 00:00:00 2001 From: Francesco Pantano Date: Mon, 8 May 2023 12:34:52 +0200 Subject: [PATCH] Add chrony_config_file_location default to avoid uncontrolled failures This patch adds a missing config that let any playbook fail if no vars are passed. Even though it's always good overriding variables, we should provide defaults for config files like any other first citizen parameter. Change-Id: I953296e22a6d8310aaf401fd9b13a71c35dbaf6a --- defaults/main.yml | 1 + tox.ini | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/defaults/main.yml b/defaults/main.yml index 6ccdca2..a65717e 100644 --- a/defaults/main.yml +++ b/defaults/main.yml @@ -19,3 +19,4 @@ chrony_manage_service: True chrony_manage_package: True chrony_service_state: started chrony_extra_options: [] +chrony_config_file_location: /etc/chrony.conf diff --git a/tox.ini b/tox.ini index b21c0e9..dfd8ec6 100644 --- a/tox.ini +++ b/tox.ini @@ -8,7 +8,7 @@ usedevelop = True install_command = pip install -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master} {opts} {packages} setenv = VIRTUAL_ENV={envdir} deps = -r{toxinidir}/test-requirements.txt -whitelist_externals = bash +allowlist_externals = bash [testenv:bindep] # Do not install any requirements. We want this to be fast and work even if