Graylog2 / graylog-ansible-role

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

various improvements #71

Closed danvaida closed 7 years ago

danvaida commented 7 years ago

@mariussturm no time for me right now to create a different branch for the build-badge to match Graylog's Travis account. Please excuse that :)

Although some of the Travis builds are failing(!), I would still suggest merging this PR as it greatly improves* the current state of the integration with Travis for running the tests.

If this PR is merged, I'll go ahead and create an issue to fix the tests that are failing (probably because of some docker+systemd quirk). I consider this issue to be out of the scope of this PR.

mariussturm commented 7 years ago

Dude, seriously, please split your PRs up into smaller chunks! There is stuff for at least 3 PRs in here. One for the Travis changes, one for the README update and one formatting PR. I will merge these massive "various improvements" PRs the last time now.