search

Graylog2 / graylog-ansible-role

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

added Pin priority for elasticsearch to 2.* to avoid upgrades to 5.x version of elasticsearch #73

Closed pablodav closed 7 years ago

pablodav commented 7 years ago

This will ensure elasticsearch will not upgrade even when some other role (like logstash) added 5.x repository of elasticsearch. It's possible in this way to safetly install logstash with graylog to add more inputs to graylog.

CLAassistant commented 7 years ago

CLA assistant check
All committers have signed the CLA.

pablodav commented 7 years ago

I have signed the CLA, check it's signed with my user pablodav, but for some reason it recognizes old commits (already merged on previous pull requests) with user pestigarribia (it doesn't exists on github).

mariussturm commented 7 years ago

Hi, thanks for the contribution! I think only the last commit is necessary for the Pin priority or? Could you rebase your branch or open a new PR with only the commit that should be merged?

pablodav commented 7 years ago

Done a rebase helped and seems to be more clear now.

mariussturm commented 7 years ago

LGTM :+1: , thank you!