radekg / terraform-provisioner-ansible

Ansible with Terraform 0.14.x
Apache License 2.0
572 stars 100 forks source link

install curl (may not be present in minimal server editions) #142

Closed lestephane closed 5 years ago

lestephane commented 5 years ago

fixes an error I encountered while using the ubuntu minimal server edition:

/tmp/tf-ansible-installer: 24: /tmp/tf-ansible-installer: curl: not found /tmp/tf-ansible-installer: 27: /tmp/tf-ansible-installer: pip: not found

More generally, consider using set -euo pipefail, otherwise the script does not stop as you see here in line 24 because the error is 'swallowed' by the use of a pipe.

radekg commented 5 years ago

Hi @lestephane, thank you for the PR. I've added set -euo pipefail as suggested.