search

Graylog2 / graylog-ansible-role

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

Cleanups & documentation #90

Closed Fra-nk closed 6 years ago

Fra-nk commented 6 years ago

Hi,

when having a look at the role I noticed a few things, which inconvenienced me. While most of these are (also) personal preference, I tried to align with the YAML standard as much as possible. Furthermore, this PR also includes minor documentation and CI changes, which came up in the meantime, e.g. wrong Debian version, some wrong strings in README, etc.

I wanted to get this PR "out of the way", before changing more fundamental things like making the storage directory configurable, etc.

For reviewing, I advice to look at each commit separately.

CLAassistant commented 6 years ago

CLA assistant check
All committers have signed the CLA.

mariussturm commented 6 years ago

Thanks for the PR! Looks good to me!