Skip to content
This repository has been archived by the owner on Feb 10, 2021. It is now read-only.

Update Ansible + Deps for security #227

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update Ansible + Deps for security #227

wants to merge 2 commits into from

Conversation

cevich
Copy link
Owner

@cevich cevich commented Oct 29, 2018

Ansible 2.7 does not appear to include breaking changes for older use
cases. However, many of Ansible's dependencies have had security-fixes, and
so deserve updating.

Signed-off-by: Chris Evich [email protected]

Ansible 2.7 does not appear to include breaking changes for older use
cases.  However, many of Ansible's dependencies have had security-fixes, and
so deserve updating.

Signed-off-by: Chris Evich <[email protected]>
If ANSIBLE_VAULT_PASSWORD_FILE is set to an empty value, then ansible
will assume it's value is '.', and complain about the execute bit being
set on the playbook directory.

ansible/ansible#42960

Work around this by defaulting the value to '/dev/null' instead of
empty.

Signed-off-by: Chris Evich <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant