sensu / sensu-ansible

An Ansible role to deploy a fully dynamic Sensu stack!
https://ansible-sensu.readthedocs.io
MIT License
126 stars 96 forks source link

Ubuntu 14.04LTS no https transport #90

Closed kevit closed 7 years ago

kevit commented 7 years ago

FAILED! => {"changed": false, "failed": true, "module_stderr": "Shared connection to 213.196.39.216 closed.\r\n", "module_stdout": "Traceback (most recent call last):\r\n File \"/tmp/ansible_ZI1hPK/ansible_module_apt_repository.py\", line 565, in <module>\r\n main()\r\n File \"/tmp/ansible_ZI1hPK/ansible_module_apt_repository.py\", line 553, in main\r\n cache.update()\r\n File \"/usr/lib/python2.7/dist-packages/apt/cache.py\", line 440, in update\r\n raise FetchFailedException(e)\r\napt.cache.FetchFailedException: E:The method driver /usr/lib/apt/methods/https could not be found.\r\n", "msg": "MODULE FAILURE", "rc": 0}

Next run TASK [cmacrae.sensu : Ensure Sensu is installed] ******************************************************************************************************************************************************************************************************************************************************************** fatal: []: FAILED! => {"changed": false, "failed": true, "msg": "No package matching 'sensu' is available"}

on the node root@box:~# apt-get update E: The method driver /usr/lib/apt/methods/https could not be found. N: Is the package apt-transport-https installed?

Ill update with the patch shortly

DavidVentura commented 7 years ago

you need to install ca-certificates and apt-transport-https

majormoses commented 7 years ago

IMHO If this is a requirement it should be installed, that's what CM does right?

smuth4 commented 7 years ago

This was actually fixed in #91, I was expecting Github to automagically close this.

majormoses commented 7 years ago

@smuth4 it did not close because of the missing "#" before the issue number in commit message.