openstack/ansible-role-chrony
Code Issues Proposed changes

Add chrony_config_file_location default to avoid uncontrolled failures

Browse Source 
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
This commit is contained in:
Francesco Pantano 2023-05-08 12:34:52 +02:00
parent 62654c88fa
commit 0111661102
No known key found for this signature in database
GPG Key ID: 0458D4D1F41BD75C
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
defaults/main.yml
View File

@ -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

2
tox.ini
View File

@ -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