search

Graylog2 / graylog-ansible-role

Ansible role which installs and configures Graylog
Apache License 2.0
212 stars 126 forks source link

Make the uri test more flexible #88

Closed evrardjp closed 5 years ago

evrardjp commented 6 years ago

By default, the deploy is testing the web_listen_uri, on the default url http://127.0.0.1:9000/.

If a deployer doesn't listen on this IP (only listening to a management interface for example), the deploy would fail, unless the testing is disabled.

We should keep having testing, but having the opportunity to test with any URL.

On top of that, the boolean to run the test or not is inverted, because it lead to confusions: (when graylog_not_testing is set to True, the test ran). Now the tests runs when graylog_testing is true (which is the default).

CLAassistant commented 6 years ago

CLA assistant check
All committers have signed the CLA.

evrardjp commented 6 years ago

The failure in travis is completely independant of this patch.