Closed pablodav closed 7 years ago
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.
:white_check_mark: pablodav
:x: pestigarribia
Hello, I have signed it, some commits are with pestigarribia account, I have modified only one to pablodav but in both is me.
El sáb., 4 de feb. de 2017 11:15, CLAassistant notifications@github.com escribió:
[image: CLA assistant check] https://cla-assistant.io/Graylog2/graylog-ansible-role?pullRequest=61 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement https://cla-assistant.io/Graylog2/graylog-ansible-role?pullRequest=61 before we can accept your contribution.
pestigarribia seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/#commits-are-not-linked-to-any-user .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Graylog2/graylog-ansible-role/pull/61#issuecomment-277448445, or mute the thread https://github.com/notifications/unsubscribe-auth/AEQPn1NRtG1Q5AwLyz7efh6JBp7ODS7Nks5rZIfggaJpZM4L3KSY .
-- Pablo.
Thank you!
Hello,
I have been working to setup graylog2 properly and find that there was some missing and important notes in the README, also I have added an option to install java from openjdk and not from oracle (keeping the default behaviour as it was in the role, then documented on the README too).
Resume:
Added graylog_install_java var
Added README example to install openjdk instead of Oracle java
fix issues with ansible 2.2 (Added examples on the README)
fix issues with elasticsearch version, explicit set the es version and java version (Documented missing information in the README about graylog incompatibility with elasticsearch 5.x)
fix issues with java version from elasticsearch role (Documented on README)
added more useful vars when you are explicit with the usage of roles. (On README)
fix wrong example for var web_endpoint_uri on README. (missing information about how webclient works)