search

sstarcher / docker-sensu

Dockerized Sensu - Client, Server, Api
MIT License
103 stars 72 forks source link

RUNTIME_INSTALL broken #5

Closed bscott closed 9 years ago

bscott commented 9 years ago

the bin/install script assumes the git download url ends with "master.tar.gz" when in fact it's "archive/[version].tar.gz"

Sensu_sensu-srv_1 | 5-10-28T05:24:39.854859970Z --2015-10-28 05:23:48--  https://github.com/sensu-plugins/sensu-plugins-sensu-plugins-disk-checks/archive/master.tar.gz
Sensu_sensu-srv_1 | 5-10-28T05:24:39.854859970Z Resolving github.com (github.com)... 192.30.252.131
Sensu_sensu-srv_1 | 5-10-28T05:24:39.854859970Z Connecting to github.com (github.com)|192.30.252.131|:443... connected.
Sensu_sensu-srv_1 | 5-10-28T05:24:39.854859970Z HTTP request sent, awaiting response... 404 Not Found
Sensu_sensu-srv_1 | 2015-10-28T05:24:39.854859970Z 2015-10-28 05:23:48 ERROR 404: Not Found.

example:

https://github.com/sensu-plugins/sensu-plugins-network-checks/archive/0.0.5.tar.gz
sstarcher commented 9 years ago

master.tar.gz exists. The issue is when using RUNTIME_INSTALL you don't need to specify 'sensu-plugins' RUNTIME_INSTALL disk-checks Your url has sensu-plugins specified twice

https://github.com/sensu-plugins/sensu-plugins-sensu-plugins-disk-checks/archive/master.tar.gz
https://github.com/sensu-plugins/sensu-plugins-disk-checks/archive/master.tar.gz
bscott commented 9 years ago

Thanks!